Improve the unit tests coverage
Bug #1978740 reported by
Gabriel Cocenza
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Juju Lint |
Fix Released
|
High
|
Martin Kalcok | ||
Bug Description
juju-lint it's only with 65% of unit tests coverage that it's too low for an important project.
We should put some effort on get this percentage close to 100% and maybe add the flag "--cov-fail-under 100" to fail unit test if it's not all covered.
Related branches
~martin-kalcok/juju-lint:unit-tests
- Eric Chen: Approve
- Gabriel Cocenza: Approve
- BootStack Reviewers: Pending requested
-
Diff: 2367 lines (+1793/-195)17 files modifiedjujulint/check_spaces.py (+1/-3)
jujulint/cli.py (+1/-1)
jujulint/cloud.py (+3/-4)
jujulint/k8s.py (+9/-19)
jujulint/lint.py (+31/-18)
jujulint/logging.py (+15/-19)
jujulint/openstack.py (+1/-5)
tests/conftest.py (+12/-2)
tests/requirements.txt (+1/-2)
tests/test_check_spaces.py (+347/-0)
tests/test_cli.py (+377/-3)
tests/test_cloud.py (+495/-112)
tests/test_jujulint.py (+225/-7)
tests/test_k8s.py (+29/-0)
tests/test_logging.py (+216/-0)
tests/test_openstack.py (+29/-0)
tox.ini (+1/-0)
| tags: | added: bseng-160 |
| Changed in juju-lint: | |
| status: | New → In Progress |
| assignee: | nobody → Martin Kalcok (martin-kalcok) |
| importance: | Undecided → High |
| Changed in juju-lint: | |
| milestone: | none → 1.0.4 |
| status: | In Progress → Fix Committed |
| Changed in juju-lint: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
