diff --git a/java/ql/src/Performance/InnerClassCouldBeStatic.ql b/java/ql/src/Performance/InnerClassCouldBeStatic.ql index 1a83db097f3..02ca5544da2 100644 --- a/java/ql/src/Performance/InnerClassCouldBeStatic.ql +++ b/java/ql/src/Performance/InnerClassCouldBeStatic.ql @@ -130,7 +130,9 @@ predicate potentiallyStatic(InnerClass c) { ) ) and // JUnit Nested test classes are required to be non-static. - not c.hasAnnotation("org.junit.jupiter.api", "Nested") + not c.hasAnnotation("org.junit.jupiter.api", "Nested") and + // There's no `static` in kotlin: + not c.getLocation().getFile().isKotlinSourceFile() } /** diff --git a/java/ql/test/kotlin/query-tests/InnerClassCouldBeStatic/InnerClassCouldBeStatic.expected b/java/ql/test/kotlin/query-tests/InnerClassCouldBeStatic/InnerClassCouldBeStatic.expected index 1fb8417d622..e69de29bb2d 100644 --- a/java/ql/test/kotlin/query-tests/InnerClassCouldBeStatic/InnerClassCouldBeStatic.expected +++ b/java/ql/test/kotlin/query-tests/InnerClassCouldBeStatic/InnerClassCouldBeStatic.expected @@ -1 +0,0 @@ -| Test.kt:3:5:5:5 | Companion | Companion should be made static, since the enclosing instance is not used. |