-
Notifications
You must be signed in to change notification settings - Fork 3
Link crawler test #1402
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
Link crawler test #1402
Conversation
|
@labkey-martyp, let me know if you'd like us to test this on our prime2507 VM. |
labkey-tchad
left a comment
There was a problem hiding this 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
left a comment
There was a problem hiding this 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?
onprc_ehr/test/src/org/labkey/test/tests/onprc_ehr/AbstractGenericONPRC_EHRTest.java
Outdated
Show resolved
Hide resolved
…ericONPRC_EHRTest.java Co-authored-by: Trey Chadick <tchad@labkey.com>
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. |
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