Merge pull request #20911 from dreis2211

* pr/20911:
  Fix unused parameters

Closes gh-20911
pull/20913/head
Stephane Nicoll 5 years ago
commit b6076f5b10

@ -57,6 +57,7 @@ class DependencyCustomizerTests {
MockitoAnnotations.initMocks(this); MockitoAnnotations.initMocks(this);
given(this.resolver.getGroupId("spring-boot-starter-logging")).willReturn("org.springframework.boot"); given(this.resolver.getGroupId("spring-boot-starter-logging")).willReturn("org.springframework.boot");
given(this.resolver.getArtifactId("spring-boot-starter-logging")).willReturn("spring-boot-starter-logging"); given(this.resolver.getArtifactId("spring-boot-starter-logging")).willReturn("spring-boot-starter-logging");
given(this.resolver.getVersion("spring-boot-starter-logging")).willReturn("1.2.3");
this.moduleNode.addClass(this.classNode); this.moduleNode.addClass(this.classNode);
this.dependencyCustomizer = new DependencyCustomizer(new GroovyClassLoader(getClass().getClassLoader()), this.dependencyCustomizer = new DependencyCustomizer(new GroovyClassLoader(getClass().getClassLoader()),
this.moduleNode, new DependencyResolutionContext() { this.moduleNode, new DependencyResolutionContext() {
@ -184,6 +185,7 @@ class DependencyCustomizerTests {
String classifier, String type, boolean transitive) { String classifier, String type, boolean transitive) {
assertThat(getMemberValue(annotationNode, "group")).isEqualTo(group); assertThat(getMemberValue(annotationNode, "group")).isEqualTo(group);
assertThat(getMemberValue(annotationNode, "module")).isEqualTo(module); assertThat(getMemberValue(annotationNode, "module")).isEqualTo(module);
assertThat(getMemberValue(annotationNode, "version")).isEqualTo(version);
if (type == null) { if (type == null) {
assertThat(annotationNode.getMember("type")).isNull(); assertThat(annotationNode.getMember("type")).isNull();
} }

@ -324,7 +324,7 @@ public abstract class Packager {
addBootBootAttributesForRepackagingLayout(attributes, (RepackagingLayout) layout); addBootBootAttributesForRepackagingLayout(attributes, (RepackagingLayout) layout);
} }
else { else {
addBootBootAttributesForPlainLayout(attributes, layout); addBootBootAttributesForPlainLayout(attributes);
} }
} }
@ -337,7 +337,7 @@ public abstract class Packager {
} }
} }
private void addBootBootAttributesForPlainLayout(Attributes attributes, Layout layout) { private void addBootBootAttributesForPlainLayout(Attributes attributes) {
attributes.putValue(BOOT_CLASSES_ATTRIBUTE, getLayout().getClassesLocation()); attributes.putValue(BOOT_CLASSES_ATTRIBUTE, getLayout().getClassesLocation());
putIfHasLength(attributes, BOOT_LIB_ATTRIBUTE, getLayout().getLibraryLocation("", LibraryScope.COMPILE)); putIfHasLength(attributes, BOOT_LIB_ATTRIBUTE, getLayout().getLibraryLocation("", LibraryScope.COMPILE));
} }

@ -103,7 +103,7 @@ public class UndertowReactiveWebServerFactory extends AbstractReactiveWebServerF
handler = UndertowCompressionConfigurer.configureCompression(getCompression(), handler); handler = UndertowCompressionConfigurer.configureCompression(getCompression(), handler);
Closeable closeable = null; Closeable closeable = null;
if (isAccessLogEnabled()) { if (isAccessLogEnabled()) {
AccessLogHandlerConfiguration accessLogHandlerConfiguration = configureAccessLogHandler(builder, handler); AccessLogHandlerConfiguration accessLogHandlerConfiguration = configureAccessLogHandler(handler);
closeable = accessLogHandlerConfiguration.closeable; closeable = accessLogHandlerConfiguration.closeable;
handler = accessLogHandlerConfiguration.accessLogHandler; handler = accessLogHandlerConfiguration.accessLogHandler;
} }
@ -147,7 +147,7 @@ public class UndertowReactiveWebServerFactory extends AbstractReactiveWebServerF
return builder; return builder;
} }
private AccessLogHandlerConfiguration configureAccessLogHandler(Undertow.Builder builder, HttpHandler handler) { private AccessLogHandlerConfiguration configureAccessLogHandler(HttpHandler handler) {
try { try {
createAccessLogDirectoryIfNecessary(); createAccessLogDirectoryIfNecessary();
XnioWorker worker = createWorker(); XnioWorker worker = createWorker();

Loading…
Cancel
Save