From f1feff55b7cfddb0a18081f4a7a6adc358625d2b Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Mon, 5 Feb 2024 10:13:07 +0000 Subject: [PATCH 01/11] Bump org.assertj:assertj-core from 3.25.1 to 3.25.3 Bumps [org.assertj:assertj-core](https://github.com/assertj/assertj) from 3.25.1 to 3.25.3. - [Release notes](https://github.com/assertj/assertj/releases) - [Commits](https://github.com/assertj/assertj/compare/assertj-build-3.25.1...assertj-build-3.25.3) --- updated-dependencies: - dependency-name: org.assertj:assertj-core dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index 86dfe65..019b13c 100644 --- a/build.gradle +++ b/build.gradle @@ -32,7 +32,7 @@ dependencies { testImplementation ("org.apache.jclouds:jclouds-core:${jcloudsVersion}:tests") testImplementation ("org.apache.jclouds.driver:jclouds-slf4j:${jcloudsVersion}") - testImplementation ('org.assertj:assertj-core:3.25.1') + testImplementation ('org.assertj:assertj-core:3.25.3') testImplementation ('ch.qos.logback:logback-core:1.4.14') testImplementation ('com.squareup.okhttp3:mockwebserver:4.12.0') testImplementation ('org.testng:testng:7.9.0') From 5709a515b9aa1f92aa57bbbd882be349a3a6c574 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Mon, 8 Apr 2024 10:57:45 +0000 Subject: [PATCH 02/11] Bump io.github.gradle-nexus.publish-plugin from 1.3.0 to 2.0.0 Bumps io.github.gradle-nexus.publish-plugin from 1.3.0 to 2.0.0. --- updated-dependencies: - dependency-name: io.github.gradle-nexus.publish-plugin dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index 86dfe65..fb8d011 100644 --- a/build.gradle +++ b/build.gradle @@ -2,7 +2,7 @@ plugins { id 'java' id 'maven-publish' id 'com.github.johnrengelman.shadow' version '7.1.1' - id "io.github.gradle-nexus.publish-plugin" version "1.3.0" + id "io.github.gradle-nexus.publish-plugin" version "2.0.0" } apply from: "$rootDir/gradle/additional-artifacts.gradle" From ebfad1e59f100ec829a7b3f70f815c2cba49c409 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Thu, 18 Apr 2024 10:53:49 +0000 Subject: [PATCH 03/11] Bump ch.qos.logback:logback-classic from 1.4.14 to 1.5.6 Bumps [ch.qos.logback:logback-classic](https://github.com/qos-ch/logback) from 1.4.14 to 1.5.6. - [Commits](https://github.com/qos-ch/logback/compare/v_1.4.14...v_1.5.6) --- updated-dependencies: - dependency-name: ch.qos.logback:logback-classic dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index 86dfe65..e7feb35 100644 --- a/build.gradle +++ b/build.gradle @@ -36,7 +36,7 @@ dependencies { testImplementation ('ch.qos.logback:logback-core:1.4.14') testImplementation ('com.squareup.okhttp3:mockwebserver:4.12.0') testImplementation ('org.testng:testng:7.9.0') - testImplementation ('ch.qos.logback:logback-classic:1.4.14') + testImplementation ('ch.qos.logback:logback-classic:1.5.6') } ext.compatibilityVersion = JavaVersion.VERSION_11 From eee7e71ae37aa7db1f339dc700ee6546168bccbe Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Mon, 22 Apr 2024 15:13:06 +0000 Subject: [PATCH 04/11] Bump ch.qos.logback:logback-core from 1.4.14 to 1.5.6 Bumps [ch.qos.logback:logback-core](https://github.com/qos-ch/logback) from 1.4.14 to 1.5.6. - [Commits](https://github.com/qos-ch/logback/compare/v_1.4.14...v_1.5.6) --- updated-dependencies: - dependency-name: ch.qos.logback:logback-core dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index 85db01f..69c4f88 100644 --- a/build.gradle +++ b/build.gradle @@ -33,7 +33,7 @@ dependencies { testImplementation ("org.apache.jclouds:jclouds-core:${jcloudsVersion}:tests") testImplementation ("org.apache.jclouds.driver:jclouds-slf4j:${jcloudsVersion}") testImplementation ('org.assertj:assertj-core:3.25.3') - testImplementation ('ch.qos.logback:logback-core:1.4.14') + testImplementation ('ch.qos.logback:logback-core:1.5.6') testImplementation ('com.squareup.okhttp3:mockwebserver:4.12.0') testImplementation ('org.testng:testng:7.9.0') testImplementation ('ch.qos.logback:logback-classic:1.5.6') From 0054a70af7f253b4d40233a81fd30930b132d0a1 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Mon, 22 Apr 2024 15:13:06 +0000 Subject: [PATCH 05/11] Bump org.testng:testng from 7.9.0 to 7.10.1 Bumps [org.testng:testng](https://github.com/testng-team/testng) from 7.9.0 to 7.10.1. - [Release notes](https://github.com/testng-team/testng/releases) - [Changelog](https://github.com/testng-team/testng/blob/master/CHANGES.txt) - [Commits](https://github.com/testng-team/testng/compare/7.9.0...7.10.1) --- updated-dependencies: - dependency-name: org.testng:testng dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index 85db01f..d7c93b0 100644 --- a/build.gradle +++ b/build.gradle @@ -35,7 +35,7 @@ dependencies { testImplementation ('org.assertj:assertj-core:3.25.3') testImplementation ('ch.qos.logback:logback-core:1.4.14') testImplementation ('com.squareup.okhttp3:mockwebserver:4.12.0') - testImplementation ('org.testng:testng:7.9.0') + testImplementation ('org.testng:testng:7.10.1') testImplementation ('ch.qos.logback:logback-classic:1.5.6') } From f16370c61aa33b945cbffe953e4cf9ece194d87b Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Mon, 29 Apr 2024 10:19:04 +0000 Subject: [PATCH 06/11] Bump org.testng:testng from 7.10.1 to 7.10.2 Bumps [org.testng:testng](https://github.com/testng-team/testng) from 7.10.1 to 7.10.2. - [Release notes](https://github.com/testng-team/testng/releases) - [Changelog](https://github.com/testng-team/testng/blob/master/CHANGES.txt) - [Commits](https://github.com/testng-team/testng/compare/7.10.1...7.10.2) --- updated-dependencies: - dependency-name: org.testng:testng dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index 88b1f25..bb7ebb0 100644 --- a/build.gradle +++ b/build.gradle @@ -35,7 +35,7 @@ dependencies { testImplementation ('org.assertj:assertj-core:3.25.3') testImplementation ('ch.qos.logback:logback-core:1.5.6') testImplementation ('com.squareup.okhttp3:mockwebserver:4.12.0') - testImplementation ('org.testng:testng:7.10.1') + testImplementation ('org.testng:testng:7.10.2') testImplementation ('ch.qos.logback:logback-classic:1.5.6') } From cc2c6c0ca4cb8e3c4c2d151fed1b3b70ec80af98 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Mon, 3 Jun 2024 10:22:34 +0000 Subject: [PATCH 07/11] Bump autoValueVersion from 1.10.4 to 1.11.0 Bumps `autoValueVersion` from 1.10.4 to 1.11.0. Updates `com.google.auto.value:auto-value-annotations` from 1.10.4 to 1.11.0 - [Release notes](https://github.com/google/auto/releases) - [Commits](https://github.com/google/auto/compare/auto-value-1.10.4...auto-value-1.11.0) Updates `com.google.auto.value:auto-value` from 1.10.4 to 1.11.0 - [Release notes](https://github.com/google/auto/releases) - [Commits](https://github.com/google/auto/compare/auto-value-1.10.4...auto-value-1.11.0) --- updated-dependencies: - dependency-name: com.google.auto.value:auto-value-annotations dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: com.google.auto.value:auto-value dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index bb7ebb0..6f22a0c 100644 --- a/build.gradle +++ b/build.gradle @@ -16,7 +16,7 @@ repositories { dependencies { ext.jcloudsVersion = '2.5.0' - ext.autoValueVersion = '1.10.4' + ext.autoValueVersion = '1.11.0' ext.autoServiceVersion = '1.1.1' ext.guiceVersion = '5.1.0' From 1b5d5bfa06a61c9541ecedecc961c7bdfb703c82 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Wed, 10 Jul 2024 10:05:12 +0000 Subject: [PATCH 08/11] Bump org.assertj:assertj-core from 3.25.3 to 3.26.3 Bumps [org.assertj:assertj-core](https://github.com/assertj/assertj) from 3.25.3 to 3.26.3. - [Release notes](https://github.com/assertj/assertj/releases) - [Commits](https://github.com/assertj/assertj/compare/assertj-build-3.25.3...assertj-build-3.26.3) --- updated-dependencies: - dependency-name: org.assertj:assertj-core dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index bb7ebb0..bc11ac5 100644 --- a/build.gradle +++ b/build.gradle @@ -32,7 +32,7 @@ dependencies { testImplementation ("org.apache.jclouds:jclouds-core:${jcloudsVersion}:tests") testImplementation ("org.apache.jclouds.driver:jclouds-slf4j:${jcloudsVersion}") - testImplementation ('org.assertj:assertj-core:3.25.3') + testImplementation ('org.assertj:assertj-core:3.26.3') testImplementation ('ch.qos.logback:logback-core:1.5.6') testImplementation ('com.squareup.okhttp3:mockwebserver:4.12.0') testImplementation ('org.testng:testng:7.10.2') From beaeb459659634ab44b815f83ecbee8679f09a0c Mon Sep 17 00:00:00 2001 From: Akhil Mehra Date: Sat, 13 Jul 2024 10:13:01 +1000 Subject: [PATCH 09/11] Issue-315: Processing Queue ID as a long value Queue ID retured from Jenkins can be a Long value. This is due to a recent update which resulted in the queue IDs returning a random long values instead of sequential integers. The jenkins-rest API expects this to be an Integer value. As a result, calls to jobsApi().buildWithParameters and then attempting to retrieve the queue ID will fail/return null. This PR fixes the JobsApi and QueueApi to deal with the new long values. Please refer to issue [315](https://github.com/cdancy/jenkins-rest/issues/315) --- ...IntegerResponse.java => LongResponse.java} | 20 +++++------ .../rest/fallbacks/JenkinsFallbacks.java | 8 ++--- .../cdancy/jenkins/rest/features/JobsApi.java | 10 +++--- .../jenkins/rest/features/QueueApi.java | 8 ++--- .../rest/parsers/LocationToQueueId.java | 10 +++--- .../jenkins/rest/BaseJenkinsApiLiveTest.java | 2 +- .../rest/features/JobsApiLiveTest.java | 36 +++++++++---------- .../rest/features/JobsApiMockTest.java | 24 ++++++------- .../rest/features/QueueApiLiveTest.java | 28 +++++++-------- 9 files changed, 73 insertions(+), 73 deletions(-) rename src/main/java/com/cdancy/jenkins/rest/domain/common/{IntegerResponse.java => LongResponse.java} (77%) diff --git a/src/main/java/com/cdancy/jenkins/rest/domain/common/IntegerResponse.java b/src/main/java/com/cdancy/jenkins/rest/domain/common/LongResponse.java similarity index 77% rename from src/main/java/com/cdancy/jenkins/rest/domain/common/IntegerResponse.java rename to src/main/java/com/cdancy/jenkins/rest/domain/common/LongResponse.java index a8fe636..bed7507 100644 --- a/src/main/java/com/cdancy/jenkins/rest/domain/common/IntegerResponse.java +++ b/src/main/java/com/cdancy/jenkins/rest/domain/common/LongResponse.java @@ -26,21 +26,21 @@ import com.google.auto.value.AutoValue; /** - * Integer response to be returned when an endpoint returns - * an integer. - * - *

When the HTTP response code is valid the `value` parameter will - * be set to the integer value while a non-valid response has the `value` set to + * Long response to be returned when an endpoint returns + * an long. + * + *

When the HTTP response code is valid the `value` parameter will + * be set to the long value while a non-valid response has the `value` set to * null along with any potential `error` objects returned from Jenkins. */ @AutoValue -public abstract class IntegerResponse implements Value, ErrorsHolder { - +public abstract class LongResponse implements Value, ErrorsHolder { + @SerializedNames({ "value", "errors" }) - public static IntegerResponse create(@Nullable final Integer value, + public static LongResponse create(@Nullable final Long value, final List errors) { - - return new AutoValue_IntegerResponse(value, + + return new AutoValue_LongResponse(value, JenkinsUtils.nullToEmpty(errors)); } } diff --git a/src/main/java/com/cdancy/jenkins/rest/fallbacks/JenkinsFallbacks.java b/src/main/java/com/cdancy/jenkins/rest/fallbacks/JenkinsFallbacks.java index c154a10..b1b9ef8 100644 --- a/src/main/java/com/cdancy/jenkins/rest/fallbacks/JenkinsFallbacks.java +++ b/src/main/java/com/cdancy/jenkins/rest/fallbacks/JenkinsFallbacks.java @@ -22,7 +22,7 @@ import static org.jclouds.http.HttpUtils.returnValueOnCodeOrNull; -import com.cdancy.jenkins.rest.domain.common.IntegerResponse; +import com.cdancy.jenkins.rest.domain.common.LongResponse; import com.cdancy.jenkins.rest.domain.common.RequestStatus; import com.cdancy.jenkins.rest.domain.common.Error; import com.cdancy.jenkins.rest.domain.crumb.Crumb; @@ -59,14 +59,14 @@ public Object createOrPropagate(final Throwable throwable) { } } - public static final class IntegerResponseOnError implements Fallback { + public static final class LongResponseOnError implements Fallback { @Override public Object createOrPropagate(final Throwable throwable) { checkNotNull(throwable, "throwable"); try { - return IntegerResponse.create(null, getErrors(throwable)); + return LongResponse.create(null, getErrors(throwable)); } catch (JsonSyntaxException e) { - return IntegerResponse.create(null, getErrors(e)); + return LongResponse.create(null, getErrors(e)); } } } diff --git a/src/main/java/com/cdancy/jenkins/rest/features/JobsApi.java b/src/main/java/com/cdancy/jenkins/rest/features/JobsApi.java index b1f9149..8deb0d9 100644 --- a/src/main/java/com/cdancy/jenkins/rest/features/JobsApi.java +++ b/src/main/java/com/cdancy/jenkins/rest/features/JobsApi.java @@ -46,7 +46,7 @@ import org.jclouds.rest.annotations.ResponseParser; import com.cdancy.jenkins.rest.binders.BindMapToForm; -import com.cdancy.jenkins.rest.domain.common.IntegerResponse; +import com.cdancy.jenkins.rest.domain.common.LongResponse; import com.cdancy.jenkins.rest.domain.common.RequestStatus; import com.cdancy.jenkins.rest.fallbacks.JenkinsFallbacks; import com.cdancy.jenkins.rest.filters.JenkinsAuthenticationFilter; @@ -164,11 +164,11 @@ boolean disable(@Nullable @PathParam("optionalFolderPath") @ParamParser(Optional @Named("jobs:build") @Path("{optionalFolderPath}job/{name}/build") - @Fallback(JenkinsFallbacks.IntegerResponseOnError.class) + @Fallback(JenkinsFallbacks.LongResponseOnError.class) @ResponseParser(LocationToQueueId.class) @Consumes("application/unknown") @POST - IntegerResponse build(@Nullable @PathParam("optionalFolderPath") @ParamParser(OptionalFolderPathParser.class) String optionalFolderPath, + LongResponse build(@Nullable @PathParam("optionalFolderPath") @ParamParser(OptionalFolderPathParser.class) String optionalFolderPath, @PathParam("name") String jobName); @Named("jobs:stop-build") @@ -203,11 +203,11 @@ RequestStatus kill(@Nullable @PathParam("optionalFolderPath") @ParamParser(Optio @Named("jobs:build-with-params") @Path("{optionalFolderPath}job/{name}/buildWithParameters") - @Fallback(JenkinsFallbacks.IntegerResponseOnError.class) + @Fallback(JenkinsFallbacks.LongResponseOnError.class) @ResponseParser(LocationToQueueId.class) @Consumes("application/unknown") @POST - IntegerResponse buildWithParameters(@Nullable @PathParam("optionalFolderPath") @ParamParser(OptionalFolderPathParser.class) String optionalFolderPath, + LongResponse buildWithParameters(@Nullable @PathParam("optionalFolderPath") @ParamParser(OptionalFolderPathParser.class) String optionalFolderPath, @PathParam("name") String jobName, @Nullable @BinderParam(BindMapToForm.class) Map> properties); diff --git a/src/main/java/com/cdancy/jenkins/rest/features/QueueApi.java b/src/main/java/com/cdancy/jenkins/rest/features/QueueApi.java index bbc1fd3..513ad3a 100644 --- a/src/main/java/com/cdancy/jenkins/rest/features/QueueApi.java +++ b/src/main/java/com/cdancy/jenkins/rest/features/QueueApi.java @@ -52,7 +52,7 @@ public interface QueueApi { /** * Get a specific queue item. - * + * * Queue items are builds that have been scheduled to run, but are waiting for a slot. * You can poll the queueItem that corresponds to a build to detect whether the build is still pending or is executing. * @param queueId The queue id value as returned by the JobsApi build or buildWithParameters methods. @@ -61,11 +61,11 @@ public interface QueueApi { @Named("queue:item") @Path("/item/{queueId}/api/json") @GET - QueueItem queueItem(@PathParam("queueId") int queueId); + QueueItem queueItem(@PathParam("queueId") long queueId); /** * Cancel a queue item before it gets built. - * + * * @param id The queue id value of the queue item to cancel. * This is the value is returned by the JobsApi build or buildWithParameters methods. * @return Always returns true due to JENKINS-21311. @@ -75,5 +75,5 @@ public interface QueueApi { @Fallback(JenkinsFallbacks.JENKINS_21311.class) @ResponseParser(RequestStatusParser.class) @POST - RequestStatus cancel(@FormParam("id") int id); + RequestStatus cancel(@FormParam("id") long id); } diff --git a/src/main/java/com/cdancy/jenkins/rest/parsers/LocationToQueueId.java b/src/main/java/com/cdancy/jenkins/rest/parsers/LocationToQueueId.java index 15b9174..78c0d0c 100644 --- a/src/main/java/com/cdancy/jenkins/rest/parsers/LocationToQueueId.java +++ b/src/main/java/com/cdancy/jenkins/rest/parsers/LocationToQueueId.java @@ -28,17 +28,17 @@ import com.google.common.collect.Lists; import com.cdancy.jenkins.rest.domain.common.Error; -import com.cdancy.jenkins.rest.domain.common.IntegerResponse; +import com.cdancy.jenkins.rest.domain.common.LongResponse; /** * Created by dancc on 3/11/16. */ @Singleton -public class LocationToQueueId implements Function { +public class LocationToQueueId implements Function { private static final Pattern pattern = Pattern.compile("^.*/queue/item/(\\d+)/$"); - public IntegerResponse apply(HttpResponse response) { + public LongResponse apply(HttpResponse response) { if (response == null) { throw new RuntimeException("Unexpected NULL HttpResponse object"); } @@ -47,12 +47,12 @@ public IntegerResponse apply(HttpResponse response) { if (url != null) { Matcher matcher = pattern.matcher(url); if (matcher.find() && matcher.groupCount() == 1) { - return IntegerResponse.create(Integer.valueOf(matcher.group(1)), null); + return LongResponse.create(Long.valueOf(matcher.group(1)), null); } } final Error error = Error.create(null, "No queue item Location header could be found despite getting a valid HTTP response.", NumberFormatException.class.getCanonicalName()); - return IntegerResponse.create(null, Lists.newArrayList(error)); + return LongResponse.create(null, Lists.newArrayList(error)); } } diff --git a/src/test/java/com/cdancy/jenkins/rest/BaseJenkinsApiLiveTest.java b/src/test/java/com/cdancy/jenkins/rest/BaseJenkinsApiLiveTest.java index 588a9f9..3dff87a 100644 --- a/src/test/java/com/cdancy/jenkins/rest/BaseJenkinsApiLiveTest.java +++ b/src/test/java/com/cdancy/jenkins/rest/BaseJenkinsApiLiveTest.java @@ -80,7 +80,7 @@ protected Iterable setupModules() { * The caller has to check the value of queueItem.executable, and if it is null, the queue item is still pending. * */ - protected QueueItem getRunningQueueItem(int queueId) throws InterruptedException { + protected QueueItem getRunningQueueItem(long queueId) throws InterruptedException { int max = 10; QueueItem queueItem = api.queueApi().queueItem(queueId); while (max > 0) { diff --git a/src/test/java/com/cdancy/jenkins/rest/features/JobsApiLiveTest.java b/src/test/java/com/cdancy/jenkins/rest/features/JobsApiLiveTest.java index 61a7648..bb8013b 100644 --- a/src/test/java/com/cdancy/jenkins/rest/features/JobsApiLiveTest.java +++ b/src/test/java/com/cdancy/jenkins/rest/features/JobsApiLiveTest.java @@ -21,7 +21,7 @@ import java.util.Map; import com.cdancy.jenkins.rest.BaseJenkinsApiLiveTest; -import com.cdancy.jenkins.rest.domain.common.IntegerResponse; +import com.cdancy.jenkins.rest.domain.common.LongResponse; import com.cdancy.jenkins.rest.domain.common.RequestStatus; import com.cdancy.jenkins.rest.domain.job.*; import com.cdancy.jenkins.rest.domain.plugins.Plugin; @@ -35,8 +35,8 @@ @Test(groups = "live", testName = "JobsApiLiveTest", singleThreaded = true) public class JobsApiLiveTest extends BaseJenkinsApiLiveTest { - private IntegerResponse queueId; - private IntegerResponse queueIdForAnotherJob; + private LongResponse queueId; + private LongResponse queueIdForAnotherJob; private Integer buildNumber; private static final String FOLDER_PLUGIN_NAME = "cloudbees-folder"; private static final String FOLDER_PLUGIN_VERSION = "latest"; @@ -58,7 +58,7 @@ public void testStopFreeStyleBuild() throws InterruptedException { String config = payloadFromResource("/freestyle-project-sleep-10-task.xml"); RequestStatus createStatus = api().create(null, FREESTYLE_JOB_NAME, config); assertTrue(createStatus.value()); - IntegerResponse qId = api().build(null, FREESTYLE_JOB_NAME); + LongResponse qId = api().build(null, FREESTYLE_JOB_NAME); assertNotNull(qId); assertTrue(qId.value() > 0); QueueItem queueItem = getRunningQueueItem(qId.value()); @@ -73,7 +73,7 @@ public void testStopFreeStyleBuild() throws InterruptedException { @Test(dependsOnMethods = "testStopFreeStyleBuild") public void testTermFreeStyleBuild() throws InterruptedException { - IntegerResponse qId = api().build(null, FREESTYLE_JOB_NAME); + LongResponse qId = api().build(null, FREESTYLE_JOB_NAME); assertNotNull(qId); assertTrue(qId.value() > 0); QueueItem queueItem = getRunningQueueItem(qId.value()); @@ -95,7 +95,7 @@ public void testTermFreeStyleBuild() throws InterruptedException { @Test(dependsOnMethods = "testTermFreeStyleBuild") public void testKillFreeStyleBuild() throws InterruptedException { - IntegerResponse qId = api().build(null, FREESTYLE_JOB_NAME); + LongResponse qId = api().build(null, FREESTYLE_JOB_NAME); assertNotNull(qId); assertTrue(qId.value() > 0); QueueItem queueItem = getRunningQueueItem(qId.value()); @@ -126,7 +126,7 @@ public void testStopPipelineBuild() throws InterruptedException { String config = payloadFromResource("/pipeline.xml"); RequestStatus createStatus = api().create(null, PIPELINE_JOB_NAME, config); assertTrue(createStatus.value()); - IntegerResponse qId = api().build(null, PIPELINE_JOB_NAME); + LongResponse qId = api().build(null, PIPELINE_JOB_NAME); assertNotNull(qId); assertTrue(qId.value() > 0); QueueItem queueItem = getRunningQueueItem(qId.value()); @@ -141,7 +141,7 @@ public void testStopPipelineBuild() throws InterruptedException { @Test(dependsOnMethods = "testStopPipelineBuild") public void testTermPipelineBuild() throws InterruptedException { - IntegerResponse qId = api().build(null, PIPELINE_JOB_NAME); + LongResponse qId = api().build(null, PIPELINE_JOB_NAME); assertNotNull(qId); assertTrue(qId.value() > 0); QueueItem queueItem = getRunningQueueItem(qId.value()); @@ -156,7 +156,7 @@ public void testTermPipelineBuild() throws InterruptedException { @Test(dependsOnMethods = "testTermPipelineBuild") public void testKillPipelineBuild() throws InterruptedException { - IntegerResponse qId = api().build(null, PIPELINE_JOB_NAME); + LongResponse qId = api().build(null, PIPELINE_JOB_NAME); assertNotNull(qId); assertTrue(qId.value() > 0); QueueItem queueItem = getRunningQueueItem(qId.value()); @@ -241,7 +241,7 @@ public void testGetBuildInfo() { BuildInfo output = api().buildInfo(null, "DevTest", buildNumber); assertNotNull(output); assertEquals("DevTest #" + buildNumber, output.fullDisplayName()); - assertEquals((int) queueId.value(), output.queueId()); + assertEquals((long) queueId.value(), output.queueId()); } @Test(dependsOnMethods = "testGetBuildInfo") @@ -255,7 +255,7 @@ public void testBuildInfoActions() throws InterruptedException { String config = payloadFromResource("/pipeline-with-action.xml"); RequestStatus createStatus = api().create(null, PIPELINE_WITH_ACTION_JOB_NAME, config); assertTrue(createStatus.value()); - IntegerResponse qId = api().build(null, PIPELINE_WITH_ACTION_JOB_NAME); + LongResponse qId = api().build(null, PIPELINE_WITH_ACTION_JOB_NAME); assertNotNull(qId); assertTrue(qId.value() > 0); QueueItem queueItem = getRunningQueueItem(qId.value()); @@ -319,7 +319,7 @@ public void testUpdateConfig() { public void testBuildJobWithParameters() { Map> params = new HashMap<>(); params.put("SomeKey", Lists.newArrayList("SomeVeryNewValue")); - IntegerResponse output = api().buildWithParameters(null, "DevTest", params); + LongResponse output = api().buildWithParameters(null, "DevTest", params); assertNotNull(output); assertTrue(output.value() > 0); assertEquals(output.errors().size(), 0); @@ -327,7 +327,7 @@ public void testBuildJobWithParameters() { @Test(dependsOnMethods = "testBuildJobWithParameters") public void testBuildJobWithNullParametersMap() { - IntegerResponse output = api().buildWithParameters(null, "DevTest", null); + LongResponse output = api().buildWithParameters(null, "DevTest", null); assertNotNull(output); assertTrue(output.value() > 0); assertEquals(output.errors().size(), 0); @@ -335,7 +335,7 @@ public void testBuildJobWithNullParametersMap() { @Test(dependsOnMethods = "testBuildJobWithNullParametersMap") public void testBuildJobWithEmptyParametersMap() { - IntegerResponse output = api().buildWithParameters(null, "DevTest", new HashMap<>()); + LongResponse output = api().buildWithParameters(null, "DevTest", new HashMap<>()); assertNotNull(output); assertNull(output.value()); assertEquals(output.errors().size(), 1); @@ -508,7 +508,7 @@ public void testGetBuildInfoOfJobInFolder() { BuildInfo output = api().buildInfo("test-folder/test-folder-1", "JobInFolder", 1); assertNotNull(output); assertTrue(output.fullDisplayName().contains("JobInFolder #1")); - assertEquals((int) queueIdForAnotherJob.value(), output.queueId()); + assertEquals((long) queueIdForAnotherJob.value(), output.queueId()); } @Test(dependsOnMethods = "testGetProgressiveText") @@ -549,7 +549,7 @@ public void testBuildWithParametersOfJobForEmptyAndNullParams() throws Interrupt Map> params = new HashMap<>(); params.put("SomeKey1", Lists.newArrayList("")); params.put("SomeKey2", null); - IntegerResponse job1 = api.jobsApi().buildWithParameters(null, "JobForEmptyAndNullParams", params); + LongResponse job1 = api.jobsApi().buildWithParameters(null, "JobForEmptyAndNullParams", params); assertNotNull(job1); assertTrue(job1.value() > 0); assertEquals(job1.errors().size(), 0); @@ -639,7 +639,7 @@ public void testGetDescriptionNonExistentJob() { @Test public void testBuildNonExistentJob() { - IntegerResponse output = api().build(null, randomString()); + LongResponse output = api().build(null, randomString()); assertNotNull(output); assertNull(output.value()); assertTrue(output.errors().size() > 0); @@ -658,7 +658,7 @@ public void testGetBuildInfoNonExistentJob() { public void testBuildNonExistentJobWithParams() { Map> params = new HashMap<>(); params.put("SomeKey", Lists.newArrayList("SomeVeryNewValue")); - IntegerResponse output = api().buildWithParameters(null, randomString(), params); + LongResponse output = api().buildWithParameters(null, randomString(), params); assertNotNull(output); assertNull(output.value()); assertTrue(output.errors().size() > 0); diff --git a/src/test/java/com/cdancy/jenkins/rest/features/JobsApiMockTest.java b/src/test/java/com/cdancy/jenkins/rest/features/JobsApiMockTest.java index a078884..1a66475 100644 --- a/src/test/java/com/cdancy/jenkins/rest/features/JobsApiMockTest.java +++ b/src/test/java/com/cdancy/jenkins/rest/features/JobsApiMockTest.java @@ -18,7 +18,7 @@ import com.cdancy.jenkins.rest.BaseJenkinsMockTest; import com.cdancy.jenkins.rest.JenkinsApi; -import com.cdancy.jenkins.rest.domain.common.IntegerResponse; +import com.cdancy.jenkins.rest.domain.common.LongResponse; import com.cdancy.jenkins.rest.domain.common.RequestStatus; import com.cdancy.jenkins.rest.domain.job.*; import com.google.common.collect.Lists; @@ -477,9 +477,9 @@ public void testBuildJob() throws Exception { JenkinsApi jenkinsApi = api(server.url("/").url()); JobsApi api = jenkinsApi.jobsApi(); try { - IntegerResponse output = api.build(null,"DevTest"); + LongResponse output = api.build(null,"DevTest"); assertNotNull(output); - assertEquals((int) output.value(), 1); + assertEquals((long) output.value(), 1); assertEquals(output.errors().size(), 0); assertSentAccept(server, "POST", "/job/DevTest/build", "application/unknown"); } finally { @@ -496,7 +496,7 @@ public void testBuildJobWithNoLocationReturned() throws Exception { JenkinsApi jenkinsApi = api(server.url("/").url()); JobsApi api = jenkinsApi.jobsApi(); try { - IntegerResponse output = api.build(null,"DevTest"); + LongResponse output = api.build(null,"DevTest"); assertNotNull(output); assertNull(output.value()); assertEquals(output.errors().size(), 1); @@ -517,7 +517,7 @@ public void testBuildJobNonExistentJob() throws Exception { JenkinsApi jenkinsApi = api(server.url("/").url()); JobsApi api = jenkinsApi.jobsApi(); try { - IntegerResponse output = api.build(null, "DevTest"); + LongResponse output = api.build(null, "DevTest"); assertNotNull(output); assertNull(output.value()); assertEquals(output.errors().size(), 1); @@ -541,9 +541,9 @@ public void testBuildJobWithParams() throws Exception { try { Map> params = new HashMap<>(); params.put("SomeKey", Lists.newArrayList("SomeVeryNewValue")); - IntegerResponse output = api.buildWithParameters(null, "DevTest", params); + LongResponse output = api.buildWithParameters(null, "DevTest", params); assertNotNull(output); - assertEquals((int) output.value(), 1); + assertEquals((long) output.value(), 1); assertEquals(output.errors().size(), 0); assertSentAccept(server, "POST", "/job/DevTest/buildWithParameters", "application/unknown"); } finally { @@ -560,9 +560,9 @@ public void testBuildJobWithNullParamsMap() throws Exception { JenkinsApi jenkinsApi = api(server.url("/").url()); JobsApi api = jenkinsApi.jobsApi(); try { - IntegerResponse output = api.buildWithParameters(null, "DevTest", null); + LongResponse output = api.buildWithParameters(null, "DevTest", null); assertNotNull(output); - assertEquals((int) output.value(), 1); + assertEquals((long) output.value(), 1); assertEquals(output.errors().size(), 0); assertSentAccept(server, "POST", "/job/DevTest/buildWithParameters", "application/unknown"); } finally { @@ -579,9 +579,9 @@ public void testBuildJobWithEmptyParamsMap() throws Exception { JenkinsApi jenkinsApi = api(server.url("/").url()); JobsApi api = jenkinsApi.jobsApi(); try { - IntegerResponse output = api.buildWithParameters(null, "DevTest", new HashMap<>()); + LongResponse output = api.buildWithParameters(null, "DevTest", new HashMap<>()); assertNotNull(output); - assertEquals((int) output.value(), 1); + assertEquals((long) output.value(), 1); assertEquals(output.errors().size(), 0); assertSentAccept(server, "POST", "/job/DevTest/buildWithParameters", "application/unknown"); } finally { @@ -599,7 +599,7 @@ public void testBuildJobWithParamsNonExistentJob() throws Exception { try { Map> params = new HashMap<>(); params.put("SomeKey", Lists.newArrayList("SomeVeryNewValue")); - IntegerResponse output = api.buildWithParameters(null, "DevTest", params); + LongResponse output = api.buildWithParameters(null, "DevTest", params); assertNotNull(output); assertNull(output.value()); assertEquals(output.errors().size(), 1); diff --git a/src/test/java/com/cdancy/jenkins/rest/features/QueueApiLiveTest.java b/src/test/java/com/cdancy/jenkins/rest/features/QueueApiLiveTest.java index 0d6ce49..9ec9c93 100644 --- a/src/test/java/com/cdancy/jenkins/rest/features/QueueApiLiveTest.java +++ b/src/test/java/com/cdancy/jenkins/rest/features/QueueApiLiveTest.java @@ -31,7 +31,7 @@ import org.testng.annotations.Test; import com.cdancy.jenkins.rest.BaseJenkinsApiLiveTest; -import com.cdancy.jenkins.rest.domain.common.IntegerResponse; +import com.cdancy.jenkins.rest.domain.common.LongResponse; import com.cdancy.jenkins.rest.domain.common.RequestStatus; import com.cdancy.jenkins.rest.domain.queue.QueueItem; @@ -58,10 +58,10 @@ public void init() { @Test public void testGetQueue() { - IntegerResponse job1 = api.jobsApi().build(null, "QueueTest"); + LongResponse job1 = api.jobsApi().build(null, "QueueTest"); assertNotNull(job1); assertEquals(job1.errors().size(), 0); - IntegerResponse job2 = api.jobsApi().build(null, "QueueTest"); + LongResponse job2 = api.jobsApi().build(null, "QueueTest"); assertNotNull(job2); assertEquals(job2.errors().size(), 0); List queueItems = api().queue(); @@ -78,10 +78,10 @@ public void testGetQueue() { @Test public void testGetPendingQueueItem() { - IntegerResponse job1 = api.jobsApi().build(null,"QueueTest"); + LongResponse job1 = api.jobsApi().build(null,"QueueTest"); assertNotNull(job1); assertEquals(job1.errors().size(), 0); - IntegerResponse job2 = api.jobsApi().build(null,"QueueTest"); + LongResponse job2 = api.jobsApi().build(null,"QueueTest"); assertNotNull(job2); assertEquals(job2.errors().size(), 0); @@ -94,10 +94,10 @@ public void testGetPendingQueueItem() { @Test public void testGetRunningQueueItem() throws InterruptedException { - IntegerResponse job1 = api.jobsApi().build(null,"QueueTest"); + LongResponse job1 = api.jobsApi().build(null,"QueueTest"); assertNotNull(job1); assertEquals(job1.errors().size(), 0); - IntegerResponse job2 = api.jobsApi().build(null,"QueueTest"); + LongResponse job2 = api.jobsApi().build(null,"QueueTest"); assertNotNull(job2); assertEquals(job2.errors().size(), 0); @@ -121,7 +121,7 @@ public void testGetRunningQueueItem() throws InterruptedException { public void testQueueItemSingleParameters() throws InterruptedException { Map> params = new HashMap<>(); params.put("SomeKey", Lists.newArrayList("SomeVeryNewValue1")); - IntegerResponse job1 = api.jobsApi().buildWithParameters(null,"QueueTestSingleParam", params); + LongResponse job1 = api.jobsApi().buildWithParameters(null,"QueueTestSingleParam", params); assertNotNull(job1); assertTrue(job1.value() > 0); assertEquals(job1.errors().size(), 0); @@ -130,7 +130,7 @@ public void testQueueItemSingleParameters() throws InterruptedException { // So we must set some different parameter values params = new HashMap<>(); params.put("SomeKey", Lists.newArrayList("SomeVeryNewValue2")); - IntegerResponse job2 = api.jobsApi().buildWithParameters(null,"QueueTestSingleParam", params); + LongResponse job2 = api.jobsApi().buildWithParameters(null,"QueueTestSingleParam", params); assertNotNull(job2); assertTrue(job2.value() > 0); assertEquals(job2.errors().size(), 0); @@ -148,7 +148,7 @@ public void testQueueItemSingleParameters() throws InterruptedException { public void testQueueItemMultipleParameters() throws InterruptedException { Map> params = new HashMap<>(); params.put("SomeKey1", Lists.newArrayList("SomeVeryNewValue1")); - IntegerResponse job1 = api.jobsApi().buildWithParameters(null, "QueueTestMultipleParams",params); + LongResponse job1 = api.jobsApi().buildWithParameters(null, "QueueTestMultipleParams",params); assertNotNull(job1); assertTrue(job1.value() > 0); assertEquals(job1.errors().size(), 0); @@ -157,7 +157,7 @@ public void testQueueItemMultipleParameters() throws InterruptedException { // So we must set some different parameter values params = new HashMap<>(); params.put("SomeKey1", Lists.newArrayList("SomeVeryNewValue2")); - IntegerResponse job2 = api.jobsApi().buildWithParameters(null, "QueueTestMultipleParams", params); + LongResponse job2 = api.jobsApi().buildWithParameters(null, "QueueTestMultipleParams", params); assertNotNull(job2); assertTrue(job2.value() > 0); assertEquals(job2.errors().size(), 0); @@ -177,7 +177,7 @@ public void testQueueItemMultipleParameters() throws InterruptedException { public void testQueueItemEmptyParameterValue() throws InterruptedException { Map> params = new HashMap<>(); params.put("SomeKey1", Lists.newArrayList("")); - IntegerResponse job1 = api.jobsApi().buildWithParameters(null, "QueueTestMultipleParams",params); + LongResponse job1 = api.jobsApi().buildWithParameters(null, "QueueTestMultipleParams",params); assertNotNull(job1); assertTrue(job1.value() > 0); assertEquals(job1.errors().size(), 0); @@ -194,10 +194,10 @@ public void testQueueItemEmptyParameterValue() throws InterruptedException { @Test public void testGetCancelledQueueItem() { - IntegerResponse job1 = api.jobsApi().build(null,"QueueTest"); + LongResponse job1 = api.jobsApi().build(null,"QueueTest"); assertNotNull(job1); assertEquals(job1.errors().size(), 0); - IntegerResponse job2 = api.jobsApi().build(null, "QueueTest"); + LongResponse job2 = api.jobsApi().build(null, "QueueTest"); assertNotNull(job2); assertEquals(job2.errors().size(), 0); From 645c64861dd3a0875e8ee939753e79d0d3d999ea Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Fri, 16 Aug 2024 10:53:03 +0000 Subject: [PATCH 10/11] Bump ch.qos.logback:logback-core from 1.5.6 to 1.5.7 Bumps [ch.qos.logback:logback-core](https://github.com/qos-ch/logback) from 1.5.6 to 1.5.7. - [Commits](https://github.com/qos-ch/logback/compare/v_1.5.6...v_1.5.7) --- updated-dependencies: - dependency-name: ch.qos.logback:logback-core dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index bb7ebb0..11f1959 100644 --- a/build.gradle +++ b/build.gradle @@ -33,7 +33,7 @@ dependencies { testImplementation ("org.apache.jclouds:jclouds-core:${jcloudsVersion}:tests") testImplementation ("org.apache.jclouds.driver:jclouds-slf4j:${jcloudsVersion}") testImplementation ('org.assertj:assertj-core:3.25.3') - testImplementation ('ch.qos.logback:logback-core:1.5.6') + testImplementation ('ch.qos.logback:logback-core:1.5.7') testImplementation ('com.squareup.okhttp3:mockwebserver:4.12.0') testImplementation ('org.testng:testng:7.10.2') testImplementation ('ch.qos.logback:logback-classic:1.5.6') From 16c62bb54e1448229e75026dbea163e53098e640 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Fri, 16 Aug 2024 10:53:06 +0000 Subject: [PATCH 11/11] Bump ch.qos.logback:logback-classic from 1.5.6 to 1.5.7 Bumps [ch.qos.logback:logback-classic](https://github.com/qos-ch/logback) from 1.5.6 to 1.5.7. - [Commits](https://github.com/qos-ch/logback/compare/v_1.5.6...v_1.5.7) --- updated-dependencies: - dependency-name: ch.qos.logback:logback-classic dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] --- build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build.gradle b/build.gradle index bb7ebb0..3cee506 100644 --- a/build.gradle +++ b/build.gradle @@ -36,7 +36,7 @@ dependencies { testImplementation ('ch.qos.logback:logback-core:1.5.6') testImplementation ('com.squareup.okhttp3:mockwebserver:4.12.0') testImplementation ('org.testng:testng:7.10.2') - testImplementation ('ch.qos.logback:logback-classic:1.5.6') + testImplementation ('ch.qos.logback:logback-classic:1.5.7') } ext.compatibilityVersion = JavaVersion.VERSION_11