fixes #975

Fix ignore pattern and test

Author: fabapp2

sbm-support-rewrite/src/main/java/org/springframework/sbm/parsers/ModuleParser.java

@@ -128,7 +128,7 @@ public List<SourceFile> processMainSources(
         mainProjectProvenance.add(sourceSet("main", dependencies, typeCache));
 
         // FIXME: 945 Why target and not all main?
-        List<Path> parsedJavaPaths = mainJavaSources.stream().map(ResourceUtil::getPath).toList();
+        List<Path> parsedJavaPaths = javaSourcesInTarget.stream().map(ResourceUtil::getPath).toList();
         Stream<SourceFile> parsedJava = cus.map(addProvenance(baseDir, mainProjectProvenance, parsedJavaPaths));
         log.debug("[%s] Scanned %d java source files in main scope.".formatted(currentProject, mainJavaSources.size()));
 

sbm-support-rewrite/src/main/resources/META-INF/sbm-support-rewrite.properties

@@ -27,4 +27,4 @@ parser.sizeThresholdMb=10
 parser.runPerSubmodule=false
 parser.failOnInvalidActiveRecipes=true
 parser.activeProfiles=default
-parser.ignoredPathPatterns=**.idea,**.git,**/target/**,target/**
+parser.ignoredPathPatterns=**.idea/**,**.git,**/target/**,target/**

sbm-support-rewrite/src/test/java/org/springframework/sbm/parsers/CompareParserRecipeRunTest.java

@@ -45,7 +45,6 @@
  * @author Fabian Krüger
  */
 @SpringBootTest(classes = {SbmSupportRewriteConfiguration.class, SbmTestConfiguration.class})
-@Issue("https://github.com/spring-projects-experimental/spring-boot-migrator/issues/975")
 public class CompareParserRecipeRunTest {
 
     @Autowired
@@ -58,9 +57,8 @@ public class CompareParserRecipeRunTest {
     private ExecutionContext executionContext;
 
     @Test
-    @DisplayName("runningRecipe")
-    @ExpectedToFail("FIXME: #975")
-    void runningRecipe() {
+    @DisplayName("Running a recipe with RewriteMavenParser should yield the same result as with RewriteProjectParser")
+    void runningARecipeWithRewriteMavenParserYieldsTheSameResultAsWithRewriteProjectParser() {
         Path baseDir = TestProjectHelper.getMavenProject("parser-recipe-run");
         RewriteProjectParsingResult sutParsingResult = sut.parse(baseDir);
         RewriteProjectParsingResult compParsingResult = comparingParser.parse(baseDir);
@@ -105,7 +103,7 @@ public J.ClassDeclaration visitClassDeclaration(J.ClassDeclaration classDecl, Ex
         // Run Parser independent from Maven
         counter.setRelease(0);
         RecipeRun sutRecipeRun = recipe.run(new InMemoryLargeSourceSet(sutParsingResult.sourceFiles()), executionContext);
-        assertThat(counter.get()).isEqualTo(3); // differs, should be 2
+        assertThat(counter.get()).isEqualTo(2); // differs, should be 2
         assertThat(sutRecipeRun.getChangeset().getAllResults()).hasSize(1); // is 0
     }
 

sbm-support-rewrite/src/test/java/org/springframework/sbm/parsers/ParserPropertiesTest.java

@@ -87,7 +87,7 @@ void defaultActiveProfiles() {
         @Test
         @DisplayName("parser.ignoredPathPatterns")
         void defaultIgnoredPathPatterns() {
-            assertThat(parserProperties.getIgnoredPathPatterns()).containsExactlyInAnyOrder("target/**", "**.git", "**/target/**", "**.idea");
+            assertThat(parserProperties.getIgnoredPathPatterns()).containsExactlyInAnyOrder("**.idea/**", "**.git", "**/target/**", "target/**");
         }
 
     }