Skip to content

Commit a9018aa

Browse files
rjernstelasticsearchmachine
andauthored
Test for duplicate transport versions (#130494) (#130511)
* Test for duplicate transport versions (#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 #130486 * [CI] Auto commit changes from spotless --------- Co-authored-by: elasticsearchmachine <[email protected]>
1 parent acf5a0b commit a9018aa

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

0 commit comments

Comments
 (0)