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

Fix the race condition in e2e test. #1295

Open
bashir2 opened this issue Feb 11, 2025 · 0 comments · May be fixed by #1312
Open

Fix the race condition in e2e test. #1295

bashir2 opened this issue Feb 11, 2025 · 0 comments · May be fixed by #1312
Assignees
Labels
bug Something isn't working P1:must As issue that definitely needs to be implemented in near future.

Comments

@bashir2
Copy link
Collaborator

bashir2 commented Feb 11, 2025

Based on this report:

There is a race condition between the BULK_EXPORT test and the controller test. Based on the dep graph and FHIR-server parameters, I think both of these tests are querying http://hapi-server:8080; but the problem is that in the controller test we are testing the incremental pipeline functionality as well. So depending on whether it runs before or after BULK_EXPORT test we may get extra resources in BULK_EXPORT test, which can fail the whole e2e test.

@bashir2 bashir2 added bug Something isn't working P1:must As issue that definitely needs to be implemented in near future. labels Feb 11, 2025
@bashir2 bashir2 self-assigned this Feb 11, 2025
@bashir2 bashir2 linked a pull request Feb 24, 2025 that will close this issue
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working P1:must As issue that definitely needs to be implemented in near future.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant