JobFunnel icon indicating copy to clipboard operation
JobFunnel copied to clipboard

Improve test coverage

Open PaulMcInnis opened this issue 3 years ago • 5 comments

Description

A cost of releasing version 3.0.0 is a signifcant loss of test coverage.

Steps to Reproduce

git clone https://github.com/PaulMcInnis/JobFunnel
cd JobFunnel
pytest

Expected behavior

We should cover Jobfunnel and scraper get/set methods with unit testing via pyest. Existing codecov was set to around 60%

Coverage

I've added Notes for all the modules that need testing coverage in a kanban here: https://github.com/PaulMcInnis/JobFunnel/projects/2

Please keep this up-to-date so that we don't duplicate eachother's work on upping the test coverage :+1:

PaulMcInnis avatar Sep 12 '20 01:09 PaulMcInnis

Now that we have a solid foundation to test on, I'll start testing. Thanks for organizing the testing suite, it looks very neat now :+1:

Looked at the new readme and I'm wondering why there isn't instructions for developers to install JobFunnel in-palce like we had before?

thebigG avatar Sep 12 '20 01:09 thebigG

Just decided it was not necessesary, since that information is common to all python packages, available as pipe install -e

PaulMcInnis avatar Sep 12 '20 01:09 PaulMcInnis

In general I've stripped down the readme a lot in the interest of keeping it very simple for a new user to understand vs providing extra information, even if it's kinda useful like the in-place command

PaulMcInnis avatar Sep 12 '20 01:09 PaulMcInnis

That makes sense. A long readme can be intimidating for users who just want to use the tool.

thebigG avatar Sep 12 '20 01:09 thebigG

I'll leave this tagged help-wanted, jic others want to help out as well. I'll help with this as well after I get do 3.0.1 milestone.

I've added a kanban (todo/inprogress/done) thing over here so we can keep track of what people are working on so we dont duplicate eachother's work: https://github.com/PaulMcInnis/JobFunnel/projects/2

PaulMcInnis avatar Sep 13 '20 01:09 PaulMcInnis