Require Jenkins 2.414.3 or newer (#1016)

* Refactor to support separate cloud rename page * spotless * update jenkins incremental * add name back in config.jelly * fix jenkins version
jenkinsci · Nov 17, 2023 · fd7fbc8 · fd7fbc8
1 parent f3d9802
commit fd7fbc8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 26 deletions.
diff --git a/pom.xml b/pom.xml
@@ -62,7 +62,7 @@
   <properties>
     <revision>1.6</revision>
     <changelist>-SNAPSHOT</changelist>
-    <jenkins.version>2.387.3</jenkins.version>
+    <jenkins.version>2.414.3</jenkins.version>
     <gitHubRepo>jenkinsci/docker-plugin</gitHubRepo>
     <!-- Our unit-tests that talk to a real docker deamon aren't very stable -->
     <surefire.rerunFailingTestsCount>3</surefire.rerunFailingTestsCount>
@@ -76,7 +76,7 @@
     <dependencies>
       <dependency>
         <groupId>io.jenkins.tools.bom</groupId>
-        <artifactId>bom-2.387.x</artifactId>
+        <artifactId>bom-2.414.x</artifactId>
         <version>2543.vfb_1a_5fb_9496d</version>
         <type>pom</type>
         <scope>import</scope>

diff --git a/src/test/java/com/nirima/jenkins/plugins/docker/DockerCloudTest.java b/src/test/java/com/nirima/jenkins/plugins/docker/DockerCloudTest.java
@@ -19,8 +19,6 @@
 import java.util.Map;
 import java.util.Set;
 import java.util.logging.Level;
-import org.hamcrest.MatcherAssert;
-import org.hamcrest.core.IsIterableContaining;
 import org.jenkinsci.plugins.docker.commons.credentials.DockerServerCredentials;
 import org.jenkinsci.plugins.docker.commons.credentials.DockerServerEndpoint;
 import org.junit.Assert;
@@ -58,34 +56,14 @@ public void testConstructor_0_10_2() {
 
     private static final String LOG_MESSAGE = "Docker cloud requires a non-blank name after Jenkins 2.402";
 
-    @Issue("JENKINS-70729") // Warn if cloud name is empty
-    @Test
-    public void testConstructorWithEmptyName() {
-        lr.record(DockerCloud.class.getName(), Level.ALL).capture(16);
-        DockerCloud cloud =
-                new DockerCloud("", new DockerAPI(new DockerServerEndpoint("uri", "credentialsId")), List.of());
-        Assert.assertEquals(cloud.getDisplayName(), "");
-        MatcherAssert.assertThat(lr.getMessages(), IsIterableContaining.hasItem(LOG_MESSAGE));
-    }
-
-    @Issue("JENKINS-70729") // Warn if cloud name is null
-    @Test
-    public void testConstructorWithNullName() {
-        lr.record(DockerCloud.class.getName(), Level.ALL).capture(16);
-        DockerCloud cloud =
-                new DockerCloud(null, new DockerAPI(new DockerServerEndpoint("uri", "credentialsId")), List.of());
-        Assert.assertEquals(cloud.getDisplayName(), null);
-        MatcherAssert.assertThat(lr.getMessages(), IsIterableContaining.hasItem(LOG_MESSAGE));
-    }
-
     @Issue("JENKINS-70729") // Handle null or empty cloud name
     @Test
     public void testCopyConstructor() {
         lr.record(DockerCloud.class.getName(), Level.ALL).capture(16);
         DockerCloud cloud =
-                new DockerCloud(null, new DockerAPI(new DockerServerEndpoint("uri", "credentialsId")), List.of());
+                new DockerCloud("tmp", new DockerAPI(new DockerServerEndpoint("uri", "credentialsId")), List.of());
+        cloud.name = null;
         Assert.assertEquals(cloud.getDisplayName(), null);
-        MatcherAssert.assertThat(lr.getMessages(), IsIterableContaining.hasItem(LOG_MESSAGE));
         String newName = "docker-cloud-" + Integer.toHexString(cloud.hashCode());
         DockerCloud copy = new DockerCloud(newName, cloud);
         Assert.assertEquals(cloud.getDockerApi(), copy.getDockerApi());