CDCgov / prime-reportstream

ReportStream is a public intermediary tool for delivery of data between different parts of the healthcare ecosystem.
https://reportstream.cdc.gov
Creative Commons Zero v1.0 Universal
73 stars 40 forks source link

E2E Link Checker Doesn't Properly Check Internal Links #15823

Closed etanb closed 1 month ago

etanb commented 1 month ago

Describe the bug

Currently, we have an end-to-end test that cycles through all the <a> links on the public facing pages of ReportStream and makes sure it returns a 200 success response. The issue with this is that, in staging, where the test runs, our app doesn't properly handle 40* error responses in our routes outside of Azure. This has allowed a "broken" link to be uncaught by the link checker E2E test.

Impact on ReportStream

Via automation, we can't check if our internal facing links are broken.

Additional context

a couple of solutions:

Acceptance Criteria

etanb commented 1 month ago

chatted with @penny-lischer and we've decided we should go for the simplest solution which is pointing relative path base paths to: https://reportstream.cdc.gov/