-
-
Notifications
You must be signed in to change notification settings - Fork 405
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
Removing usage of yield in test_utils.py #234
Conversation
Works perfect now( with the fix also in #232 )
@wkerzendorf please take a look. |
|
@wkerzendorf The test_utils.py doesn't work currently even if we plug in the wrong values, because of the usage of yield and I think this PR solves it. I think we should merge it before we make any major changes to the files. What do you think ? |
Changes Unknown when pulling d20a187 on kaushik94:tuple_to_string into * on tardis-sn:master*. |
Removing usage of yield in test_utils.py
@kaushik94 well done! |
Thanks @wkerzendorf . |
Please take a look at #233. Also added a test for
species_tuple_to_string
. @wkerzendorf please take a look.