Skip to content

test(node): Attempt to unflake docker-based node integration tests#21905

Merged
Lms24 merged 2 commits into
developfrom
lms/test-unflake-node-docker-compose-integration-tests
Jul 7, 2026
Merged

test(node): Attempt to unflake docker-based node integration tests#21905
Lms24 merged 2 commits into
developfrom
lms/test-unflake-node-docker-compose-integration-tests

Conversation

@Lms24

@Lms24 Lms24 commented Jul 1, 2026

Copy link
Copy Markdown
Member

I've had a few knex and mysql integration test timeouts recently. Clanker thinks this is attributed to us not determinisitcally stopping the correct docker containers in tests running simultaneously that start/stop containers. No guarantee that this works but probably worth a shot.

@Lms24 Lms24 marked this pull request as ready for review July 2, 2026 09:56
@Lms24 Lms24 requested a review from a team as a code owner July 2, 2026 09:56
@Lms24 Lms24 requested review from JPeer264, andreiborza and mydea and removed request for a team July 2, 2026 09:56
@Lms24 Lms24 self-assigned this Jul 2, 2026
Comment thread dev-packages/node-integration-tests/utils/runner/createRunner.ts

@JPeer264 JPeer264 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Let's hope for the best.

@Lms24 Lms24 force-pushed the lms/test-unflake-node-docker-compose-integration-tests branch from 3a5fe8e to 5ce16b2 Compare July 7, 2026 08:09
@Lms24 Lms24 merged commit 851edb3 into develop Jul 7, 2026
56 checks passed
@Lms24 Lms24 deleted the lms/test-unflake-node-docker-compose-integration-tests branch July 7, 2026 09:07
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