From 1b2f12f3bafbe9c2ac82bf75242105ad87bb3e25 Mon Sep 17 00:00:00 2001
From: Lorenzo Bettini <lorenzo.bettini@gmail.com>
Date: Wed, 4 Dec 2024 14:25:22 +0100
Subject: [PATCH 1/3] re-enabled integration tests for xtend-archetype

---
 full-build.sh                                         |  3 +--
 org.eclipse.xtend.maven.archetype/pom.xml             | 11 +++++++----
 .../resources/META-INF/maven/archetype-metadata.xml   |  2 +-
 .../resources/projects/first/archetype.properties     |  1 +
 4 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/full-build.sh b/full-build.sh
index d1e9da6c545..82330aac90e 100755
--- a/full-build.sh
+++ b/full-build.sh
@@ -2,7 +2,6 @@
 
 MVN_ARGS=(\
   -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn \
-  -Dit-archetype-tests-skip=true \
 )
 
 # args parsing inspired by https://gist.github.com/jehiah/855086
@@ -22,7 +21,7 @@ while [ "$1" != "" ]; do
       exit
       ;;
     --no-tests)
-      MVN_ARGS+=(-DskipTests=true)
+      MVN_ARGS+=(-DskipTests=true -Dit-archetype-tests-skip=true)
       ;;
     --local-repository)
       MVN_ARGS+=" -Dmaven.repo.local=$VALUE"
diff --git a/org.eclipse.xtend.maven.archetype/pom.xml b/org.eclipse.xtend.maven.archetype/pom.xml
index cfca1f4881c..5f6b5c41f9e 100644
--- a/org.eclipse.xtend.maven.archetype/pom.xml
+++ b/org.eclipse.xtend.maven.archetype/pom.xml
@@ -12,7 +12,8 @@
 
 	<properties>
 		<root-dir>${basedir}/..</root-dir>
-		<it-archetype-tests-skip>true</it-archetype-tests-skip>
+		<it-archetype-tests-skip>false</it-archetype-tests-skip>
+		<it-archetype-tests-debug>false</it-archetype-tests-debug>
 	</properties>
 
 	<build>
@@ -53,14 +54,16 @@
 				<artifactId>maven-archetype-plugin</artifactId>
 				<configuration>
 					<skip>${it-archetype-tests-skip}</skip>
-					<debug>true</debug>
+					<debug>${it-archetype-tests-debug}</debug>
 				</configuration>
 				<executions>
 					<execution>
 						<goals>
 							<goal>integration-test</goal>
 						</goals>
-						<configuration><debug>true</debug></configuration>
+						<configuration>
+							<debug>${it-archetype-tests-debug}</debug>
+						</configuration>
 					</execution>
 				</executions>
 			</plugin>
@@ -84,7 +87,7 @@
 				</file>
 			</activation>
 			<properties>
-				<it-tests-skip>true</it-tests-skip>
+				<it-archetype-tests-skip>true</it-archetype-tests-skip>
 			</properties>
 		</profile>
 	</profiles>
diff --git a/org.eclipse.xtend.maven.archetype/src/main/resources/META-INF/maven/archetype-metadata.xml b/org.eclipse.xtend.maven.archetype/src/main/resources/META-INF/maven/archetype-metadata.xml
index c5a92b2250c..ffa54f457b6 100644
--- a/org.eclipse.xtend.maven.archetype/src/main/resources/META-INF/maven/archetype-metadata.xml
+++ b/org.eclipse.xtend.maven.archetype/src/main/resources/META-INF/maven/archetype-metadata.xml
@@ -9,7 +9,7 @@
 			<defaultValue>UTF-8</defaultValue>
 		</requiredProperty>
 		<requiredProperty key="javaVersion">
-			<defaultValue>1.8</defaultValue>
+			<defaultValue>17</defaultValue>
 		</requiredProperty>
 	</requiredProperties>
 	<fileSets>
diff --git a/org.eclipse.xtend.maven.archetype/src/test/resources/projects/first/archetype.properties b/org.eclipse.xtend.maven.archetype/src/test/resources/projects/first/archetype.properties
index be7e417f87a..06f281f182b 100644
--- a/org.eclipse.xtend.maven.archetype/src/test/resources/projects/first/archetype.properties
+++ b/org.eclipse.xtend.maven.archetype/src/test/resources/projects/first/archetype.properties
@@ -2,5 +2,6 @@ sourceEncoding=UTF-8
 groupId=integrationtest.group
 artifactId=integrationtest.artifactId
 version=1.0.0-SNAPSHOT
+javaVersion=17
 package=org.eclipse.xtend.xtend.archetype.integrationtest
 packageInPathFormat=org/eclipse/xtend/xtend-archetype/integrationtest

From 0cf3b4e06f4feb61bcfb64b05f51f7cb7b7f9192 Mon Sep 17 00:00:00 2001
From: Lorenzo Bettini <lorenzo.bettini@gmail.com>
Date: Wed, 4 Dec 2024 14:53:51 +0100
Subject: [PATCH 2/3] use the invoker plugin to install dependencies before
 archetype ITs

---
 org.eclipse.xtend.maven.archetype/pom.xml | 32 +++++++++++++++++++++--
 org.eclipse.xtend.maven.parent/pom.xml    |  4 +++
 org.eclipse.xtend.maven.plugin/pom.xml    |  1 -
 3 files changed, 34 insertions(+), 3 deletions(-)

