chore(bidi): add csv report #34107
chore(bidi): add csv report #34107
Conversation
Test results for "tests 1"7 flaky37396 passed, 650 skipped Merge workflow run. |
| row.push(test.expectedStatus); | ||
| row.push(test.outcome()); | ||
| const result = test.results.find(r => r.error); | ||
| const errorMessage = stripAnsi(result?.error?.message.replace(/\s+/g, ' ').trim().substring(0, 1024)); |
There was a problem hiding this comment.
@yury-s i've used the 1024 cap as an example. Maybe we can make that as well configurable?
Also does it produce a message without all the escape characters for colors? That would be pretty neat.
There was a problem hiding this comment.
Also it looks like I'm not able to specify the csv reporter locally with the --report argument. I assume this is something special for Bidi only. What could I use here to not have to run my own converter on the created json file?
There was a problem hiding this comment.
I intentionally added it in CI branch only in the config, feel free to add it in the other branch too:
There was a problem hiding this comment.
Maybe we can make that as well configurable?
Feel free to send PRs adding options you like.
Also does it produce a message without all the escape characters for colors? That would be pretty neat.
Yes, stripAnsi does that.
There was a problem hiding this comment.
I intentionally added it in CI branch only in the config, feel free to add it in the other branch too:
Created PR #34167.
Maybe we can make that as well configurable?
Feel free to send PRs adding options you like.
I'm going to wait until I feel it's necessary to add.
No description provided.