-
Notifications
You must be signed in to change notification settings - Fork 154
Description
Codecov seems to be behaving oddly: the coverage report in #1332 shows a substantial loss in coverage (which was around 84% a few months ago). On top of that, the lines covered has nearly doubled, so there may be lines being counted towards the "coverage" but not actually needed for testing.
Also, the CI still seems to pass despite the lower coverage on both the patch and overall project.
We should double-check that:
- Coverage is calculating correctly
- Patch/project coverage below 80% actually fails the CI
- Testing-relevant lines in feat(ssh): Add SSH agent forwarding #1332 are being included, and non-relevant ones excluded
Some more context from #1327:
SSH Implementation PR:
@fabiovincenzi reported PR
#1332 for SSH agent forwarding is open, with additional tests added.
Coverage numbers discussed; some discrepancies between CI and codecov reporting noted.
Review requested from all, especially those deploying SSH features.
@jescalada to raise an issue to investigate coverage reporting accuracy.