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

assertionEnded is not called on reporters #1264

Closed
berkus opened this issue Apr 25, 2018 · 2 comments
Closed

assertionEnded is not called on reporters #1264

berkus opened this issue Apr 25, 2018 · 2 comments

Comments

@berkus
Copy link
Contributor

berkus commented Apr 25, 2018

Description

I included TAPReporter, enabled it via cmdline but report always comes out as 1..0 with nothing else. After debugging for a while I found that assertionEnded method of the reporter is never called, and this is crucial place for actually counting the results.

Any ideas why that might be?

Steps to reproduce

  1. #include "catch_reporters_tap.hpp"
  2. test -r tap

Extra information

  • Catch version: v2.2.2
  • Operating System: macOS
  • Compiler+version: Apple LLVM version 9.1.0 (clang-902.0.39.1)
    Target: x86_64-apple-darwin17.5.0
@berkus
Copy link
Contributor Author

berkus commented Apr 25, 2018

I see that void RunContext::assertionPassed() { does not call assertionEnded() with success.

@berkus
Copy link
Contributor Author

berkus commented Apr 26, 2018

And the fix was

        ConfigData data;
        data.showSuccessfulTests = true; // important for TAPReporter

I guess TAPReporter could set this internally in constructor, but i didn't test that yet.

@berkus berkus closed this as completed Apr 26, 2018
horenmar added a commit that referenced this issue Jul 14, 2018
By opting the JUnit and XML reporters into it, we no longer run
into problem where they underreport the results without `-s` flag.

Related to #1264, #1267, #1310
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant