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.

Sign up for GitHub

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

Jump to bottom

[WIP] Improve reporting #171

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

[WIP] Improve reporting #171

wants to merge 7 commits into from

Conversation

ozars
Copy link
Contributor

@ozars ozars commented Aug 18, 2018 •
edited
Loading

Features discussed in #166.

  • Prepend suite name while reporting tests.
  • Create test results for test cases, separate from individual tests.
  • Report errors in unchecked setups in test case results. Also report subsequent tests as failed due to failure in test case setups.
  • Unchecked test case teardowns need to override the fork mode to CK_NOFORK, so that reported failures do not kill the unit test program.
  • Report errors in unchecked teardowns in test case results. Tests included in tests cases will report their own results, independent from failure of unchecked teardowns.
  • Update documentation to reflect changes.

@ozars
Ignore ctags file
3fc746e
@ozars
Replace tabs with 8 spaces
d03714b
@ozars
Add suite name parameter to TestResult
c4f8c1c 
Pass suite pointer or suite name parameters to functions which eventually
create TestResult object
@ozars
Add suite name to default test printing format
7fa65af 
Suite name is prepended before test case now.

Unit tests are also updated accordingly.
@ozars
Fix typo in test for the expected output
fc9ddaa
@ozars
Add suite name to TAP output
e8bf9e6
@ozars
Add suite name to subunit output
06d1d70
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ozars