-
Notifications
You must be signed in to change notification settings - Fork 51
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
Make TestsRunner.test.ts and TestsRunner.test.ts independent. #317
Conversation
Changes AnalysisCommit SHA: 350f4c0 API ChangesSummaryNO CHANGES ReportThe full API changes report is available at: https://github.com/opensearch-project/opensearch-api-specification/actions/runs/9391821068/artifacts/1572914731 API Coverage
|
Did you create |
Signed-off-by: dblock <[email protected]>
Signed-off-by: dblock <[email protected]>
ff25978
to
350f4c0
Compare
Yes. I come from the point of view that tests should be able to run in parallel and not be interfering with each-other. But I understand why integration tests may be different. Would you prefer a solution where we make these tests "integration" and run them with |
Yes. I think running integration tests with
and have 2
|
Closing in favor of #320. |
Description
If you run
npm run test
the two test suites will conflict as they both create/delete the same indices (books). Split the fixtures, called one books and the other shoes.Use
beforeAll
to avoid global setup.Renamed
tools/tests/tester/overall_result.test.ts
tohelpers.test.ts
to match the file being tested.Removes the need for
--runInBand
, runs faster and with defaults.The only downside is duplication of the YAML tests.
Issues Resolved
Part of #313.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.