Skip to content
Snippets Groups Projects

Patterns for 5000000 blocks

Merged Dariusz Kędzierski requested to merge dk-patterns-5000000 into master
1 unresolved thread

Added patterns and modified test for 5000000 blocks. Test are in separate directory.

Merge request reports

Merged by Bartek WronaBartek Wrona 4 years ago (Jul 10, 2020 1:16pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
1 import os
  • reopened

  • Ok made small rework:

    • Patterns and test definitions moved to proper directories depending on block number

    • Benchmark moved to proper directory

    • Test runners modified to take directory name as a parameter

    @bwrona @Ickiewicz

    Examples: ./run_api_tests.sh https://api.hive.blog full_sync contain for tests in full_sync dir.

    ./run_api_tests.sh https://api.hive.blog 5000000 contain for tests in 5000000 dir.

    Edited by Dariusz Kędzierski
  • @dkedzierski remove scripts file ( *.py, *.sh) from 5000000 directory. Can we rename to 5000000 to 5mln ? I don't like to check how many zeros are there

    Edited by Marcin
  • @dkedzierski tests do not pass

    ./run_api_tests.sh https://api.hive.blog full_sync contain
    ERROR:Validator/Error details:Extractor: Extractor Type: jsonpath_mini,  Query: "result", Templated?: False
    Expected is templated, raw value: full_sync/$api/$method
    Test Group condenser_api FAILED: : 18/22 Tests Passed!
    
    ERROR:Validator/Error details:Extractor: Extractor Type: jsonpath_mini,  Query: "result", Templated?: False
    Expected is templated, raw value: full_sync/$api/$method
    Test Group tags_api FAILED: : 7/11 Tests Passed!

    Similar effect with ./run_api_tests.sh https://api.hive.blog full_sync equal

    Edited by Marcin
  • I'll take a look. Most likely some values have changed in time.

  • added 1 commit

    • ba0466dc - Fixes in full_sync tests, json schema added

    Compare with previous version

  • @bwrona @Ickiewicz please take a look into last commit

  • merged

  • Bartek Wrona mentioned in commit 1710d711

    mentioned in commit 1710d711

  • Please register or sign in to reply
    Loading