PaulMcInnis / JobFunnel

Scrape job websites into a single spreadsheet with no duplicates.
MIT License
1.83k stars 214 forks source link

-Added tests for cli.py #116

Closed thebigG closed 3 years ago

thebigG commented 3 years ago

Added tests for cli.py

Hello everyone,

hope you are all doing well!

Description

I have added unit testing for cli.py. I tried my best to make the unit tests as well-scoped as possible. This approach is slow, but I prefer it because it makes tests easy to read and it makes points of failure very easy to spot.

I took the liberty of creating an incorrect version of our test yaml file called incorrect_test_config.py under our tests/data folder. This way we can ensure that JobFunnel is behaving how it's supposed to when given incorrect data in a configuration file. Hopefully this change makes sense.

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.

Checklist:

Please mark any boxes that have been completed.