Skip to content
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

MCR-4123 Fix contract actions missing on certain submissions #2445

Merged
merged 5 commits into from
May 17, 2024
Merged

Conversation

haworku
Copy link
Contributor

@haworku haworku commented May 14, 2024

Summary

Resolves an issue where a specific VAL submission (see ticket) is missing some of its related contract actions

Related issues

https://jiraent.cms.gov/browse/MCR-4123

Test cases covered

  • I tested this by pulling down VAL data locally. I saw that the faulty rate submission summary for MCR-FL-0210-PCCME-RATE-20240502-20240509-CERTIFICATION-20240404 was loading correctly.
  • Tried for awhile to write a failing test. I wasn't able to achieve that. The test continues to pass with or without the fix added.
    • I have added the test as skip for now.
    • I would like a reviewer to take a look at the test. Do you see an option for what I should do to try and get a legitimate failing test for the bug that resolves for the fix? Otherwise, I will remove the test, just merge this fix as is without a test.

QA guidance

  • QA regression check for Contract Actions behavior on rate summary - add, remove, edit linked rates and check that rate summary still displays as expected

@haworku haworku marked this pull request as ready for review May 15, 2024 14:45
@@ -352,4 +353,82 @@ describe('fetchRate', () => {
'2024-02-02'
)
})

// TEST NOT FAILING WHEN EXPECTED, NEED TO FIGURE OUT HOW TO GET RATE REVISIONS OUT OF SYNC
it.skip('returns correct linked contractRevisions, even when rate has changed', async()=>{
Copy link
Contributor

Choose a reason for hiding this comment

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

I couldn't think of a way to get revisions out of sync. I wonder if anything outside the workflow happened. Feature flagging, migrations, or maybe something in the API silently errored and continued with the request.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for checking it out. I will delete this test then... but appreciate the eyes and the QA

Copy link
Contributor

@JasonLin0991 JasonLin0991 left a comment

Choose a reason for hiding this comment

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

LGTM.

@haworku haworku merged commit c4c29e7 into main May 17, 2024
28 checks passed
@haworku haworku deleted the MCR-4123 branch May 17, 2024 14:39
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.

None yet

2 participants