Skip to content

Commit b55c7cf

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 93ff611 commit b55c7cf

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;
@@ -256,4 +257,17 @@ public void testDenseTransportVersions() {
256257
fail(msg.toString());
257258
}
258259
}
260+
261+
262+
public void testDuplicateConstants() {
263+
List<TransportVersion> tvs = TransportVersions.getAllVersions().stream().sorted().toList();
264+
TransportVersion previous = tvs.get(0);
265+
for (int i = 1; i < tvs.size(); i++) {
266+
TransportVersion next = tvs.get(i);
267+
if (next.id() == previous.id()) {
268+
throw new AssertionError("Duplicate transport version id: " + next.id());
269+
}
270+
previous = next;
271+
}
272+
}
259273
}

0 commit comments

Comments
 (0)