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.

Sign up for GitHub

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

Jump to bottom

Ticket #861 replace assertTrue with assertEquals #862

Closed

Simulant87 wants to merge 1 commit into stleary:master from Simulant87:861-1-improve-assertions

Contributor

Simulant87 commented Feb 22, 2024

Regarding #861

replace assertTrue with assertEquals statements to see the difference between actual and expected output on a failing test.

I can extend on this for all the test classes.


          stleary#861 replace assertTrue with assertEquals

46142d9

Owner

stleary commented Mar 3, 2024

No, thank you. The existing assertTrue expressions work, and I don't see a compelling reason to change them, especially across all tests.

stleary closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment