Update dependency org.jvnet.hudson.plugins:analysis-pom to v7.3.0 (#245)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Ulli Hafner <[email protected]>
jenkinsci · Feb 27, 2024 · 60efc73 · 60efc73
1 parent 7140c3c
commit 60efc73
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 12 deletions.
diff --git a/pom.xml b/pom.xml
@@ -5,7 +5,7 @@
   <parent>
     <groupId>org.jvnet.hudson.plugins</groupId>
     <artifactId>analysis-pom</artifactId>
-    <version>7.1.0</version>
+    <version>7.3.0</version>
     <relativePath />
   </parent>
 
@@ -18,12 +18,11 @@
   <description>Defines an API for Jenkins to publish checks to SCM platforms.</description>
 
   <properties>
-    <revision>2.0.3</revision>
+    <revision>2.1.0</revision>
     <changelist>-SNAPSHOT</changelist>
     <gitHubRepo>jenkinsci/${project.artifactId}-plugin</gitHubRepo>
 
-    <testcontainers.version>1.19.0</testcontainers.version>
-    <plugin-util-api.version>4.1.0</plugin-util-api.version>
+    <testcontainers.version>1.19.6</testcontainers.version>
   </properties>
 
   <licenses>
@@ -45,7 +44,6 @@
     <dependency>
       <groupId>io.jenkins.plugins</groupId>
       <artifactId>plugin-util-api</artifactId>
-      <version>${plugin-util-api.version}</version>
     </dependency>
     <dependency>
         <groupId>io.jenkins.plugins</groupId>
@@ -58,7 +56,6 @@
     <dependency>
       <groupId>io.jenkins.plugins</groupId>
       <artifactId>plugin-util-api</artifactId>
-      <version>${plugin-util-api.version}</version>
       <classifier>tests</classifier>
       <scope>test</scope>
     </dependency>

diff --git a/src/test/java/io/jenkins/plugins/checks/status/BuildStatusChecksPublisherITest.java b/src/test/java/io/jenkins/plugins/checks/status/BuildStatusChecksPublisherITest.java
@@ -156,15 +156,15 @@ public void shouldPublishStageDetails() {
         ChecksDetails details = checksDetails.get(0);
         assertThat(details.getStatus()).isEqualTo(ChecksStatus.QUEUED);
         assertThat(details.getConclusion()).isEqualTo(ChecksConclusion.NONE);
-        assertThat(details.getName()).isPresent().get().isEqualTo("Test Status");
+        assertThat(details.getName()).isPresent().contains("Test Status");
         assertThat(details.getOutput()).isNotPresent();
 
         // Details 1, first stage
         details = checksDetails.get(1);
         assertThat(details.getStatus()).isEqualTo(ChecksStatus.IN_PROGRESS);
         assertThat(details.getConclusion()).isEqualTo(ChecksConclusion.NONE);
         assertThat(details.getOutput()).isPresent().get().satisfies(output -> {
-            assertThat(output.getTitle()).isPresent().get().isEqualTo("In progress");
+            assertThat(output.getTitle()).isPresent().contains("In progress");
             assertThat(output.getSummary()).isPresent().get().satisfies(StringUtils::isBlank);
             assertThat(output.getText()).isPresent().get().asString().contains("* Simple Stage *(running)*");
         });
@@ -174,16 +174,16 @@ public void shouldPublishStageDetails() {
         assertThat(details.getOutput()).isPresent().get().satisfies(output -> {
             assertThat(output.getSummary()).isPresent().get().satisfies(StringUtils::isBlank);
             assertThat(output.getText()).isPresent().get().satisfies(text -> {
-                assertThat(output.getTitle()).isPresent().get().isEqualTo("In progress");
-                assertThat(text).matches(Pattern.compile(".*\\* Simple Stage \\*\\([^)]+\\)\\*.*", Pattern.DOTALL));
-                assertThat(text).contains("  * In parallel *(running)*");
+                assertThat(output.getTitle()).isPresent().contains("In progress");
+                assertThat(text).doesNotContain("* Simple Stage *(running)*");
+                assertThat(text).contains("* Simple Stage", "* In parallel *(running)*");
             });
         });
 
         // Details 6, p1s1 has finished and emitted unstable
         details = checksDetails.get(6);
         assertThat(details.getOutput()).isPresent().get().satisfies(output -> {
-            assertThat(output.getTitle()).isPresent().get().isEqualTo("In parallel/p1/p1s1: warning in 'unstable' step");
+            assertThat(output.getTitle()).isPresent().contains("In parallel/p1/p1s1: warning in 'unstable' step");
             assertThat(output.getSummary()).isPresent().get().asString().isEqualToIgnoringNewLines(""
                     + "### `In parallel / p1 / p1s1 / Set stage result to unstable`\n"
                     + "Warning in `unstable` step, with arguments `something went wrong`.\n"