Add negative test for Play Framework hybrid application rejection

Convert skipped test into active validation that verifies the buildpack
correctly rejects applications with ambiguous Play version markers (both
2.1 and 2.2). This validates security/validation behavior and prevents
regressions in version detection logic.

Test validates error message contains 'Play Framework application version
cannot be determined' as expected from factory.rb:47.

Author: ramonskie

src/integration/play_test.go

@@ -0,0 +1,141 @@
+package integration_test
+
+import (
+	"path/filepath"
+	"testing"
+
+	"github.com/cloudfoundry/switchblade"
+	"github.com/sclevine/spec"
+
+	. "github.com/onsi/gomega"
+)
+
+func testPlay(platform switchblade.Platform, fixtures string) func(*testing.T, spec.G, spec.S) {
+	return func(t *testing.T, context spec.G, it spec.S) {
+		var (
+			Expect = NewWithT(t).Expect
+			name   string
+		)
+
+		it.Before(func() {
+			var err error
+			name, err = switchblade.RandomName()
+			Expect(err).NotTo(HaveOccurred())
+		})
+
+		it.After(func() {
+			if name != "" {
+				Expect(platform.Delete.Execute(name)).To(Succeed())
+			}
+		})
+
+		context("with Play Framework 2.0", func() {
+			it("successfully deploys a Play 2.0 dist application", func() {
+				deployment, logs, err := platform.Deploy.
+					WithEnv(map[string]string{
+						"BP_JAVA_VERSION": "11",
+					}).
+					Execute(name, filepath.Join(fixtures, "container_play_2.0_dist"))
+				Expect(err).NotTo(HaveOccurred(), logs.String)
+
+				Expect(logs.String()).To(ContainSubstring("Java Buildpack"))
+				Expect(deployment.ExternalURL).NotTo(BeEmpty())
+			})
+		})
+
+		context("with Play Framework 2.1", func() {
+			it("successfully deploys a Play 2.1 dist application", func() {
+				deployment, logs, err := platform.Deploy.
+					WithEnv(map[string]string{
+						"BP_JAVA_VERSION": "11",
+					}).
+					Execute(name, filepath.Join(fixtures, "container_play_2.1_dist"))
+				Expect(err).NotTo(HaveOccurred(), logs.String)
+
+				Expect(logs.String()).To(ContainSubstring("Java Buildpack"))
+				Expect(deployment.ExternalURL).NotTo(BeEmpty())
+			})
+
+			it("successfully deploys a staged Play 2.1 application", func() {
+				deployment, logs, err := platform.Deploy.
+					WithEnv(map[string]string{
+						"BP_JAVA_VERSION": "11",
+					}).
+					Execute(name, filepath.Join(fixtures, "container_play_2.1_staged"))
+				Expect(err).NotTo(HaveOccurred(), logs.String)
+
+				Expect(logs.String()).To(ContainSubstring("Java Buildpack"))
+				Expect(deployment.ExternalURL).NotTo(BeEmpty())
+			})
+		})
+
+		context("with Play Framework 2.2", func() {
+			it("successfully deploys a Play 2.2 dist application", func() {
+				deployment, logs, err := platform.Deploy.
+					WithEnv(map[string]string{
+						"BP_JAVA_VERSION": "11",
+					}).
+					Execute(name, filepath.Join(fixtures, "container_play_2.2_dist"))
+				Expect(err).NotTo(HaveOccurred(), logs.String)
+
+				Expect(logs.String()).To(ContainSubstring("Java Buildpack"))
+				Expect(deployment.ExternalURL).NotTo(BeEmpty())
+			})
+
+			it("successfully deploys a staged Play 2.2 application", func() {
+				deployment, logs, err := platform.Deploy.
+					WithEnv(map[string]string{
+						"BP_JAVA_VERSION": "11",
+					}).
+					Execute(name, filepath.Join(fixtures, "container_play_2.2_staged"))
+				Expect(err).NotTo(HaveOccurred(), logs.String)
+
+				Expect(logs.String()).To(ContainSubstring("Java Buildpack"))
+				Expect(deployment.ExternalURL).NotTo(BeEmpty())
+			})
+
+			it("handles Play 2.2 application without bat file", func() {
+				deployment, logs, err := platform.Deploy.
+					WithEnv(map[string]string{
+						"BP_JAVA_VERSION": "11",
+					}).
+					Execute(name, filepath.Join(fixtures, "container_play_2.2_minus_bat_file"))
+				Expect(err).NotTo(HaveOccurred(), logs.String)
+
+				Expect(logs.String()).To(ContainSubstring("Java Buildpack"))
+				Expect(deployment.ExternalURL).NotTo(BeEmpty())
+			})
+		})
+
+		context("with hybrid Play applications", func() {
+			it("rejects ambiguous Play 2.1/2.2 hybrid application", func() {
+				// This test validates that the buildpack correctly rejects ambiguous fixtures
+				// that contain both Play 2.1 (staged/) and Play 2.2 (lib/) version markers.
+				// This is expected security/validation behavior to prevent deployment of
+				// malformed applications. See: lib/java_buildpack/util/play/factory.rb:47
+				_, logs, err := platform.Deploy.
+					WithEnv(map[string]string{
+						"BP_JAVA_VERSION": "11",
+					}).
+					Execute(name, filepath.Join(fixtures, "container_play_2.1_2.2_hybrid"))
+				Expect(err).To(HaveOccurred(), logs.String)
+
+				Expect(logs.String()).To(ContainSubstring("Play Framework application version cannot be determined"))
+			})
+		})
+
+		context("with JRE version selection", func() {
+			it("deploys Play application with Java 17", func() {
+				deployment, logs, err := platform.Deploy.
+					WithEnv(map[string]string{
+						"BP_JAVA_VERSION": "17",
+					}).
+					Execute(name, filepath.Join(fixtures, "container_play_2.2_dist"))
+				Expect(err).NotTo(HaveOccurred(), logs.String)
+
+				Expect(logs.String()).To(ContainSubstring("Java Buildpack"))
+				Expect(deployment.ExternalURL).NotTo(BeEmpty())
+			})
+		})
+	}
+}