Skip to content

Commit 01da252

Browse files
Christoph Büscherkcm
Christoph Büscher
authored andcommitted
[Test] Remove dead code from ExceptionSerializationTests (#34713)
The `ignore` set contains entries of type Class<?>, but the check is performed on Path objects. This always returns false so is useless currently. Looking at the first commit of this test that already shows this behaviour this never excluded anything, so it can be removed.
1 parent 468c753 commit 01da252

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

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

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -133,11 +133,7 @@ public void testExceptionRegistration()
133133

134134
@Override
135135
public FileVisitResult preVisitDirectory(Path dir, BasicFileAttributes attrs) throws IOException {
136-
Path next = pkgPrefix.resolve(dir.getFileName());
137-
if (ignore.contains(next)) {
138-
return FileVisitResult.SKIP_SUBTREE;
139-
}
140-
pkgPrefix = next;
136+
pkgPrefix = pkgPrefix.resolve(dir.getFileName());
141137
return FileVisitResult.CONTINUE;
142138
}
143139

0 commit comments

Comments
 (0)