From 28a34f8c563f9608847536da6e04ad046f6c5ea3 Mon Sep 17 00:00:00 2001
From: Mark Waite <mark.earl.waite@gmail.com>
Date: Thu, 3 Aug 2023 12:51:54 -0600
Subject: [PATCH] Run parallel tests in CI (#198)

* Remove unused dependency references

* Run parallel tests in CI

The plugin includes several long running tests that benefit from using
at least one JVM per core on the test machine.
---
 Jenkinsfile | 17 +++++++++++++----
 pom.xml     | 11 -----------
 2 files changed, 13 insertions(+), 15 deletions(-)

diff --git a/Jenkinsfile b/Jenkinsfile
index c83ea212..633fc886 100644
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -1,7 +1,16 @@
 // Specify configurations explicitly to test against newer LTS.
 // See also https://github.com/jenkins-infra/pipeline-library/pull/145
 
-buildPlugin(useContainerAgent: true, configurations: [
-  [ platform: 'linux', jdk: '11' ],
-  [ platform: 'windows', jdk: '17' ],
-])
+buildPlugin(
+  // Run a JVM per core in tests
+  forkCount: '1C',
+  // Container agents start faster and are easier to administer
+  useContainerAgent: true,
+  // Show failures on all configurations
+  failFast: false,
+  // Test Java 11 and Java 17
+  configurations: [
+    [platform: 'linux',   jdk: '17'], // Linux first for coverage report on ci.jenkins.io
+    [platform: 'windows', jdk: '11'],
+  ]
+)
diff --git a/pom.xml b/pom.xml
index b6ab6e5c..623a4ed7 100644
--- a/pom.xml
+++ b/pom.xml
@@ -165,17 +165,6 @@
           <scope>import</scope>
           <type>pom</type>
         </dependency>
-        <dependency>
-          <groupId>javax.annotation</groupId>
-          <artifactId>javax.annotation-api</artifactId>
-          <version>1.3.2</version>
-        </dependency>
-        <dependency>
-          <groupId>commons-net</groupId>
-          <artifactId>commons-net</artifactId>
-          <version>3.9.0</version>
-          <scope>test</scope>
-        </dependency>
       </dependencies>
     </dependencyManagement>