Skip to content

Commit 4d19f1f

Browse files
committed
Test for duplicate transport versions (elastic#130494)
We used to have an assertion during transport version loading that duplicate ids were not found, but it appears to have been lost in refactorings. This commit adds a test to ensure duplicate ids do not occur. relates elastic#130486
1 parent acf5a0b commit 4d19f1f

File tree

1 file changed

+14
-0
lines changed

1 file changed

+14
-0
lines changed

server/src/test/java/org/elasticsearch/TransportVersionTests.java

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
import java.lang.reflect.Modifier;
1616
import java.util.Collections;
1717
import java.util.Map;
18+
import java.util.List;
1819
import java.util.Set;
1920
import java.util.TreeSet;
2021
import java.util.regex.Matcher;
@@ -258,4 +259,17 @@ public void testDenseTransportVersions() {
258259
fail(msg.toString());
259260
}
260261
}
262+
263+
264+
public void testDuplicateConstants() {
265+
List<TransportVersion> tvs = TransportVersions.getAllVersions().stream().sorted().toList();
266+
TransportVersion previous = tvs.get(0);
267+
for (int i = 1; i < tvs.size(); i++) {
268+
TransportVersion next = tvs.get(i);
269+
if (next.id() == previous.id()) {
270+
throw new AssertionError("Duplicate transport version id: " + next.id());
271+
}
272+
previous = next;
273+
}
274+
}
261275
}

0 commit comments

Comments
 (0)