Fix test and running of jmockit

jenkinsci · Mar 9, 2022 · 3843dd5 · 3843dd5
1 parent 2ccf16f
commit 3843dd5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 11 deletions.
diff --git a/pom.xml b/pom.xml
@@ -158,6 +158,11 @@
                         </goals>
                     </execution>
                 </executions>
+                <configuration>
+                    <argLine>
+                        -javaagent:"${settings.localRepository}"/org/jmockit/jmockit/1.41/jmockit-1.41.jar
+                    </argLine>
+                </configuration>
             </plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
@@ -182,16 +187,6 @@
                     <suppressionsLocation>${project.basedir}/checkstyle-suppressions.xml</suppressionsLocation>
                 </configuration>
             </plugin>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-surefire-plugin</artifactId>
-                <version>2.22.2</version>
-                <configuration>
-                    <argLine>
-                        -javaagent:"${settings.localRepository}"/org/jmockit/jmockit/1.41/jmockit-1.41.jar
-                    </argLine>
-                </configuration>
-            </plugin>
         </plugins>
     </build>
 

diff --git a/src/test/java/io/jenkins/plugins/credentials/secretsmanager/GithubAppCredentialsIT.java b/src/test/java/io/jenkins/plugins/credentials/secretsmanager/GithubAppCredentialsIT.java
@@ -57,7 +57,7 @@ public void shouldSupportListView() {
 
         // Then
         assertThat(list)
-                .containsOption(APP_ID + "/******", foo.getName());
+                .containsOption(foo.getName(), foo.getName());
     }
 
     @Test