Skip to content

Conversation

@labkey-martyp
Copy link
Contributor

@labkey-martyp labkey-martyp commented Jul 24, 2025

Rationale

Link crawler test validates links in the EHR folder. ONPRC tests have not used this folder in the past. This converts to using the EHR folder for tests, which more closely matches the production server.

Related Pull Requests

Changes

  • Remove container path override
  • Update setup to configure EHR folder
  • Update tests to start in the EHR folder

@brentlogan
Copy link
Collaborator

@labkey-martyp, let me know if you'd like us to test this on our prime2507 VM.

Copy link
Member

@labkey-tchad labkey-tchad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AbstractONPRC_EHRTest should override testCrawlEhrLinks so that it only gets run in ONPRC_EHRTest but not ONPRC_EHRTest2 or ONPRC_BillingTest

labkey-tchad
labkey-tchad previously approved these changes Jul 31, 2025
Copy link
Member

@labkey-tchad labkey-tchad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was ONPRC_EHRTest2 just working in the wrong folder all this time?

…ericONPRC_EHRTest.java

Co-authored-by: Trey Chadick <tchad@labkey.com>
@labkey-martyp
Copy link
Contributor Author

Was ONPRC_EHRTest2 just working in the wrong folder all this time?

Yes, they've all been using the project folder instead of the EHR folder. I've had it on my backlog to fix for a long time. Seemed like a good time to fix it.

@labkey-tchad labkey-tchad self-requested a review August 3, 2025 20:18
@labkey-martyp labkey-martyp merged commit 3a9307d into release25.7-SNAPSHOT Aug 3, 2025
7 checks passed
@labkey-martyp labkey-martyp deleted the 25.7_fb_ehr_link_crawler branch August 3, 2025 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants