Skip to content

Add assertion error message#4810

Merged
sksamuel merged 8 commits intokotest:masterfrom
belljun3395:feature/add-assertion-error-message
Mar 30, 2025
Merged

Add assertion error message#4810
sksamuel merged 8 commits intokotest:masterfrom
belljun3395:feature/add-assertion-error-message

Conversation

@belljun3395
Copy link
Copy Markdown
Contributor

@belljun3395 belljun3395 commented Mar 16, 2025

Fixes #4785

Generate the message for AssertionError in the same way as it is generated in MultiAssertionError

@belljun3395 belljun3395 requested a review from a team as a code owner March 16, 2025 02:45
@belljun3395 belljun3395 reopened this Mar 22, 2025
@sksamuel sksamuel enabled auto-merge March 29, 2025 18:05
@sksamuel
Copy link
Copy Markdown
Member

👍🏻

@sksamuel
Copy link
Copy Markdown
Member

Need to run apiDump

@sksamuel
Copy link
Copy Markdown
Member

nm I've done it :)

@belljun3395
Copy link
Copy Markdown
Contributor Author

@sksamuel Thank you 😄 Is there anything else I should do?

@sksamuel
Copy link
Copy Markdown
Member

@sksamuel Thank you 😄 Is there anything else I should do?

Nope, I'll get this merged tonight :)

@sksamuel sksamuel added this pull request to the merge queue Mar 30, 2025
Merged via the queue into kotest:master with commit d0531ed Mar 30, 2025
7 checks passed
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.

Wrongly reported location of failed assertion inside assertSoftly

2 participants