accept methodName==null (bug in PMD in generic methods)

nidi3 · nidi3 · commit c9d10f5b2d42 · 2020-11-17T23:55:25.000+01:00
diff --git a/code-assert-core/src/main/java/guru/nidi/codeassert/config/LocationMatcher.java b/code-assert-core/src/main/java/guru/nidi/codeassert/config/LocationMatcher.java
@@ -55,7 +55,8 @@ public boolean matches(String packageName, String className, String methodName)
         final boolean matchesClass = matchesAll(loc.method)
                 ? matchesClassPattern(loc.clazz, className)
                 : matchesPattern(loc.clazz, className);
-        return matchesPattern(loc.pack, packageName) && matchesClass && matchesPattern(loc.method, methodName);
+        return matchesPattern(loc.pack, packageName) && matchesClass &&
+                matchesPattern(loc.method, methodName == null ? "" : methodName);
     }
 
     public boolean matchesLanguage(Language language) {

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,8 @@ public boolean matches(String packageName, String className, String methodName)`
`55`	`55`	`final boolean matchesClass = matchesAll(loc.method)`
`56`	`56`	`? matchesClassPattern(loc.clazz, className)`
`57`	`57`	`: matchesPattern(loc.clazz, className);`
`58`		`- return matchesPattern(loc.pack, packageName) && matchesClass && matchesPattern(loc.method, methodName);`
	`58`	`+ return matchesPattern(loc.pack, packageName) && matchesClass &&`
	`59`	`+ matchesPattern(loc.method, methodName == null ? "" : methodName);`
`59`	`60`	`}`
`60`	`61`
`61`	`62`	`public boolean matchesLanguage(Language language) {`