MetroCS / redistricting

Experimentation with geopolitical redistricting
GNU Affero General Public License v3.0
5 stars 75 forks source link

[UserStory] Re-establish test coverage metrics assessment in build #141

Open ghost opened 2 years ago

ghost commented 2 years ago

User Story

Essential components

Story

As a Maintainer I want a working test coverage evaluation as part of the build process so that measures of the adequacy of tests for the code base is evaluated.

Acceptance Criteria

Scenario 1
Given ant coverage command has been executed, when viewing the associated report then accurate test coverage evaluation is displayed.

Supporting Information

Test coverage is no longer evaluated. This appears to be a consequence of changes in test classes made to migrate from JUnit 5 Vintage to JUnit 5 Jupiter resulting in the Jacoco tool evaluation no longer working as intended.

Dependencies

Depends On

Dependents

jody commented 2 years ago

"contributor" is not one of the specified stakeholder types in Vision and Stakeholders.

The story rationale uses an undefined term, "test range".

Acceptance criterion Scenario 1 has vague specification: "when reports html file is opened".

ghost commented 2 years ago

I've updated to maintainer, removed the undefined term, and included more detail in acceptance criteria and scenario 1.

jody commented 2 years ago

Thanks! This is a relatively high priority issue!!

(I made a few wording updates.)