Skip to content

Commit 7226322

Browse files
committed
Fix warnings.
1 parent 9e314d4 commit 7226322

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

src/test/java/io/jenkins/plugins/util/PluginArchitectureRules.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -110,13 +110,13 @@ public final class PluginArchitectureRules {
110110
.and().haveRawReturnType(ofAllowedClasses(ComboBoxModel.class, ListBoxModel.class))
111111
.should().beAnnotatedWith(POST.class)
112112
.andShould().bePublic()
113-
.andShould(havePermissionCHeck());
113+
.andShould(checkPermissions());
114114

115115
private static FormValidationSignaturePredicate ofAllowedValidationMethodSignatures() {
116116
return new FormValidationSignaturePredicate();
117117
}
118118

119-
private static HavePermissionCheck havePermissionCHeck() {
119+
private static HavePermissionCheck checkPermissions() {
120120
return new HavePermissionCheck();
121121
}
122122

@@ -139,7 +139,7 @@ public void check(final JavaMethod item, final ConditionEvents events) {
139139

140140
if (callsFromSelf.stream().anyMatch(
141141
javaCall -> javaCall.getTarget().getOwner().getFullName().equals(JenkinsFacade.class.getName())
142-
&& javaCall.getTarget().getName().equals("hasPermission"))) {
142+
&& "hasPermission".equals(javaCall.getTarget().getName()))) {
143143
return;
144144
}
145145
events.add(SimpleConditionEvent.violated(item,
@@ -149,17 +149,17 @@ public void check(final JavaMethod item, final ConditionEvents events) {
149149
}
150150

151151
private static class AllowedClasses extends DescribedPredicate<JavaClass> {
152-
private final List<String> allowedClasses;
152+
private final List<String> allowedClassNames;
153153

154154
AllowedClasses(final Class<?>... classes) {
155155
super("raw return type of any of %s", Arrays.toString(classes));
156156

157-
allowedClasses = Arrays.stream(classes).map(Class::getName).collect(Collectors.toList());
157+
allowedClassNames = Arrays.stream(classes).map(Class::getName).collect(Collectors.toList());
158158
}
159159

160160
@Override
161161
public boolean apply(final JavaClass input) {
162-
return allowedClasses.contains(input.getFullName());
162+
return allowedClassNames.contains(input.getFullName());
163163
}
164164
}
165165

0 commit comments

Comments
 (0)