Skip to content

Conversation

@timmywil
Copy link
Member

Summary

3.x version of #5443

Add the actual and expected messages to the reporter.

Also, I realized that test.errors contained all the failed assertions and test.assertions contained only passed ones, so I was able to clean up that code a bit.

Checklist

@timmywil timmywil requested a review from mgol March 14, 2024 00:17
@timmywil timmywil added the Tests label Mar 14, 2024
@timmywil timmywil added this to the 3.7.2 milestone Mar 14, 2024
Copy link
Member

@mgol mgol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving with the same comment as in #5443 (review)

@timmywil timmywil merged commit f8be4a5 into jquery:3.x-stable Mar 14, 2024
@timmywil timmywil deleted the reporter-3.x branch March 14, 2024 13:05
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

2 participants