During a recent review of the ci process, it was observed that the code coverage requirements of 85% was not actually being enforced. Instead it was always passing due to seeing 0 lines to cover hence 100%.
With #2136 this behaviour was corrected to use the collated coverage for all appraisals however, not all gems are meeting the required 85% coverage. To Address this a minimum coverage override has been added for the offending gems. This issue will track this work
In some cases such as:
This can be achieved by adjusting Appraisals logic.
But others tests need to be written which will be tracked via sub-issues with the goal being the removal of the overrides.
During a recent review of the ci process, it was observed that the code coverage requirements of 85% was not actually being enforced. Instead it was always passing due to seeing 0 lines to cover hence 100%.
With #2136 this behaviour was corrected to use the collated coverage for all appraisals however, not all gems are meeting the required 85% coverage. To Address this a minimum coverage override has been added for the offending gems. This issue will track this work
In some cases such as:
This can be achieved by adjusting Appraisals logic.
But others tests need to be written which will be tracked via sub-issues with the goal being the removal of the overrides.