diff --git a/org.eclipse.xtend.maven.archetype/pom.xml b/org.eclipse.xtend.maven.archetype/pom.xml
index 5f6b5c41f9e..14c01ed9458 100644
--- a/org.eclipse.xtend.maven.archetype/pom.xml
+++ b/org.eclipse.xtend.maven.archetype/pom.xml
@@ -14,6 +14,8 @@
 		<root-dir>${basedir}/..</root-dir>
 		<it-archetype-tests-skip>false</it-archetype-tests-skip>
 		<it-archetype-tests-debug>false</it-archetype-tests-debug>
+		<!-- The local Maven repository for our IT tests -->
+		<localMavenRepositoryPath>${project.basedir}/target/.m2test</localMavenRepositoryPath>
 	</properties>
 
 	<build>
@@ -50,11 +52,37 @@
 			</plugins>
 		</pluginManagement>
 		<plugins>
+			<!-- install test dependencies, including the xtend-maven-plugin in the local .m2 repository
+				that will be used by the integration test to build the test project generated with the archetype -->
+			<plugin>
+				<groupId>org.apache.maven.plugins</groupId>
+				<artifactId>maven-invoker-plugin</artifactId>
+				<configuration>
+					<extraArtifacts>
+						<!-- our BOM is not a dependency (it's just imported), thus, we must
+							install it explicitly in the local maven repository -->
+						<extraArtifact>org.eclipse.xtext:xtext-dev-bom:${project.version}:pom</extraArtifact>
+					</extraArtifacts>
+					<localRepositoryPath>${localMavenRepositoryPath}</localRepositoryPath>
+				</configuration>
+				<executions>
+					<execution>
+						<id>install dependencies in local test repo</id>
+						<phase>pre-integration-test</phase>
+						<goals>
+							<goal>install</goal>
+						</goals>
+						<configuration>
+							<scope>test</scope>
+						</configuration>
+					</execution>
+				</executions>
+			</plugin>
 			<plugin>
 				<artifactId>maven-archetype-plugin</artifactId>
 				<configuration>
-					<skip>${it-archetype-tests-skip}</skip>
 					<debug>${it-archetype-tests-debug}</debug>
+					<localRepositoryPath>${localMavenRepositoryPath}</localRepositoryPath>
 				</configuration>
 				<executions>
 					<execution>
@@ -62,7 +90,7 @@
 							<goal>integration-test</goal>
 						</goals>
 						<configuration>
-							<debug>${it-archetype-tests-debug}</debug>
+							<skip>${it-archetype-tests-skip}</skip>
 						</configuration>
 					</execution>
 				</executions>
diff --git a/org.eclipse.xtend.maven.parent/pom.xml b/org.eclipse.xtend.maven.parent/pom.xml
index b6002a28098..cdea2edec20 100644
--- a/org.eclipse.xtend.maven.parent/pom.xml
+++ b/org.eclipse.xtend.maven.parent/pom.xml
@@ -110,6 +110,10 @@
 					<artifactId>maven-source-plugin</artifactId>
 					<version>3.2.1</version>
 				</plugin>
+				<plugin>
+					<artifactId>maven-invoker-plugin</artifactId>
+					<version>3.5.0</version>
+				</plugin>
 			</plugins>
 		</pluginManagement>
 	</build>
diff --git a/org.eclipse.xtend.maven.plugin/pom.xml b/org.eclipse.xtend.maven.plugin/pom.xml
index f006b463a9e..c8798f6eda1 100644
--- a/org.eclipse.xtend.maven.plugin/pom.xml
+++ b/org.eclipse.xtend.maven.plugin/pom.xml
@@ -174,7 +174,6 @@
 			<plugin>
 				<groupId>org.apache.maven.plugins</groupId>
 				<artifactId>maven-invoker-plugin</artifactId>
-				<version>3.5.0</version>
 				<configuration>
 					<extraArtifacts>
 						<!-- our BOM is not a dependency (it's just imported), thus, we must

From 6fc2ee3ebd795db27eda0dc704209ff755a0eb82 Mon Sep 17 00:00:00 2001
From: Lorenzo Bettini <lorenzo.bettini@gmail.com>
Date: Wed, 4 Dec 2024 15:08:37 +0100
Subject: [PATCH 3/3] added README file

---
 org.eclipse.xtend.maven.archetype/README.md | 5 +++++
 1 file changed, 5 insertions(+)
 create mode 100644 org.eclipse.xtend.maven.archetype/README.md

diff --git a/org.eclipse.xtend.maven.archetype/README.md b/org.eclipse.xtend.maven.archetype/README.md
new file mode 100644
index 00000000000..aa2845d0c07
--- /dev/null
+++ b/org.eclipse.xtend.maven.archetype/README.md
@@ -0,0 +1,5 @@
+To run the integration tests, the `maven.home` variable must defined, i.e., passed to the Maven build with `-Dmaven.home=...`.
+
+For example, if the Maven executable is `/usr/local/bin/mvn`, you need to pass `-Dmaven.home=/usr/local` (i.e., without the `bin` subdirectory).
+
+The Maven Invoker Plugin's goal `install` is attached to the `pre-integration-test` phase, in order to install the needed Maven artifacts of Xtext/Xtend into the `target/.m2test` folder, which will be then used by the integration test to build the test project generated from the archetype.
\ No newline at end of file