Skip to content

Conversation

@reta
Copy link
Contributor

@reta reta commented Oct 29, 2025

Description

Use TemporaryFolder JUnit @rule to manage temporary folders in tests

Related Issues

This is a follow up on #18921 to replace custom folder management with JUnit in tests.

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

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.

Signed-off-by: Andriy Redko <drreta@gmail.com>
@reta reta requested a review from a team as a code owner October 29, 2025 01:03
@reta reta added :test Adding or fixing a test skip-changelog labels Oct 29, 2025
@reta
Copy link
Contributor Author

reta commented Oct 29, 2025

@beanuwave @cwperks would appreciate if you could take a look folks, thank you.

Copy link
Member

@cwperks cwperks left a comment

Choose a reason for hiding this comment

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

Thank you @reta ! I guess we will need the suppressions until OpenSearch switches to JUnit 5.

@reta
Copy link
Contributor Author

reta commented Oct 29, 2025

Thank you @reta ! I guess we will need the suppressions until OpenSearch switches to JUnit 5.

Thanks @cwperks , yeah, but I suspect we would need Apache Lucene to make a move :)

@github-actions
Copy link
Contributor

❕ Gradle check result for 9934613: UNSTABLE

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

@codecov
Copy link

codecov bot commented Oct 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.16%. Comparing base (8dd1286) to head (9934613).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #19806      +/-   ##
============================================
+ Coverage     73.12%   73.16%   +0.03%     
- Complexity    71099    71140      +41     
============================================
  Files          5754     5754              
  Lines        325296   325296              
  Branches      47052    47052              
============================================
+ Hits         237868   237998     +130     
+ Misses        68282    68125     -157     
- Partials      19146    19173      +27     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@reta reta merged commit 83cb686 into opensearch-project:main Oct 29, 2025
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog :test Adding or fixing a test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants