Skip to content

Include group context on failure output - #38

Merged
ValentinH merged 1 commit into
ValentinH:mainfrom
lennym:lennym/add-group-context
Mar 21, 2023
Merged

Include group context on failure output#38
ValentinH merged 1 commit into
ValentinH:mainfrom
lennym:lennym/add-group-context

Conversation

@lennym

@lennym lennym commented Mar 21, 2023

Copy link
Copy Markdown
Contributor

If there is a call to console.group around a log|warn|error then it is being swallowed and so when tests fail the output of console.group is not emitted to the console.

Include any group context in the flushUnexpectedConsoleCalls output so it can be debugged.

If there is a call to `console.group` around a `log|warn|error` then it is being swallowed and so when tests fail the output of `console.group` is not emitted to the console.

Include any group context in the `flushUnexpectedConsoleCalls` output so it can be debugged.
@lennym

lennym commented Mar 21, 2023

Copy link
Copy Markdown
Contributor Author

I made a slight mistake on my earlier PR in that it completely silenced any output from console.group in the failure logs. It made our tests impossible to debug. Apologies for that.

@ValentinH
ValentinH merged commit ad27c13 into ValentinH:main Mar 21, 2023
@ValentinH

Copy link
Copy Markdown
Owner

Thanks for the hotfix. I'll release it in around a hour.

@lennym
lennym deleted the lennym/add-group-context branch March 21, 2023 12:54
@ValentinH

Copy link
Copy Markdown
Owner

@lennym

lennym commented Mar 21, 2023

Copy link
Copy Markdown
Contributor Author

Fantastic. Thank you very much.

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