Skip to content

Commit 583f285

Browse files
author
Christoph Büscher
authored
[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 f91cc13 commit 583f285

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)