department-of-veterans-affairs / va.gov-team

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
283 stars 203 forks source link

QA Standards - Authenticated Experience - Profile - Authenticated Menu #65194

Open it-harrison opened 1 year ago

it-harrison commented 1 year ago

Status

[2024-04-18] The PW team hasn't taken action on this yet because the Header will be overhauled (this year?), and we'll take a look at the test percentage after such has happened.

General Information

VFS team name

Authenticated Experience - Profile

VFS product name

Authenticated Menu

VFS feature name

Authenticated Menu Modifications

Point of Contact/Reviewers

@it-harrison - Ian Harrison - QA

QA Standards

Regression Test Plan

Explanation of failure to meet standard (if applicable):

VFS Guidance

Platform directions

mtcA6 commented 1 year ago

@it-harrison is this launch blocking?

@adamwhitlock1 can you take a look at this and add points? Hoping we could pick this up in the next sprint, launch blocking or not.

it-harrison commented 1 year ago

@mtcA6 No, it is not launch-blocking. Only e2e tests are launch-blocking.

adamwhitlock1 commented 1 year ago

Statements that are uncovered by unit tests are not from the work we added, and is from previous work by site wide codeowners. We will bring this up with that team, but wont be updating their tests at this point.

mtcA6 commented 1 year ago

followed up on this thread to see if sitewide team is able to cover this

mtcA6 commented 1 year ago

Not owned by auth-exp but sitewide team also seems unlikely to create the test coverage

jilladams commented 1 year ago

Sitewide Public Websites team delivery manager chiming in here. Am I understanding correctly that this: Unit Test Coverage Lines %: 90 Functions %: 100 Statements %: 71.42 Branches %: 94.44

Is the problem? Is there someplace to see the test output to understand the gaps? If this is not launch blocking, between Travis's team and mine, we'll need to assess what's not covered, and prioritize accordingly.

jilladams commented 1 year ago

Also worth noting: the header is headed toward a rebuild that we'll be designing/researching in Q4, and and hoping to start build before EOY, so we may not prioritize unit tests for header code that will get overhauled under that project.

it-harrison commented 1 year ago

@jilladams yes, every category needs to be >= 75% but this is not currently launch blocking.

jilladams commented 1 year ago

Thanks @it-harrison . Where can we see how the report generated that #, in order to assess where we need to add coverage?

adamwhitlock1 commented 1 year ago

Platform is working on making this report public and published with deployments, but you can run the report locally by running this command on vets-website:

yarn test:coverage src/platform/**/*.unit.spec.js* --coverage-html

That will take about a minute to run, and then it will spit out a whole coverage folder in the root of vets-website. If you open the coverage/index.html file in your browser then you will see the coverages of every file/component and you can navigate into each file to see where coverage is missing.

Looks like this: Screenshot 2023-09-26 at 8 47 38 AM