coveralls
commented
6 years ago Pull Request Test Coverage Report for Build 174
- 11 of 19 (57.89%) changed or added relevant lines in 6 files are covered.
- No unchanged relevant lines lost coverage.
- Overall coverage remained the same at 10.619%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
|---|---|---|---|---|---|
| packages/acceptit-Core.package/ACCompiler.class/instance/compile.in.classified.notifying.ifFail..st | 0 | 1 | 0.0% | ||
| packages/acceptit-Core.package/ACScenarioCompiler.class/instance/buildCode..st | 0 | 1 | 0.0% | ||
| packages/acceptit-Core.package/ACScenarioParser.class/instance/parseTree..st | 0 | 6 | 0.0% | ||
| <!-- | Total: | 11 | 19 | 57.89% | --> |
| Totals | |
|---|---|
| Change from base Build 170: | 0.0% |
| Covered Lines: | 103 |
| Relevant Lines: | 970 |
Removing the old method with 4 variables in favor of a new one with only 2 that has way cleaner code (small methods etc.) and is at least partially unit tested