Skip to content
This repository has been archived by the owner on Aug 13, 2019. It is now read-only.

Add tests for tsdb cli tool #673

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Add tests for tsdb cli tool #673

wants to merge 13 commits into from

Commits on Aug 9, 2019

  1. Primitive printing of block in CLI unit test 

    - Moving necessary functions into different package (only copy)
- Output not checked yet

Signed-off-by: obitech <[email protected]>
    @obitech
    obitech committed Aug 9, 2019
    Configuration menu
    Copy the full SHA
    dd0ee20 View commit details
    Browse the repository at this point in the history

  2. Setup Example output test 

    Signed-off-by: obitech <[email protected]>
    @obitech
    obitech committed Aug 9, 2019
    Configuration menu
    Copy the full SHA
    0957af6 View commit details
    Browse the repository at this point in the history

  3. Let printBlocks() take a writer for better testability, write unit test 

    Signed-off-by: obitech <[email protected]>
    @obitech
    obitech committed Aug 9, 2019
    Configuration menu
    Copy the full SHA
    4cad92c View commit details
    Browse the repository at this point in the history

  4. Adjust syntax according to newer commits 

    Signed-off-by: obitech <[email protected]>
    @obitech
    obitech committed Aug 9, 2019
    Configuration menu
    Copy the full SHA
    c18ea5e View commit details
    Browse the repository at this point in the history

  5. Fix gofmt in main_test 

    Signed-off-by: obitech <[email protected]>
    @obitech
    obitech committed Aug 9, 2019
    Configuration menu
    Copy the full SHA
    58712f9 View commit details
    Browse the repository at this point in the history

  6. Fix circular import 

    Signed-off-by: obitech <[email protected]>
    @obitech
    obitech committed Aug 9, 2019
    Configuration menu
    Copy the full SHA
    eebcb60 View commit details
    Browse the repository at this point in the history

  7. Apply suggestions from code review 

    Co-Authored-By: Krasi Georgiev <[email protected]>
Signed-off-by: obitech <[email protected]>
    @obitech @krasi-georgiev
    obitech and krasi-georgiev committed Aug 9, 2019
    Configuration menu
    Copy the full SHA
    e5c9d53 View commit details
    Browse the repository at this point in the history

  8. Remove uncessary db.Open(), add full stops in comments 

    Signed-off-by: obitech <[email protected]>
    @obitech
    obitech committed Aug 9, 2019
    Configuration menu
    Copy the full SHA
    661fa1c View commit details
    Browse the repository at this point in the history

  9. Better testability of analyzeBlock command, start tests 

    - Factor out `extractBlock()` to retrieve specific block from db.Blocks()
- Write tests for said function
- Simple, non-output test for `analyzeBlock()` command

Signed-off-by: obitech <[email protected]>
    @obitech
    obitech committed Aug 9, 2019
    Configuration menu
    Copy the full SHA
    7a61b29 View commit details
    Browse the repository at this point in the history

  10. Add tests for analyzeBlock CLI command 

    Signed-off-by: obitech <[email protected]>
    @obitech
    obitech committed Aug 9, 2019
    Configuration menu
    Copy the full SHA
    67c92d8 View commit details
    Browse the repository at this point in the history

Commits on Aug 11, 2019

  1. Move testing functions back into tsdb package, remove duplicates from… 

    … existing test files

Signed-off-by: obitech <[email protected]>
    @obitech
    obitech committed Aug 11, 2019
    Configuration menu
    Copy the full SHA
    4dd7e7f View commit details
    Browse the repository at this point in the history

  2. Rename Mock{CreateBlock,GenSeries} to CreateBlock, GenSeries 

    Signed-off-by: obitech <[email protected]>
    @obitech
    obitech committed Aug 11, 2019
    Configuration menu
    Copy the full SHA
    5dad441 View commit details
    Browse the repository at this point in the history

  3. Fix comments for CreateBlock and GenSeries 

    Signed-off-by: obitech <[email protected]>
    @obitech
    obitech committed Aug 11, 2019
    Configuration menu
    Copy the full SHA
    c6f75f5 View commit details
    Browse the repository at this point in the history