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

Adjusted Async projection test with IoC injection to remove flakiness #2756

Merged
merged 2 commits into from
Oct 20, 2023

Conversation

oskardudycz
Copy link
Collaborator

@oskardudycz oskardudycz commented Oct 19, 2023

The issue was in the race condition, as Async Daemon in tests was started both through the hosted service and then manually to wait for the projection shard.

Added also dedicated DaemonLockId value for async projection with IoC integration tests. It seems like it may be clashing with other async daemons running on the same database when using the default lock id.

This test ensures that the projection configuration actually works.
The inline projection is configured to automatically delete the
document upon receiving a deletion event.
@oskardudycz oskardudycz force-pushed the fix_flaky_ioc_projection_tests branch 4 times, most recently from 33bd984 to d65c772 Compare October 19, 2023 14:50
@oskardudycz oskardudycz force-pushed the fix_flaky_ioc_projection_tests branch 2 times, most recently from 0c04fa1 to fa24e48 Compare October 20, 2023 07:26
The issue was in the Daemon run both as hosted service and then manually to wait for shards.
@oskardudycz oskardudycz force-pushed the fix_flaky_ioc_projection_tests branch from fa24e48 to 3dcbb37 Compare October 20, 2023 07:30
@oskardudycz oskardudycz merged commit 86cf572 into master Oct 20, 2023
13 checks passed
@oskardudycz oskardudycz deleted the fix_flaky_ioc_projection_tests branch October 20, 2023 07:49
@oskardudycz oskardudycz added this to the 6.3.0 milestone Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants