-
Notifications
You must be signed in to change notification settings - Fork 186
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
fix: fix test report in case of multiline test name #535
fix: fix test report in case of multiline test name #535
Conversation
@@ -1120,6 +1120,21 @@ const skipExceptionMessageJsonOuput = [ | |||
'type': 'testStart', | |||
'time': 2757 | |||
}, | |||
{ | |||
'test': { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we move this test after id: 28 so that they are in order? Also we should update the testDone event to have a different timestamp
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @felangel for reviewing. Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dear Reviewers @felangel @tomarra @erickzanardo @dewinjm , if you have a bit littre time to reviews my 4 PRs. We need them in our projet. Thank you in advance
17bb3a3
to
b8f8276
Compare
b8f8276
to
63365c8
Compare
63365c8
to
e9bfc27
Compare
b77c97a
to
ecb384f
Compare
Description
This PR fix a bug in case of mutliline test name. The truncate algorithm give a too short name and thus not helpful to find the test
Type of Change