Reuse supplied import options in ClassFileImporter.importClasspath()

Before this commit, all custom import options were ignored and replaced with DO_NOT_INCLUDE_ARCHIVES. From this commit on DO_NOT_INCLUDE_ARCHIVES will be added to the already supplied import options. Fixes TNG#296
rweisleder · Mar 7, 2020 · 670c089 · 670c089
1 parent a220200
commit 670c089
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/archunit/src/main/java/com/tngtech/archunit/core/importer/ClassFileImporter.java b/archunit/src/main/java/com/tngtech/archunit/core/importer/ClassFileImporter.java
@@ -248,7 +248,7 @@ public JavaClasses importPackagesOf(Collection<Class<?>> classes) {
      */
     @PublicAPI(usage = ACCESS)
     public JavaClasses importClasspath() {
-        return importClasspath(new ImportOptions().with(ImportOption.Predefined.DO_NOT_INCLUDE_ARCHIVES));
+        return importClasspath(importOptions.with(ImportOption.Predefined.DO_NOT_INCLUDE_ARCHIVES));
     }
 
     /**

diff --git a/archunit/src/test/java/com/tngtech/archunit/core/importer/ClassFileImporterSlowTest.java b/archunit/src/test/java/com/tngtech/archunit/core/importer/ClassFileImporterSlowTest.java
@@ -40,6 +40,14 @@ public void imports_the_classpath() {
         assertThatClasses(javaBaseClasses).doNotContain(ClassFileImporter.class, getClass(), Rule.class);
     }
 
+    @Test
+    public void withImportOption_importClasspath() {
+        JavaClasses classes = new ClassFileImporter().withImportOption(ImportOption.Predefined.DO_NOT_INCLUDE_TESTS).importClasspath();
+
+        assertThatClasses(classes).contain(ClassFileImporter.class);
+        assertThatClasses(classes).doNotContain(getClass(), Rule.class, String.class);
+    }
+
     @Test
     public void imports_packages() {
         JavaClasses classes = new ClassFileImporter().importPackages(