Take into account ignoreChangesInFiles + fix associated tests

pom.xml

@@ -6,7 +6,7 @@
     <groupId>no.sparebank1</groupId>
     <artifactId>turbo-maven-plugin</artifactId>
     <packaging>maven-plugin</packaging>
-    <version>1.0.0</version>
+    <version>1.0.1</version>
 
     <name>turbo-maven-plugin Maven Mojo</name>
     <description>

src/main/java/no/sparebank1/turbo/SourceFileFinder.java

@@ -17,24 +17,25 @@ public List<String> getSourceFiles(final String projectRoot, final String ignore
         List<String> filesInSrc = findFilesInDirRecursively(projectRoot + "/src");
         sourceFiles.addAll(filesInSrc);
         //Remove files that are configured not to be included:
-        List<String> prunedSourcefiles = pruneSourefiles(sourceFiles, ignoreChangesInFiles);
+        List<String> prunedSourcefiles = pruneSourcefiles(sourceFiles, ignoreChangesInFiles);
 
         return prunedSourcefiles;
     }
 
-    List<String> pruneSourefiles(final List<String> sourcefiles, final String ignoreChangesInFiles) {
+    List<String> pruneSourcefiles(List<String> sourcefiles, String ignoreChangesInFiles) {
+        List<String> ignoredFiles = ignoreChangesInFiles != null ? Arrays.asList(ignoreChangesInFiles.split(",")) : new ArrayList<>();
         List<String> prunedSourcefiles = new ArrayList<>(sourcefiles);
-        sourcefiles.forEach(sourcefile -> {
-            if (sourcefile.contains("swagger.json") ||
-                    sourcefile.contains(".iml") ||
-                    sourcefile.contains("editorconfig") ||
-                    sourcefile.contains("docker-image-id")) {
-                prunedSourcefiles.remove(sourcefile);
-            }
-        });
+        sourcefiles.forEach(sourcefile -> prunedSourcefiles.removeIf(sourceFile -> containsAnyIgnoredFile(sourceFile, ignoredFiles)));
         return prunedSourcefiles;
     }
 
+    boolean containsAnyIgnoredFile(String sourceFile, List< String > ignoredFiles) {
+        return ignoredFiles.stream()
+                 .anyMatch(ignoredFile -> ignoredFile != null &&
+                         !ignoredFile.trim().isEmpty() &&
+                         sourceFile.contains(ignoredFile));
+    }
+
     List<String> findFilesInDir(final String dir) {
         File dirFile = new File(dir);
         List<File> list = Arrays.asList(dirFile.listFiles());

src/test/bar/pom.xml

@@ -0,0 +1 @@
+sometext

src/test/foo/src/bar1.txt

@@ -0,0 +1 @@
+bar1

src/test/foo/src/bar2.txt

@@ -0,0 +1 @@
+bar2

src/test/java/no/sparebank1/turbo/ArtifactAnalyzerTest.java

@@ -91,8 +91,8 @@ private List<String> getListOfSources() {
         String path = new File(".").getAbsolutePath();
         ArrayList<String> listOfSources = new ArrayList<>();
         listOfSources.add(path + "/src/test/foo/pom.xml");
-        listOfSources.add(path + "/src/test/foo/bar/bar1.txt");
-        listOfSources.add(path + "/src/test/foo/bar/bar2.txt");
+        listOfSources.add(path + "/src/test/foo/src/bar1.txt");
+        listOfSources.add(path + "/src/test/foo/src/bar2.txt");
         return listOfSources;
     }
 
@@ -141,4 +141,4 @@ private File addArtifact(final String m2Repository, final String groupId, final
         return file;
     }
 
-}
+}

src/test/java/no/sparebank1/turbo/SourceFileFinderTest.java

@@ -1,5 +1,6 @@
 package no.sparebank1.turbo;
 
+import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.io.File;
@@ -9,9 +10,19 @@
 
 class SourceFileFinderTest {
     @Test
-    public void shall_find_all_files_in_subdirectories() {
+    void shall_find_all_files_in_subdirectories() {
+        String path1 = new File(".").getAbsolutePath() + "/src/test/foo";
+        String path2 = new File(".").getAbsolutePath() + "/src/test/bar";
+        assertEquals( 3, new SourceFileFinder().getSourceFiles(path1, "").size() );
+        assertEquals( 1, new SourceFileFinder().getSourceFiles(path2, "").size() );
+    }
+
+    @Test
+    public void shall_find_with_ignore_files() {
         String path = new File(".").getAbsolutePath() + "/src/test/foo";
-        List<String> sourcefiles = new SourceFileFinder().getSourceFiles(path, "");
-        assertTrue(true);
+        assertEquals( 1, new SourceFileFinder().getSourceFiles(path, "bar1,bar2").size() );
+        assertEquals( 1, new SourceFileFinder().getSourceFiles(path, "bar").size() );
+        assertEquals( 3, new SourceFileFinder().getSourceFiles(path, null).size() );
     }
+
 }