Skip to content

Commit d66f1b6

Browse files
authored
Fix failing LuceneFilesExtensionsTests testIsLuceneExtension case (#135069)
This fixes a small randomization issue leading to occasional AOOB in this test. Closes #134970
1 parent 90aab25 commit d66f1b6

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

muted-tests.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -558,9 +558,6 @@ tests:
558558
- class: org.elasticsearch.action.search.BatchedQueryPhaseIT
559559
method: testNumReducePhases
560560
issue: https://github.com/elastic/elasticsearch/issues/134943
561-
- class: org.elasticsearch.index.store.LuceneFilesExtensionsTests
562-
method: testIsLuceneExtension
563-
issue: https://github.com/elastic/elasticsearch/issues/134970
564561
- class: org.elasticsearch.reservedstate.service.RepositoriesFileSettingsIT
565562
method: testSettingsApplied
566563
issue: https://github.com/elastic/elasticsearch/issues/126748

qa/evil-tests/src/test/java/org/elasticsearch/index/store/LuceneFilesExtensionsTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ public void testIsLuceneExtension() {
5151
assertFalse(LuceneFilesExtensions.isLuceneExtension("bcde"));
5252
String randomStringWithLuceneExtension = randomAlphanumericOfLength(10)
5353
+ "."
54-
+ LuceneFilesExtensions.values()[randomInt(LuceneFilesExtensions.values().length) - 1].getExtension();
54+
+ LuceneFilesExtensions.values()[randomInt(LuceneFilesExtensions.values().length - 1)].getExtension();
5555
String extension = IndexFileNames.getExtension(randomStringWithLuceneExtension);
5656
assertTrue(extension + " should be considered a Lucene extension", LuceneFilesExtensions.isLuceneExtension(extension));
5757
String upperCaseExtension = extension.toUpperCase(Locale.ROOT);

0 commit comments

Comments
 (0)