Code review comment for lp:~gandelman-a/charm-helpers/sync_include_hints

Revision history for this message
Adam Gandelman (gandelman-a) wrote :

Hmm. Looks like they are running by default:

nosetests -v tests/
...
tests.payload.test_execd.ExecDTestCase.test_execd_submodule_list ... ok
It properly branches the correct helpers branch ... ok
It correctly finds the correct install path within a charm ... ok
It ensures all subdirectories of a parent are python importable ... ok
It extracts multiple options from an included item ... ok
...
Ran 324 tests in 3.461s (with tests/tools/)

vs.

Ran 306 tests in 1.600s (after removing tests/tools)

However, tools/ was not being included in the coverage report, though. I've updated setup.cfg to include it.

« Back to merge proposal