Skip to content

Commit 222ccd2

Browse files
committed
Review: Unify use of ANONYMOUS_CLASSES predicate
In `PublicAPIRules` we can now also use the predefined `JavaClass.Predicates.ANONYMOUS_CLASSES`. Signed-off-by: Peter Gafert <[email protected]>
1 parent 0fd7cd3 commit 222ccd2

File tree

1 file changed

+2
-10
lines changed

1 file changed

+2
-10
lines changed

archunit-integration-test/src/test/java/com/tngtech/archunit/PublicAPIRules.java

+2-10
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
import static com.tngtech.archunit.base.DescribedPredicate.anyElementThat;
3030
import static com.tngtech.archunit.base.DescribedPredicate.doNot;
3131
import static com.tngtech.archunit.base.DescribedPredicate.not;
32+
import static com.tngtech.archunit.core.domain.JavaClass.Predicates.ANONYMOUS_CLASSES;
3233
import static com.tngtech.archunit.core.domain.JavaClass.Predicates.assignableTo;
3334
import static com.tngtech.archunit.core.domain.JavaClass.Predicates.equivalentTo;
3435
import static com.tngtech.archunit.core.domain.JavaClass.Predicates.resideInAPackage;
@@ -188,15 +189,6 @@ private boolean enumMethod(JavaMethod methodToCheck, String name, Class<?>... pa
188189
};
189190
}
190191

191-
private static DescribedPredicate<JavaClass> anonymousClass() {
192-
return new DescribedPredicate<JavaClass>("anonymous class") {
193-
@Override
194-
public boolean apply(JavaClass input) {
195-
return input.isAnonymousClass();
196-
}
197-
};
198-
}
199-
200192
private static DescribedPredicate<JavaMember> declaredInClassIn(String packageIdentifier) {
201193
return declaredIn(resideInAPackage(packageIdentifier).as("class in '%s'", packageIdentifier));
202194
}
@@ -286,7 +278,7 @@ private boolean isPublicAPISuperMethod(JavaMethod candidate, JavaMethod methodTo
286278
private static DescribedPredicate<JavaMember> relevantArchUnitMembers() {
287279
return not(inheritedFromObjectOrEnum())
288280
.and(not(declaredIn(assignableTo(Annotation.class))))
289-
.and(not(declaredIn(anonymousClass())))
281+
.and(not(declaredIn(ANONYMOUS_CLASSES)))
290282
.and(not(declaredIn(internal())))
291283
.and(not(declaredInClassIn(THIRDPARTY_PACKAGE_IDENTIFIER)))
292284
.as("relevant members");

0 commit comments

Comments
 (0)