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

Removed assertion roullete (test smell) #406

Open

leofernandesmo wants to merge 2 commits into plutext:master from leofernandesmo:master

leofernandesmo commented Jul 26, 2020

The idea is to have just one assert per test method.
Besides:

I changed the signature of some methods to make them more expressive.
I removed some comments and print statements

leofernandesmo added 2 commits

July 26, 2020 12:13


          Removed print statements

bc9bf1e


          Removed assertion roullete

83795d3

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

Labels

None yet