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

feat: print info statements after each test step instead of being at the end #673

Closed
wants to merge 2 commits into from

Conversation

fokion
Copy link
Contributor

@fokion fokion commented May 31, 2023

Have the info statements displayed as the tests are running instead of being in the end

Screenshot 2023-05-31 at 09 43 28

Have the info statements displayed as the tests are running instead of being in the end

Signed-off-by: Fokion Sotiropoulos <fokion.s@gmail.com>
Signed-off-by: Fokion Sotiropoulos <fokion.s@gmail.com>
@fokion fokion changed the title update the printing of the info statements feat: print info statements after each test step instead of being at the end May 31, 2023
@kilianpaquier
Copy link
Contributor

Hello, this need should be taken into account with this PR #670

@fokion
Copy link
Contributor Author

fokion commented May 31, 2023

@kilianpaquier when you merge your PR I can rebase and have any changes to it applied 😄

@kilianpaquier
Copy link
Contributor

Hey @fokion, what I meant is that it's now covered here 0b4842d#diff-3684f1fb3b7ef3bc47130ad281d0dd58cd2f8a32ddb2a3d1a7d71f604b566ecbR391

step information are printed in verbose more directly after the step. And in non-verbose mode at the end of the case and only if there're errors 0b4842d#diff-52a3166d581f50aab63aa6ddfca34f53649c94dc7614f3b320adfe9d541329ceR173

@fokion
Copy link
Contributor Author

fokion commented May 31, 2023

awesome 😄 so I can close this PR then

@fokion fokion closed this May 31, 2023
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

Successfully merging this pull request may close these issues.

2 participants