tballison opened a new pull request, #16544: URL: https://github.com/apache/lucene/pull/16544
`TestTrackingTmpOutputDirectoryWrapper#testCopyFromCleansUpOnFailure`, added in #16530, fails on some seeds. Reported by @javanna on #16530, seen on [Lucene-main-Linux #66586](https://jenkins.thetaphi.de/job/Lucene-main-Linux/66586/): ``` gradlew test --tests TestTrackingTmpOutputDirectoryWrapper.testCopyFromCleansUpOnFailure -Dtests.seed=FEF8B573E1CF44D1 -Dtests.locale=en-IN -Dtests.timezone=Mexico/BajaNorte -Dtests.asserts=false -Dtests.file.encoding=UTF-8 ``` ``` java.lang.AssertionError: temp file must be deleted from backing dir after failed copyFrom expected:<0> but was:<1> ``` Test bug, not a production one — the leftover file is `extra0`, planted by `ExtrasFS` and passed through by `MockDirectoryWrapper#listAll()`, so `listAll().length == 0` is the wrong assertion. `copyFrom` deletes the temp file correctly. Fixed by filtering extras before asserting, as in `TestSegmentInfos`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
