Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
5 tasks done
ghost opened this issue Dec 3, 2021 · 3 comments
Open
5 tasks done

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

ghost opened this issue Dec 3, 2021 · 3 comments
Labels
E: 2 Effort: 2 Story-Points S: Major Severity: Major - important concern user story User Story

Comments

@ghost
Copy link

ghost commented Dec 3, 2021

User Story

Essential components

  • Title describes the story
  • Stakeholder type is identified
  • Outcome is described
  • Rationale is explicit
  • Acceptance criteria are verifiable and from the perspective of the stakeholder

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

  • Test coverage metric evaluation is available in the automated build process that measures the amount of coverage the tests have over source code.

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

@ghost ghost added the user story User Story label Dec 3, 2021
@jody
Copy link
Contributor

jody commented Dec 4, 2021

"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".

@jody jody assigned ghost Dec 4, 2021
@ghost
Copy link
Author

ghost commented Dec 5, 2021

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

@jody jody added E: 2 Effort: 2 Story-Points S: Major Severity: Major - important concern labels Dec 5, 2021
@jody jody changed the title [UserStory] add test coverage feature [UserStory] Re-establish test coverage metrics assessment in build Dec 5, 2021
@jody
Copy link
Contributor

jody commented Dec 5, 2021

Thanks! This is a relatively high priority issue!!

(I made a few wording updates.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E: 2 Effort: 2 Story-Points S: Major Severity: Major - important concern user story User Story
Projects
Development

No branches or pull requests

1 participant