Closed thebigG closed 4 years ago
Nice job! +1 I will merge it to the master. Any idea what you want to as the next task?
Thanks!
I really enjoyed playing around with testing so I'm thinking of writing tests for parser.py
next. It certainly looks a bit more complex but I think I can manage.
Pull Request Template
Hello everyone!
I tried describing my process as best I could below. Hope it's clear enough.
Description
Added tests on
jobfunnel/tests/test_validate.py
that test all functions injobfunnel/config/validate.py
. My thinking with tests was the following: Test each granular unit and then test those units as a whole. I also usedattr_list
intest_validate.py
as a global pool for all invalid data to be used in the tests. These new tests have been added:List any additional libraries that will be affected:
jobfunnel/tests/test_validate.py
List any developers that will be affected or those who you had merge conflicts with.Context of change
Please add options that are relevant and mark any boxes that apply.
Type of change
Please mark any boxes that apply.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.
test_validate.py
by itselfChecklist:
Please mark any boxes that have been completed.