Skip to content

/ Catch2

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

Jump to bottom

The reporter interface needs new events for test case as a whole #2107

Open
horenmar opened this issue Nov 20, 2020 · 1 comment
Open

The reporter interface needs new events for test case as a whole #2107

horenmar opened this issue Nov 20, 2020 · 1 comment
Labels
Feature Request Good First Issue
Milestone
3.0

Comments

@horenmar
Copy link
Member

@horenmar horenmar commented Nov 20, 2020

The current testCaseStarting and testCaseEnding events can (and usually do) fire multiple times for a single TEST_CASE. This is because they fire every time TEST_CASE is entered/left, which happens multiple times when there are SECTIONs inside it. We want to provide reporter member function that is called when a TEST_CASE is about to be entered the first time, and when it has exited the last time.

Possible names testCaseBeginning and testCaseFinishing? Definitely open to better suggestions.
@horenmar horenmar added this to the 3.0 milestone Nov 20, 2020
@sudara
Copy link

@sudara sudara commented Feb 14, 2021

Maybe going with the explicit testCaseFirstEntered and testCaseLastExited would prevent confusion with the existing Starting/Ending events.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature Request Good First Issue
Projects
None yet
Milestone
3.0
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@sudara @horenmar