Skip to content
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

Expected and actual are swapped #207

Merged
merged 1 commit into from
Jul 17, 2023
Merged

Conversation

planger
Copy link
Member

@planger planger commented Jul 17, 2023

This change merely changes the order of arguments to match the semantics of the assertion methods and thus produce correct output. This change however does not change the test itself.

This change merely changes the order of arguments to match the semantics
of the assertion methods and thus produce correct output. This change
however does not change the test itself.
@planger planger requested a review from tortmayr July 17, 2023 14:10
Copy link
Contributor

@tortmayr tortmayr left a comment

Choose a reason for hiding this comment

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

Looks good to me

@tortmayr tortmayr self-requested a review July 17, 2023 14:19
@planger planger merged commit 355cb4d into master Jul 17, 2023
5 checks passed
@planger planger deleted the fix-sourcemodelwatcher-test branch July 17, 2023 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants