department-of-veterans-affairs / vets-website

Frontend for VA.gov
Creative Commons Zero v1.0 Universal
242 stars 126 forks source link

VACMS-16191 Add test coverage for Resources & Support app #33122

Closed randimays closed 6 days ago

randimays commented 1 week ago

Are you removing, renaming or moving a folder in this PR?

:warning: TeamSites :warning:

Did you change site-wide styles, platform utilities or other infrastructure?

Summary

The Resources & Support app had some opportunities for unit test coverage and a Cypress suite.

Unit tests

Notes:

Before

284718072-5f8de440-6b6a-41e7-a820-7465cc82373c

After

Screenshot 2024-11-20 at 2 09 03 PM

Cypress

Our Cypress suite has:

Related issue(s)

https://github.com/department-of-veterans-affairs/va.gov-cms/issues/16191

Testing done

Tested the app manually locally and ran all unit and Cypress tests. No functionality should have changed so I didn't add screenshots here.

What areas of the site does it impact?

/resources and /resources/search

Quality Assurance & Testing

Error Handling

jilladams commented 1 week ago

@randimays this is the only open Sitewide vets-website PR I see right now. Once you move out of Draft, can you confirm if Curt's notes in devops channel are correct that you are able to merge with just a team approval, and not Platform approval?

randimays commented 6 days ago

@jilladams Confirmed that I was able to merge this with only Chris's approval because the changes were siloed to the src/applications folder. A lot of our work crosses over into the src/platform folder so we have to wait on Platform when that happens (example).