Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The test reporter is reporting wrong counts #23

Closed
leostera opened this issue Mar 29, 2023 · 0 comments · Fixed by #33
Closed

The test reporter is reporting wrong counts #23

leostera opened this issue Mar 29, 2023 · 0 comments · Fixed by #33
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@leostera
Copy link
Member

When running the command below, we see all the right things are being handled BUT the counts of the different events are off.

We have built 5 things, and we have cached 5 things, but there is no mention of the 11 tests that were ran and cached.

; warp test --public-store-metadata-url http://localhost:9000 --force-redownload ./test/verl_SUITE.erl
     Started http://localhost:9000/tricorder/beam/manifest.json
    Readying http://localhost:9000/tricorder/beam/manifest.json
       Ready http://localhost:9000/tricorder/beam/manifest.json
   Cache-hit https://rules.warp.build/toolchains/openssl  
   Cache-hit https://rules.warp.build/toolchains/erlang  
   Cache-hit src/verl_parser.erl  
   Cache-hit src/verl.erl  
        PASS ./test/verl_SUITE.erl parse_test (CACHED)
        PASS ./test/verl_SUITE.erl compare_test (CACHED)
        PASS ./test/verl_SUITE.erl between_test (CACHED)
        PASS ./test/verl_SUITE.erl compile_requirement_test (CACHED)
        PASS ./test/verl_SUITE.erl ./test/verl_SUITE.erl (CACHED)
        PASS ./test/verl_SUITE.erl lte_test (CACHED)
        PASS ./test/verl_SUITE.erl lt_test (CACHED)
        PASS ./test/verl_SUITE.erl gt_test (CACHED)
        PASS ./test/verl_SUITE.erl is_match_test (CACHED)
        PASS ./test/verl_SUITE.erl parse_requirement_test (CACHED)
        PASS ./test/verl_SUITE.erl eq_test (CACHED)
        PASS ./test/verl_SUITE.erl gte_test (CACHED)
    Finished ./test/verl_SUITE.erl in 865ms (5 built, 5 cached)

What we'd expect to see would be:

    Finished ./test/verl_SUITE.erl in 865ms (11 passed, 0 errors, 5 built, 16 cached)

And also, one specific line where we built a library (the test/verl_SUITE.erl library) reads:

        PASS ./test/verl_SUITE.erl ./test/verl_SUITE.erl (CACHED)

where it should read:

   Cache-hit ./test/verl_SUITE.erl
@leostera leostera added enhancement New feature or request good first issue Good for newcomers labels Mar 29, 2023
@Enkronan Enkronan linked a pull request Apr 4, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants