diff --git a/spring-cloud-contract-stub-runner/spring-cloud-contract-stub-runner/src/main/java/org/springframework/cloud/contract/stubrunner/AetherStubDownloader.java b/spring-cloud-contract-stub-runner/spring-cloud-contract-stub-runner/src/main/java/org/springframework/cloud/contract/stubrunner/AetherStubDownloader.java index 338fc2e7c3..bf771d1211 100644 --- a/spring-cloud-contract-stub-runner/spring-cloud-contract-stub-runner/src/main/java/org/springframework/cloud/contract/stubrunner/AetherStubDownloader.java +++ b/spring-cloud-contract-stub-runner/spring-cloud-contract-stub-runner/src/main/java/org/springframework/cloud/contract/stubrunner/AetherStubDownloader.java @@ -16,12 +16,6 @@ package org.springframework.cloud.contract.stubrunner; -import static java.nio.file.Files.createTempDirectory; -import static org.springframework.cloud.contract.stubrunner.AetherFactories.newRepositories; -import static org.springframework.cloud.contract.stubrunner.AetherFactories.newRepositorySystem; -import static org.springframework.cloud.contract.stubrunner.AetherFactories.newSession; -import static org.springframework.cloud.contract.stubrunner.util.ZipCategory.unzipTo; - import java.io.File; import java.io.IOException; import java.net.URI; @@ -47,6 +41,12 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; import org.springframework.util.StringUtils; +import static java.nio.file.Files.createTempDirectory; +import static org.springframework.cloud.contract.stubrunner.AetherFactories.newRepositories; +import static org.springframework.cloud.contract.stubrunner.AetherFactories.newRepositorySystem; +import static org.springframework.cloud.contract.stubrunner.AetherFactories.newSession; +import static org.springframework.cloud.contract.stubrunner.util.ZipCategory.unzipTo; + /** * @author Mariusz Smykula */ @@ -78,7 +78,6 @@ public class AetherStubDownloader implements StubDownloader { * @param repositorySystem * @param remoteRepositories - remote artifact repositories * @param session - * @param workOffline */ public AetherStubDownloader(RepositorySystem repositorySystem, List remoteRepositories, RepositorySystemSession session) { @@ -98,7 +97,7 @@ public class AetherStubDownloader implements StubDownloader { private File unpackedJar(String resolvedVersion, String stubsGroup, String stubsModule, String classifier) { - log.info("Resolved version is" + resolvedVersion); + log.info("Resolved version is [" + resolvedVersion + "]"); if (!StringUtils.hasText(resolvedVersion)) { log.warn("Stub for group [" + stubsGroup + "] module [" + stubsModule + "] and classifier [" + classifier + "] not found in " @@ -108,11 +107,11 @@ public class AetherStubDownloader implements StubDownloader { Artifact artifact = new DefaultArtifact(stubsGroup, stubsModule, classifier, ARTIFACT_EXTENSION, resolvedVersion); ArtifactRequest request = new ArtifactRequest(artifact, remoteRepos, null); - log.info("Resolving artifact " + artifact - + " using remote repositories " + remoteRepos); + log.info("Resolving artifact [" + artifact + + "] using remote repositories " + remoteRepos); try { ArtifactResult result = repositorySystem.resolveArtifact(session, request); - log.info("Resolved artifact " + artifact + "to " + log.info("Resolved artifact [" + artifact + "] to " + result.getArtifact().getFile()); File temporaryFile = unpackStubJarToATemporaryFolder( result.getArtifact().getFile().toURI()); @@ -121,7 +120,7 @@ public class AetherStubDownloader implements StubDownloader { } catch (Exception e) { log.warn( - "Exception occured while trying to download a stub for group [" + "Exception occurred while trying to download a stub for group [" + stubsGroup + "] module [" + stubsModule + "] and classifier [" + classifier + "] in " + remoteRepos, e); @@ -133,8 +132,8 @@ public class AetherStubDownloader implements StubDownloader { private String getVersion(String stubsGroup, String stubsModule, String version, String classifier) { if (!StringUtils.hasText(version) || LATEST_VERSION_IN_IVY.equals(version)) { - log.info("Desired version is " + version - + " - will try to resolve the latest version"); + log.info("Desired version is [" + version + + "] - will try to resolve the latest version"); return resolveHighestArtifactVersion(stubsGroup, stubsModule, classifier); } log.info("Will try to resolve version " + version); diff --git a/spring-cloud-contract-stub-runner/spring-cloud-contract-stub-runner/src/main/java/org/springframework/cloud/contract/stubrunner/StubServer.java b/spring-cloud-contract-stub-runner/spring-cloud-contract-stub-runner/src/main/java/org/springframework/cloud/contract/stubrunner/StubServer.java index 4adb289f80..29a2fe19ca 100644 --- a/spring-cloud-contract-stub-runner/spring-cloud-contract-stub-runner/src/main/java/org/springframework/cloud/contract/stubrunner/StubServer.java +++ b/spring-cloud-contract-stub-runner/spring-cloud-contract-stub-runner/src/main/java/org/springframework/cloud/contract/stubrunner/StubServer.java @@ -56,7 +56,8 @@ public class StubServer { public StubServer start() { wireMockServer.start(); - log.info("Started stub server for project ${stubConfiguration.toColonSeparatedDependencyNotation()} on port ${wireMockServer.port()}"); + log.info("Started stub server for project [" + stubConfiguration.toColonSeparatedDependencyNotation() + + "] on port " + wireMockServer.port()); registerStubMappings(); return this; } @@ -114,7 +115,7 @@ public class StubServer { for (WiremockMappingDescriptor mappingDescriptor : sortedMappings) { try { wireMock.register(mappingDescriptor.getMapping()); - log.debug("Registered stub mappings from $mappingDescriptor.descriptor"); + log.debug("Registered stub mappings from [" + mappingDescriptor.descriptor + "]"); } catch (Exception e) { log.warn("Failed to register the stub mapping ["+ mappingDescriptor + "]", e); }