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

Fix test templates #3

Merged
merged 3 commits into from Dec 2, 2019
Merged

Conversation

@bennettscience
Copy link
Contributor

bennettscience commented Dec 2, 2019

Fixed a bug in the test template which called the part1() answer in test_part_2 instead of part2(). I also made the test more verbose in the terminal when run. Updated all test files.

+ bugfix to call challenge.part2() in test_part_2

Imported entire unittest library and set verbosity in `main()` to
give full results in the terminal when run.
@bennettscience

This comment has been minimized.

Copy link
Contributor Author

bennettscience commented Dec 2, 2019

Caught another bug where utils.load_input was hardcoded to load day_1.txt. Updated to load the day variable passed instead.

@lachlan-eagling

This comment has been minimized.

Copy link
Owner

lachlan-eagling commented Dec 2, 2019

Thanks again @bennettscience for fixing these up! 🙏 Really appreciate it.

@lachlan-eagling lachlan-eagling merged commit 9dd666a into lachlan-eagling:master Dec 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.