Coverage tests have several errors

[Michael McCune]

while running the coverage tests there are several errors that prevent the tests from completing. these errors seem to be based around the functionality of the tools/coverage.sh script file.

during the first stage of the coverage tests, the script file will attempt to harvest the test results. this phase shows many errors relating to missing plugins. the failure of this phase causes the resulting phases in the script to fail as they return bad results from the output.

steps to reproduce:

1. clone a fresh sahara repo
2. run `tox -e cover`

results:

many errors (included in attached file) with the final result being

    Ran 926 tests in 32.171s (+0.191s)
    FAILED (id=2, failures=106, skips=5)
    error: testr failed (1)
    Allowed to introduce missing lines : 4
    Missing lines in master : to
    Missing lines in proposed change : to
    /home/mike/tmp/sahara/tools/cover.sh: line 52: [: to: integer expression expected
    Name Stmts Miss Cover
    Please write more unit tests, we should keep our test coverage :(
    ERROR: InvocationError: '/home/mike/tmp/sahara/tools/cover.sh'

recommend refactoring the coverage tests to be more resilient and fix these errors.

[Sergey Reshetnyak (sreshetniak)]

Workaround: need to remove old coverage reports:

$ rm -fr .coverage.*

[Michael McCune (mimccune)]

Sergey, i've done this from a fresh repo. even with those files removed i get the same results.

[Michael McCune (mimccune)]

I am going to close this out as it is no longer reproducible.