development-guide icon indicating copy to clipboard operation
development-guide copied to clipboard

Add test coverage for python scripts

Open hillaryj opened this issue 5 years ago • 1 comments

@hillaryj commented on Tue Oct 27 2020

This repository has no actual code testing, a .coveragerc file, or a test workflow, despite containing python scripts in the project_setup/scripts directory and requiring coverage in setup.py.

Acceptance Criteria

  • [ ] Add test coverage for scripts
  • [ ] Add .coveragerc file
  • [ ] Integrate with coveralls

@mcdonnnj commented on Wed Oct 28 2020

I think we should discuss separating the guide from the tools. It would be nice if the tools were their own package to install instead in my opinion. This would make it easier to add new tools and have a functional CI/CD setup for those tools.


@jsf9k commented on Wed Oct 28 2020

I think we should discuss separating the guide from the tools. It would be nice if the tools were their own package to install instead in my opinion. This would make it easier to add new tools and have a functional CI/CD setup for those tools.

I think this is a grand idea. Do you want to create an issue so we don't lose track of this?

hillaryj avatar Dec 01 '20 18:12 hillaryj

This is blocked pending the move of the tools and scripts mentioned in https://github.com/cisagov/development-guide/issues/33

hillaryj avatar Dec 01 '20 18:12 hillaryj