Skip to content

Commit 5718d2f

Browse files
committed
Make snapshot unit-test less failure-prone
1 parent 9c0bd6c commit 5718d2f

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

src/test/java/io/jenkins/plugins/credentials/secretsmanager/factory/BaseAwsJsonCredentialsSnapshotTakerTest.java

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -111,10 +111,15 @@ public void snapshotGivenInstanceThenCreatesSnapshot() throws Exception {
111111
final Method[] allMethods = credentialBeingSnapshotted.getDeclaredMethods();
112112
final Map<Method, Object> expectedResults = new IdentityHashMap<>();
113113
for (final Method m : allMethods) {
114-
if (m.getParameterCount() != 0) {
114+
if (m.getParameterCount() != 0 || !m.isAccessible()) {
115+
continue;
116+
}
117+
final Object expectedValue;
118+
try {
119+
expectedValue = m.invoke(credential);
120+
} catch (IllegalAccessException e) {
115121
continue;
116122
}
117-
final Object expectedValue = m.invoke(credential);
118123
expectedResults.put(m, expectedValue);
119124
}
120125
final String expectedId = credential.getId();

0 commit comments

Comments
 (0)