From 0145820b840da4410e18366a6242ae06fde6d41f Mon Sep 17 00:00:00 2001
From: Basil Crow <me@basilcrow.com>
Date: Thu, 16 Feb 2023 14:42:42 -0800
Subject: [PATCH] Fix opt-in for Maven Failsafe plugin

---
 .../test/hook/AnalysisPomExecutionHook.java   | 32 +++++++++++++++++++
 ...luginWithFailsafeIntegrationTestsHook.java |  3 +-
 2 files changed, 34 insertions(+), 1 deletion(-)
 create mode 100644 src/main/java/org/jenkins/tools/test/hook/AnalysisPomExecutionHook.java

diff --git a/src/main/java/org/jenkins/tools/test/hook/AnalysisPomExecutionHook.java b/src/main/java/org/jenkins/tools/test/hook/AnalysisPomExecutionHook.java
new file mode 100644
index 000000000..87e2d7c82
--- /dev/null
+++ b/src/main/java/org/jenkins/tools/test/hook/AnalysisPomExecutionHook.java
@@ -0,0 +1,32 @@
+package org.jenkins.tools.test.hook;
+
+import java.util.Map;
+import java.util.Set;
+import org.jenkins.tools.test.model.PomData;
+
+/**
+ * Custom execution hook for plugins whose parent is {@code org.jvnet.hudson.plugins:analysis-pom}.
+ * These plugins use Maven Failsafe Plugin in their test suites.
+ */
+public class AnalysisPomExecutionHook extends PluginWithFailsafeIntegrationTestsHook {
+
+    private static final Set<String> ARTIFACT_IDS =
+            Set.of(
+                    "analysis-model-api",
+                    "bootstrap5-api",
+                    "checks-api",
+                    "echarts-api",
+                    "font-awesome-api",
+                    "forensics-api",
+                    "jquery3-api",
+                    "plugin-util-api",
+                    "popper2-api");
+
+    @Override
+    public boolean check(Map<String, Object> info) {
+        PomData data = (PomData) info.get("pomData");
+        return "io.jenkins.plugins".equals(data.groupId)
+                && ARTIFACT_IDS.contains(data.artifactId)
+                && "hpi".equals(data.getPackaging());
+    }
+}
diff --git a/src/main/java/org/jenkins/tools/test/hook/PluginWithFailsafeIntegrationTestsHook.java b/src/main/java/org/jenkins/tools/test/hook/PluginWithFailsafeIntegrationTestsHook.java
index cefb3fea4..28beb488b 100644
--- a/src/main/java/org/jenkins/tools/test/hook/PluginWithFailsafeIntegrationTestsHook.java
+++ b/src/main/java/org/jenkins/tools/test/hook/PluginWithFailsafeIntegrationTestsHook.java
@@ -7,7 +7,8 @@
  * Workaround for those plugins with integration tests since they need execute the
  * failsafe:integration-test goal before execution.
  */
-public class PluginWithFailsafeIntegrationTestsHook extends PluginWithIntegrationTestsHook {
+public abstract class PluginWithFailsafeIntegrationTestsHook
+        extends PluginWithIntegrationTestsHook {
 
     @Override
     public Collection<String> getGoals() {