radeklat / delfino

A toolbox of command line helper script, wrapping tools used during Python development.
MIT License
12 stars 3 forks source link

Fix test detection when debug flag is not provided #15

Closed jacksmith15 closed 2 years ago

jacksmith15 commented 2 years ago

Currently if the --debug is not provided to delfino test, then an empty positional argument is passed to pytest. This is interpreted as "current directory" by pytest, which means that each test type runs the full suite of tests available under the project root directory (assuming the command is run from the root).

This PR updates the pytest invocation to omit the -s argument if --debug is not supplied.

codecov[bot] commented 2 years ago

Codecov Report

Merging #15 (83c4554) into main (627e509) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   49.46%   49.46%           
=======================================
  Files          19       19           
  Lines         659      659           
  Branches       87       87           
=======================================
  Hits          326      326           
  Misses        330      330           
  Partials        3        3           
Flag Coverage Δ
integration_tests 6.06% <ø> (-43.25%) :arrow_down:
total 49.31% <ø> (ø)
unit_tests 49.31% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/delfino/commands/test.py 41.55% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 627e509...83c4554. Read the comment docs.