[Core] Log warning when resources could not be loaded

Fixes: #2212, #2229

Author: mpkorstanje

CHANGELOG.md

@@ -17,6 +17,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Fixed
  * [Core] Pass class loader to ServiceLoader.load invocations ([#2220](https://github.com/cucumber/cucumber-jvm/issues/2220) M.P. Korstanje)
+ * [Core] Log warnings when classes or resource could not be loaded ([#2235](https://github.com/cucumber/cucumber-jvm/issues/2235) M.P. Korstanje)
 
 ## [6.9.1] (2020-12-14)
 

core/src/main/java/io/cucumber/core/resource/ClasspathScanner.java

@@ -14,6 +14,7 @@
 import java.util.function.Predicate;
 import java.util.function.Supplier;
 
+import static io.cucumber.core.resource.ClasspathSupport.classPathScanningExplanation;
 import static io.cucumber.core.resource.ClasspathSupport.determineFullyQualifiedClassName;
 import static io.cucumber.core.resource.ClasspathSupport.getUrisForPackage;
 import static io.cucumber.core.resource.ClasspathSupport.requireValidPackageName;
@@ -95,16 +96,21 @@ private Function<Path, Consumer<Path>> processClassFiles(
     ) {
         return baseDir -> classFile -> {
             String fqn = determineFullyQualifiedClassName(baseDir, basePackageName, classFile);
-            try {
-                Optional.of(getClassLoader().loadClass(fqn))
-                        .filter(classFilter)
-                        .ifPresent(classConsumer);
-            } catch (ClassNotFoundException | NoClassDefFoundError e) {
-                log.debug(e, () -> "Failed to load class " + fqn);
-            }
+            safelyLoadClass(fqn)
+                    .filter(classFilter)
+                    .ifPresent(classConsumer);
         };
     }
 
+    private Optional<Class<?>> safelyLoadClass(String fqn) {
+        try {
+            return Optional.ofNullable(getClassLoader().loadClass(fqn));
+        } catch (ClassNotFoundException | NoClassDefFoundError e) {
+            log.warn(e, () -> "Failed to load class '" + fqn + "'.\n" + classPathScanningExplanation());
+        }
+        return Optional.empty();
+    }
+
     public List<Class<?>> scanForClassesInPackage(String packageName) {
         return scanForClassesInPackage(packageName, NULL_FILTER);
     }

core/src/main/java/io/cucumber/core/resource/ClasspathSupport.java

@@ -153,4 +153,14 @@ public static URI rootPackageUri() {
         return URI.create(CLASSPATH_SCHEME_PREFIX + RESOURCE_SEPARATOR_CHAR);
     }
 
+    public static String classPathScanningExplanation() {
+        return "By default Cucumber scans the entire classpath for step definitions.\n" +
+                "You can restrict this by configuring the glue path.\n" +
+                "\n" +
+                "Examples:\n" +
+                " - @CucumberOptions(glue = \"com.example.application\")\n" +
+                " - src/test/resources/junit-platform.properties   cucumber.glue=com.example.application\n" +
+                " - src/test/resources/cucumber.properties         cucumber.glue=com.example.application\n";
+    }
+
 }

core/src/main/java/io/cucumber/core/resource/JarUriFileSystemService.java

@@ -101,7 +101,7 @@ private static CloseablePath handleJarUriScheme(URI uri) throws IOException, URI
 
     private static CucumberException nestedJarEntriesAreUnsupported(URI uri) {
         return new CucumberException("" +
-                "The resource " + uri + " is located in a nested jar.\n" +
+                "The resource '" + uri + "' is located in a nested jar.\n" +
                 "\n" +
                 "This typically happens when trying to run Cucumber inside a Spring Boot Executable Jar.\n" +
                 "Cucumber currently doesn't support classpath scanning in nested jars.\n" +

core/src/main/java/io/cucumber/core/resource/PathScanner.java

@@ -6,6 +6,7 @@
 import java.io.IOException;
 import java.net.URI;
 import java.net.URISyntaxException;
+import java.nio.file.FileSystemNotFoundException;
 import java.nio.file.FileVisitOption;
 import java.nio.file.FileVisitResult;
 import java.nio.file.Path;
@@ -22,10 +23,14 @@
 
 class PathScanner {
 
+    private static final Logger log = LoggerFactory.getLogger(PathScanner.class);
+
     void findResourcesForUri(URI baseUri, Predicate<Path> filter, Function<Path, Consumer<Path>> consumer) {
         try (CloseablePath closeablePath = open(baseUri)) {
             Path baseDir = closeablePath.getPath();
             findResourcesForPath(baseDir, filter, consumer);
+        } catch (FileSystemNotFoundException e) {
+            log.warn(e, () -> "Failed to find resources for '" + baseUri + "'");
         } catch (IOException | URISyntaxException e) {
             throw new RuntimeException(e);
         }

core/src/test/java/io/cucumber/core/resource/ResourceScannerTest.java

@@ -124,6 +124,12 @@ void scanForResourcesDirectory() {
             new File("src/test/resources/io/cucumber/core/resource/test/spaces in name resource.txt").toURI()));
     }
 
+    @Test
+    void shouldThrowIfForResourcesPathNotExist() {
+        File file = new File("src/test/resources/io/cucumber/core/does/not/exist");
+        assertThrows(IllegalArgumentException.class, () -> resourceScanner.scanForResourcesPath(file.toPath()));
+    }
+
     @Test
     @DisabledOnOs(value = OS.WINDOWS,
             disabledReason = "Only works if repository is explicitly cloned activated symlinks and " +

java/src/main/java/io/cucumber/java/MethodScanner.java

@@ -1,14 +1,20 @@
 package io.cucumber.java;
 
+import io.cucumber.core.logging.Logger;
+import io.cucumber.core.logging.LoggerFactory;
+
 import java.lang.annotation.Annotation;
 import java.lang.reflect.Method;
 import java.lang.reflect.Modifier;
 import java.util.function.BiConsumer;
 
+import static io.cucumber.core.resource.ClasspathSupport.classPathScanningExplanation;
 import static io.cucumber.java.InvalidMethodException.createInvalidMethodException;
 
 final class MethodScanner {
 
+    private static final Logger log = LoggerFactory.getLogger(MethodScanner.class);
+
     private MethodScanner() {
     }
 
@@ -21,11 +27,21 @@ static void scan(Class<?> aClass, BiConsumer<Method, Annotation> consumer) {
         if (!isInstantiable(aClass)) {
             return;
         }
-        for (Method method : aClass.getMethods()) {
+        for (Method method : safelyGetMethods(aClass)) {
             scan(consumer, aClass, method);
         }
     }
 
+    private static Method[] safelyGetMethods(Class<?> aClass) {
+        try {
+            return aClass.getMethods();
+        } catch (NoClassDefFoundError e) {
+            log.warn(e,
+                () -> "Failed to load methods of class '" + aClass.getName() + "'.\n" + classPathScanningExplanation());
+        }
+        return new Method[0];
+    }
+
     private static boolean isInstantiable(Class<?> clazz) {
         boolean isNonStaticInnerClass = !Modifier.isStatic(clazz.getModifiers()) && clazz.getEnclosingClass() != null;
         return Modifier.isPublic(clazz.getModifiers()) && !Modifier.isAbstract(clazz.getModifiers())