kopia lustrzana https://github.com/JOSM/MapWithAI
Fix some issues when tests were run together
Signed-off-by: Taylor Smock <taylor.smock@kaart.com>pull/1/head
rodzic
11adb96777
commit
6f3aef98c3
|
@ -55,7 +55,6 @@ public class MapWithAITestRules extends JOSMTestRules {
|
||||||
|
|
||||||
public MapWithAITestRules() {
|
public MapWithAITestRules() {
|
||||||
super();
|
super();
|
||||||
super.assertionsInEDT();
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -113,10 +112,16 @@ public class MapWithAITestRules extends JOSMTestRules {
|
||||||
.extensions(new WireMockUrlTransformer()).dynamicPort());
|
.extensions(new WireMockUrlTransformer()).dynamicPort());
|
||||||
wireMock.start();
|
wireMock.start();
|
||||||
MapWithAIDataUtils.setPaintStyleUrl(replaceUrl(wireMock, MapWithAIDataUtils.getPaintStyleUrl()));
|
MapWithAIDataUtils.setPaintStyleUrl(replaceUrl(wireMock, MapWithAIDataUtils.getPaintStyleUrl()));
|
||||||
|
// Avoid cases where tests could write the wiremock url to some fields.
|
||||||
|
if (currentReleaseUrl == null) {
|
||||||
currentReleaseUrl = DataAvailability.getReleaseUrl();
|
currentReleaseUrl = DataAvailability.getReleaseUrl();
|
||||||
|
}
|
||||||
DataAvailability.setReleaseUrl(replaceUrl(wireMock, DataAvailability.getReleaseUrl()));
|
DataAvailability.setReleaseUrl(replaceUrl(wireMock, DataAvailability.getReleaseUrl()));
|
||||||
Config.getPref().put("osm-server.url", wireMock.baseUrl());
|
Config.getPref().put("osm-server.url", wireMock.baseUrl());
|
||||||
|
// Avoid cases where tests could write the wiremock url to some fields.
|
||||||
|
if (sourceSites == null) {
|
||||||
sourceSites = MapWithAILayerInfo.getImageryLayersSites();
|
sourceSites = MapWithAILayerInfo.getImageryLayersSites();
|
||||||
|
}
|
||||||
MapWithAILayerInfo.setImageryLayersSites(sourceSites.stream().map(t -> replaceUrl(wireMock, t))
|
MapWithAILayerInfo.setImageryLayersSites(sourceSites.stream().map(t -> replaceUrl(wireMock, t))
|
||||||
.filter(Objects::nonNull).collect(Collectors.toList()));
|
.filter(Objects::nonNull).collect(Collectors.toList()));
|
||||||
MapWithAIConflationCategory.setConflationJsonLocation(
|
MapWithAIConflationCategory.setConflationJsonLocation(
|
||||||
|
@ -158,9 +163,16 @@ public class MapWithAITestRules extends JOSMTestRules {
|
||||||
List<LoggedRequest> requests = wireMock.findUnmatchedRequests().getRequests();
|
List<LoggedRequest> requests = wireMock.findUnmatchedRequests().getRequests();
|
||||||
requests.forEach(r -> Logging.error(r.getAbsoluteUrl()));
|
requests.forEach(r -> Logging.error(r.getAbsoluteUrl()));
|
||||||
assertTrue(requests.isEmpty());
|
assertTrue(requests.isEmpty());
|
||||||
DataAvailability.setReleaseUrl(currentReleaseUrl);
|
|
||||||
Config.getPref().put("osm-server.url", null);
|
Config.getPref().put("osm-server.url", null);
|
||||||
|
// Avoid cases where tests could write the wiremock url to some fields.
|
||||||
|
if (currentReleaseUrl != null) {
|
||||||
|
DataAvailability.setReleaseUrl(currentReleaseUrl);
|
||||||
|
currentReleaseUrl = null;
|
||||||
|
}
|
||||||
|
if (sourceSites != null) {
|
||||||
MapWithAILayerInfo.setImageryLayersSites(sourceSites);
|
MapWithAILayerInfo.setImageryLayersSites(sourceSites);
|
||||||
|
sourceSites = null;
|
||||||
|
}
|
||||||
MapWithAIConflationCategory.resetConflationJsonLocation();
|
MapWithAIConflationCategory.resetConflationJsonLocation();
|
||||||
resetMapWithAILayerInfo();
|
resetMapWithAILayerInfo();
|
||||||
}
|
}
|
||||||
|
|
Ładowanie…
Reference in New Issue