Skip to content

Commit bf7f28a

Browse files
jderegclaude
andcommitted
Remove flaky wall-clock timing assertion from MultiKeyMap typed array test
The hard 100ms threshold caused CI failures on slower runners (293ms observed). Correctness assertions for all 1000 entries are retained. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 01a97f2 commit bf7f28a

1 file changed

Lines changed: 1 addition & 7 deletions

File tree

src/test/java/com/cedarsoftware/util/MultiKeyMapTypedArrayProcessingTest.java

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -202,18 +202,12 @@ void testTypedArrayProcessingPerformance() {
202202
map.put(array, "value" + i);
203203
}
204204

205-
// Test lookup performance - should use fast typed processing
206-
long startTime = System.nanoTime();
205+
// Test lookup correctness with many entries
207206
for (int i = 0; i < 1000; i++) {
208207
int[] lookupArray = {i, i + 1, i + 2};
209208
String result = map.get(lookupArray);
210209
assertEquals("value" + i, result);
211210
}
212-
long endTime = System.nanoTime();
213-
214-
// Performance test should complete reasonably quickly
215-
long durationMs = (endTime - startTime) / 1_000_000;
216-
assertTrue(durationMs < 100, "int[] processing should be fast, took " + durationMs + "ms");
217211

218212
assertEquals(1000, map.size());
219213
}

0 commit comments

Comments
 (0)