Skip to content

Conversation

@carlpartridge
Copy link
Collaborator

🎫 Ticket

https://jira.cms.gov/browse/...

🛠 Changes

Removed hard-coded values from unit-tests related to EOY/runout mode.

ℹ️ Context

Some unit tests related to EOY mode were failing.

🧪 Validation

Local testing and linting.

(NOW(), NOW(), 8, 'T.C9994.ACO.ZC8R68.D250909.T2135390', 'C9994', NOW(), 68, 'Completed', 1),
(NOW(), NOW(), 8, 'T.C9996.ACO.ZC8Y89.D250909.T2135490', 'C9996', NOW(), 89, 'Completed', 0),
(NOW(), NOW(), 8, 'T.C9996.ACO.ZC8R25.D250909.T2135590', 'C9996', NOW(), 25, 'Completed', 1),
(NOW(), NOW(), 8, 'T.C9996.ACO.ZC8R25.D250909.T2135590', 'C9996', NOW(), to_char(NOW(), 'yy')::int, 'Completed', 1),
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I dont think these changes affected the specific failing unit-tests but seemed like a good thing to update.

@carlpartridge carlpartridge marked this pull request as ready for review January 2, 2026 18:56
@carlpartridge carlpartridge requested a review from a team as a code owner January 2, 2026 18:56
Copy link
Contributor

@michaeljvaldes michaeljvaldes left a comment

Choose a reason for hiding this comment

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

Fwiw we have a few other references in other tests to a specific performance year (e.g. pgx_repository.test, repository.test) but I don't think those need to be updated.

@michaeljvaldes michaeljvaldes requested a review from a team January 6, 2026 14:48
@michaeljvaldes michaeljvaldes merged commit c530239 into main Jan 6, 2026
6 checks passed
@michaeljvaldes michaeljvaldes deleted the carl-fix-unit-tests-for-EOY branch January 6, 2026 16:38
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