Polish
This commit is contained in:
@@ -292,7 +292,7 @@ class WireMockProperties {
|
||||
}
|
||||
|
||||
public boolean isPortDynamic() {
|
||||
return portDynamic;
|
||||
return this.portDynamic;
|
||||
}
|
||||
|
||||
public void setPortDynamic(boolean portDynamic) {
|
||||
@@ -300,7 +300,7 @@ class WireMockProperties {
|
||||
}
|
||||
|
||||
public boolean isHttpsPortDynamic() {
|
||||
return httpsPortDynamic;
|
||||
return this.httpsPortDynamic;
|
||||
}
|
||||
|
||||
public void setHttpsPortDynamic(boolean httpsPortDynamic) {
|
||||
|
||||
@@ -101,6 +101,13 @@ public final class WireMockTestExecutionListener extends AbstractTestExecutionLi
|
||||
boolean httpPortDynamic = wireMockProperties.wireMock.getServer().isPortDynamic();
|
||||
boolean httpsPortDynamic = wireMockProperties.wireMock.getServer()
|
||||
.isHttpsPortDynamic();
|
||||
if (log.isDebugEnabled()) {
|
||||
int httpPort = wireMockProperties.wireMock.getServer().getPort();
|
||||
int httpsPort = wireMockProperties.wireMock.getServer().getPort();
|
||||
log.debug("Http port [" + httpPort + "] dynamic [" + httpPortDynamic + "]"
|
||||
+ " https port [" + httpsPort + "] dynamic [" + httpsPortDynamic
|
||||
+ "]");
|
||||
}
|
||||
return !httpPortDynamic || !httpsPortDynamic;
|
||||
}
|
||||
|
||||
|
||||
@@ -50,8 +50,8 @@ public class AutoConfigureWireMockApplicationTests {
|
||||
|
||||
@Test
|
||||
public void portsAreFixed() {
|
||||
boolean httpPortDynamic = wireMockProperties.getServer().isPortDynamic();
|
||||
boolean httpsPortDynamic = wireMockProperties.getServer().isHttpsPortDynamic();
|
||||
boolean httpPortDynamic = this.wireMockProperties.getServer().isPortDynamic();
|
||||
boolean httpsPortDynamic = this.wireMockProperties.getServer().isHttpsPortDynamic();
|
||||
assertThat(!httpPortDynamic || !httpsPortDynamic).isTrue();
|
||||
}
|
||||
|
||||
|
||||
@@ -45,8 +45,8 @@ public class AutoConfigureWireMockAutoStubsApplicationTests {
|
||||
|
||||
@Test
|
||||
public void portsAreNotFixed() {
|
||||
boolean httpPortDynamic = wireMockProperties.getServer().isPortDynamic();
|
||||
boolean httpsPortDynamic = wireMockProperties.getServer().isHttpsPortDynamic();
|
||||
boolean httpPortDynamic = this.wireMockProperties.getServer().isPortDynamic();
|
||||
boolean httpsPortDynamic = this.wireMockProperties.getServer().isHttpsPortDynamic();
|
||||
assertThat(!httpPortDynamic || !httpsPortDynamic).isFalse();
|
||||
}
|
||||
|
||||
|
||||
@@ -39,4 +39,5 @@ public class AutoConfigureWireMockFilesApplicationTests {
|
||||
assertThat(this.service.go())
|
||||
.isEqualToIgnoringWhitespace("{\"message\":\"Hello Root\"}");
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
@@ -50,8 +50,8 @@ public class AutoConfigureWireMockHttpsPortApplicationTests {
|
||||
|
||||
@Test
|
||||
public void portsAreFixed() {
|
||||
boolean httpPortDynamic = wireMockProperties.getServer().isPortDynamic();
|
||||
boolean httpsPortDynamic = wireMockProperties.getServer().isHttpsPortDynamic();
|
||||
boolean httpPortDynamic = this.wireMockProperties.getServer().isPortDynamic();
|
||||
boolean httpsPortDynamic = this.wireMockProperties.getServer().isHttpsPortDynamic();
|
||||
assertThat(!httpPortDynamic || !httpsPortDynamic).isTrue();
|
||||
}
|
||||
|
||||
|
||||
@@ -46,4 +46,5 @@ public class AutoConfigureWireMockRandomPortApplicationTests {
|
||||
.withHeader("Content-Type", "text/plain").withBody("Hello World!")));
|
||||
assertThat(this.service.go()).isEqualTo("Hello World!");
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
@@ -50,9 +50,10 @@ public class AutoConfigureWireMockRandomPortHttpsApplicationTests {
|
||||
|
||||
@Test
|
||||
public void portsAreNotFixed() {
|
||||
boolean httpPortDynamic = wireMockProperties.getServer().isPortDynamic();
|
||||
boolean httpsPortDynamic = wireMockProperties.getServer().isHttpsPortDynamic();
|
||||
assertThat(!httpPortDynamic || !httpsPortDynamic).isFalse();
|
||||
boolean httpPortDynamic = this.wireMockProperties.getServer().isPortDynamic();
|
||||
boolean httpsPortDynamic = this.wireMockProperties.getServer().isHttpsPortDynamic();
|
||||
assertThat(httpPortDynamic).isTrue();
|
||||
assertThat(httpsPortDynamic).isTrue();
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
@@ -45,4 +45,5 @@ public class AutoConfigureWireMockSamePortApplicationTests {
|
||||
.withHeader("Content-Type", "text/plain").withBody("Hello World2!")));
|
||||
assertThat(this.service.go2()).isEqualTo("Hello World2!");
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
@@ -40,4 +40,5 @@ public class AutoConfigureWireMockStubsAndMultipleFilesApplicationTests {
|
||||
assertThat(this.service.go())
|
||||
.isEqualToIgnoringWhitespace("{\"message\":\"Hello Root\"}");
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
@@ -38,4 +38,5 @@ public class AutoConfigureWireMockStubsApplicationWithSlashTests {
|
||||
public void contextLoads() throws Exception {
|
||||
assertThat(this.service.go()).isEqualTo("Hello World");
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user