Adjust tests and fixture

Author: kiril-keranov

src/integration/frameworks_test.go

@@ -671,13 +671,16 @@ func testFrameworks(platform switchblade.Platform, fixtures string) func(*testin
 						}).
 						WithEnv(map[string]string{
 							"BP_JAVA_VERSION":                        "11",
+							"JBP_CONFIG_JAVA_CF_ENV":                 "'{enabled: false}'",
 							"JBP_CONFIG_SPRING_AUTO_RECONFIGURATION": "'{enabled: true}'",
 						}).
 						Execute(name, filepath.Join(fixtures, "frameworks", "auto_reconfiguration_servlet_3"))
 					Expect(err).NotTo(HaveOccurred(), logs.String)
 
 					// Spring Auto-reconfiguration should be detected for Spring apps with services
 					Expect(logs.String()).To(ContainSubstring("Spring Auto-reconfiguration"))
+					// Spring Auto-reconfiguration should only be provided in the absence of Java Cf Env
+					Expect(logs.String()).NotTo(ContainSubstring("Java CF Env"))
 					Eventually(deployment).Should(matchers.Serve(ContainSubstring("")))
 				})
 
@@ -695,6 +698,8 @@ func testFrameworks(platform switchblade.Platform, fixtures string) func(*testin
 						Execute(name, filepath.Join(fixtures, "frameworks", "auto_reconfiguration_servlet_3"))
 					Expect(err).NotTo(HaveOccurred(), logs.String)
 
+					// Java Cf Env should be provided as it is enabled by default for Spring3 apps
+					Expect(logs.String()).To(ContainSubstring("Java CF Env"))
 					// Should not install when explicitly disabled
 					Expect(logs.String()).NotTo(ContainSubstring("Spring Auto-reconfiguration"))
 					Eventually(deployment).Should(matchers.Serve(ContainSubstring("")))

src/integration/spring_boot_test.go

@@ -49,21 +49,6 @@ func testSpringBoot(platform switchblade.Platform, fixtures string) func(*testin
 			})
 		})
 
-		context("with Spring Auto-reconfiguration", func() {
-			it("detects Spring Framework", func() {
-				deployment, logs, err := platform.Deploy.
-					WithEnv(map[string]string{
-						"BP_JAVA_VERSION": "11",
-					}).
-					Execute(name, filepath.Join(fixtures, "containers", "spring_boot_staged"))
-				Expect(err).NotTo(HaveOccurred(), logs.String)
-
-				// Spring auto-reconfiguration should be detected
-				Expect(logs.String()).To(ContainSubstring("Java Buildpack"))
-				Eventually(deployment).Should(matchers.Serve(Not(BeEmpty())))
-			})
-		})
-
 		context("with embedded Tomcat", func() {
 			it("starts successfully", func() {
 				deployment, logs, err := platform.Deploy.
@@ -85,11 +70,13 @@ func testSpringBoot(platform switchblade.Platform, fixtures string) func(*testin
 			it("includes java-cfenv when Spring Boot is detected", func() {
 				deployment, logs, err := platform.Deploy.
 					WithEnv(map[string]string{
-						"BP_JAVA_VERSION":       "11",
-						"JBP_CONFIG_JAVA_CFENV": "{enabled: true}",
+						"BP_JAVA_VERSION":        "11",
+						"JBP_CONFIG_JAVA_CF_ENV": "{enabled: true}",
 					}).
 					Execute(name, filepath.Join(fixtures, "containers", "spring_boot_staged"))
 				Expect(err).NotTo(HaveOccurred(), logs.String)
+				// Fixture is not a Spring3 app, so Java CF Env detect should not pass
+				Expect(logs.String()).NotTo(ContainSubstring("Java CF Env"))
 
 				Eventually(deployment).Should(matchers.Serve(Not(BeEmpty())))
 			})