[bs-73] Anonymous classes cannot be used in @Bean definitions in .groovy scripts

* Added a test for each of the classes loaded by the SpringApplication
* If it's an anonymous class or looks like a Groovy closure we ignore it
* The CLI sample job.groovy also modified to take advantage

[Fixes #48718891]

pom.xml

@@ -252,7 +252,7 @@
 			<dependency>
 				<groupId>org.codehaus.groovy</groupId>
 				<artifactId>groovy</artifactId>
-				<version>2.1.2</version>
+				<version>2.1.3</version>
 			</dependency>
 			<dependency>
 				<groupId>org.eclipse.jetty</groupId>

spring-bootstrap-cli/samples/job.groovy

@@ -14,7 +14,12 @@ class JobConfig {
 
 	@Bean
 	protected Tasklet tasklet() {
-		return new SampleTasklet()
+		return new Tasklet() { 
+          @Override
+          RepeatStatus execute(StepContribution contribution, ChunkContext context) {
+            return RepeatStatus.FINISHED
+          }
+        }
 	}
 
 	@Bean
@@ -28,13 +33,6 @@ class JobConfig {
 	}
 }
 
-class SampleTasklet implements Tasklet {
-	@Override
-	RepeatStatus execute(StepContribution contribution, ChunkContext context) {
-		return RepeatStatus.FINISHED
-	}
-}
-
 import org.springframework.util.StringUtils
 import groovy.util.logging.Log
 

spring-bootstrap-cli/src/main/java/org/springframework/bootstrap/cli/CleanCommand.java

@@ -34,6 +34,10 @@ import org.apache.ivy.util.FileUtil;
  */
 public class CleanCommand extends OptionParsingCommand {
 
+	private static enum Layout {
+		IVY, MAVEN;
+	}
+
 	private OptionSpec<Void> allOption;
 
 	public CleanCommand() {
@@ -55,28 +59,18 @@ public class CleanCommand extends OptionParsingCommand {
 
 	@Override
 	protected void run(OptionSet options) throws Exception {
+		clean(options, getGrapesHome(options), Layout.IVY);
+		clean(options, getMavenHome(options), Layout.MAVEN);
+	}
 
-		String dir = System.getenv("GROOVY_HOME");
-		String userdir = System.getProperty("user.home");
+	private void clean(OptionSet options, File root, Layout layout) {
 
-		File home;
-		if (dir == null || !new File(dir).exists()) {
-			dir = userdir;
-			home = new File(dir, ".groovy");
-		} else {
-			home = new File(dir);
-		}
-		if (dir == null || !new File(dir).exists()) {
+		if (root == null || !root.exists()) {
 			return;
 		}
 
-		if (!home.exists()) {
-			return;
-		}
-
-		File grapes = new File(home, "grapes");
 		ArrayList<String> specs = new ArrayList<String>(options.nonOptionArguments());
-		if (!specs.contains("org.springframework.bootstrap")) {
+		if (!specs.contains("org.springframework.bootstrap") && layout == Layout.IVY) {
 			specs.add(0, "org.springframework.bootstrap");
 		}
 		for (String spec : specs) {
@@ -86,8 +80,7 @@ public class CleanCommand extends OptionParsingCommand {
 				group = spec.substring(0, spec.indexOf(":"));
 				module = spec.substring(spec.indexOf(":") + 1);
 			}
-			File file = module == null ? new File(grapes, group) : new File(new File(
-					grapes, group), module);
+			File file = getModulePath(root, group, module, layout);
 			if (file.exists()) {
 				if (options.has(this.allOption)
 						|| group.equals("org.springframework.bootstrap")) {
@@ -102,7 +95,52 @@ public class CleanCommand extends OptionParsingCommand {
 				}
 			}
 		}
-
 	}
 
+	private File getModulePath(File root, String group, String module, Layout layout) {
+		File parent = root;
+		if (layout == Layout.IVY) {
+			parent = new File(parent, group);
+		} else {
+			for (String path : group.split("\\.")) {
+				parent = new File(parent, path);
+			}
+		}
+
+		if (module == null) {
+			return parent;
+		}
+		return new File(parent, module);
+	}
+
+	private File getGrapesHome(OptionSet options) {
+
+		String dir = System.getenv("GROOVY_HOME");
+		String userdir = System.getProperty("user.home");
+
+		File home;
+		if (dir == null || !new File(dir).exists()) {
+			dir = userdir;
+			home = new File(dir, ".groovy");
+		} else {
+			home = new File(dir);
+		}
+		if (dir == null || !new File(dir).exists()) {
+			return null;
+		}
+
+		File grapes = new File(home, "grapes");
+		return grapes;
+	}
+
+	private File getMavenHome(OptionSet options) {
+		String dir = System.getProperty("user.home");
+
+		if (dir == null || !new File(dir).exists()) {
+			return null;
+		}
+		File home = new File(dir);
+		File grapes = new File(new File(home, ".m2"), "repository");
+		return grapes;
+	}
 }

spring-bootstrap-cli/src/main/scripts/spring

@@ -71,4 +71,4 @@ if $cygwin; then
     CLASSPATH=`cygpath --path --mixed "$CLASSPATH"`
 fi
 
-${JAVA_HOME}/bin/java -cp "$CLASSPATH" org.springframework.bootstrap.cli.SpringBootstrapCli $*
+${JAVA_HOME}/bin/java ${JAVA_OPTS} -cp "$CLASSPATH" org.springframework.bootstrap.cli.SpringBootstrapCli $*

spring-bootstrap/src/main/java/org/springframework/bootstrap/BeanDefinitionLoader.java

@@ -116,8 +116,12 @@ class BeanDefinitionLoader {
 	private int load(Object source) {
 		Assert.notNull(source, "Source must not be null");
 		if (source instanceof Class<?>) {
-			this.annotatedReader.register((Class<?>) source);
-			return 1;
+			Class<?> type = (Class<?>) source;
+			if (isComponent(type)) {
+				this.annotatedReader.register(type);
+				return 1;
+			}
+			return 0;
 		}
 
 		if (source instanceof Resource) {
@@ -149,6 +153,15 @@ class BeanDefinitionLoader {
 		throw new IllegalArgumentException("Invalid source '" + source + "'");
 	}
 
+	private boolean isComponent(Class<?> type) {
+		// Nested anonymous classes are not eligible for registration, nor are groovy
+		// closures
+		if (type.isAnonymousClass() || type.getName().contains("$_closure")) {
+			return false;
+		}
+		return true;
+	}
+
 	/**
 	 * Simple {@link TypeFilter} used to ensure that specified {@link Class} sources are
 	 * not accidentally re-added during scanning.

spring-bootstrap/src/main/java/org/springframework/bootstrap/SpringApplication.java

@@ -297,11 +297,11 @@ public class SpringApplication {
 			}
 		}
 
-		if (context instanceof AbstractApplicationContext) {
+		if (context instanceof AbstractApplicationContext && this.environment != null) {
 			((AbstractApplicationContext) context).setEnvironment(this.environment);
 		}
 
-		if (context instanceof GenericApplicationContext) {
+		if (context instanceof GenericApplicationContext && this.resourceLoader != null) {
 			((GenericApplicationContext) context).setResourceLoader(this.resourceLoader);
 		}
 	}