Skip to content

Commit dd433fe

Browse files
Test for duplicate transport versions (elastic#130494) (elastic#130512)
* 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 * [CI] Auto commit changes from spotless --------- Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
1 parent 93ff611 commit dd433fe

File tree

1 file changed

+13
-0
lines changed

1 file changed

+13
-0
lines changed

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

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414

1515
import java.lang.reflect.Modifier;
1616
import java.util.Collections;
17+
import java.util.List;
1718
import java.util.Map;
1819
import java.util.Set;
1920
import java.util.TreeSet;
@@ -256,4 +257,16 @@ public void testDenseTransportVersions() {
256257
fail(msg.toString());
257258
}
258259
}
260+
261+
public void testDuplicateConstants() {
262+
List<TransportVersion> tvs = TransportVersions.getAllVersions().stream().sorted().toList();
263+
TransportVersion previous = tvs.get(0);
264+
for (int i = 1; i < tvs.size(); i++) {
265+
TransportVersion next = tvs.get(i);
266+
if (next.id() == previous.id()) {
267+
throw new AssertionError("Duplicate transport version id: " + next.id());
268+
}
269+
previous = next;
270+
}
271+
}
259272
}

0 commit comments

Comments
 (0)