Skip to content

add functional test02: mark negative Covid case reviewed#776

Open
hawley-skylight wants to merge 4 commits intomainfrom
ph/APP-285-add-functional-test-02
Open

add functional test02: mark negative Covid case reviewed#776
hawley-skylight wants to merge 4 commits intomainfrom
ph/APP-285-add-functional-test-02

Conversation

@hawley-skylight
Copy link
Copy Markdown
Contributor

@hawley-skylight hawley-skylight commented Apr 29, 2026

Description

Add functional test 02: mark a negative Covid Lab Report as reviewed.

Related Issue

APP-285

Additional Notes

Additional notes and context if necessary

Checklist

  • I have ensured that the pull request is of a manageable size, allowing it to be reviewed within a single session.
  • I have reviewed my changes to ensure they are clear, concise, and well-documented.
  • I have updated the documentation, if applicable.
  • I have added or updated test cases to cover my changes, if applicable.

@hawley-skylight hawley-skylight requested a review from a team as a code owner April 29, 2026 23:02
Comment on lines +308 to +324
"CONDITION_KEY": 239,
"COPY_TO_PROVIDER_KEY": 1,
"INVESTIGATION_KEY": 1,
"LAB_RPT_DT_KEY": 1,
"LAB_TEST_KEY": 2,
"LAB_TEST_TECHNICIAN_KEY": 1,
"LAB_TEST_UID": 22000008,
"LDF_GROUP_KEY": 1,
"MORB_RPT_KEY": 1,
"ORDERING_ORG_KEY": 1,
"ORDERING_PROVIDER_KEY": 3,
"PERFORMING_LAB_KEY": 1,
"RECORD_STATUS_CD": "ACTIVE",
"REPORTING_LAB_KEY": 3,
"RESULT_COMMENT_GRP_KEY": 1,
"SPECIMEN_COLLECTOR_KEY": 1,
"TEST_RESULT_GRP_KEY": 1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I believe the *_KEY values might change based on future tests and cause a failure. For example, if a different test that creates a LAB_TEST_KEY entry runs prior to this test, the value 2 will no longer be correct here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

All but CONDITION_KEY? CONDITION_KEY is unlikely to be changed by other tests.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yea, i believe that CONDITION_KEY would be fairly static.

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.

2 participants