Skip to content

Commit b9b45c2

Browse files
committed
Simplify Fixture Loading
Since package is always same, so we can resolve the whole path in the base class. Also put the context information in file name at the end, so that fixtures show up near the test class in the IDE. Signed-off-by: Sven Strittmatter <sven.strittmatter@iteratec.com>
1 parent 95afb10 commit b9b45c2

23 files changed

+11
-11
lines changed

src/test/java/io/securecodebox/persistence/defectdojo/service/EndpointServiceTest.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,16 @@
11
package io.securecodebox.persistence.defectdojo.service;
22

33
import io.securecodebox.persistence.defectdojo.model.Endpoint;
4-
import io.securecodebox.persistence.defectdojo.model.User;
54
import org.junit.jupiter.api.Test;
65

76
import java.io.IOException;
87
import java.net.URISyntaxException;
98

109
import static com.github.tomakehurst.wiremock.client.WireMock.*;
11-
import static org.junit.jupiter.api.Assertions.*;
1210
import static org.hamcrest.MatcherAssert.assertThat;
13-
import static org.hamcrest.Matchers.*;
11+
import static org.hamcrest.Matchers.containsInAnyOrder;
12+
import static org.hamcrest.Matchers.hasSize;
13+
import static org.junit.jupiter.api.Assertions.assertAll;
1414

1515
/**
1616
* Tests for {@link EndpointService}
@@ -24,7 +24,7 @@ void search() throws URISyntaxException, IOException {
2424
get("/api/v2/endpoints/?offset=0&limit=100")
2525
.willReturn(
2626
ok()
27-
.withBody(readResponseBodyFromFixture("io/securecodebox/persistence/defectdojo/service/fixture_EndpointService.json"))
27+
.withBody(readResponseBodyFromFixture("EndpointService_response_fixture.json"))
2828
)
2929
);
3030

src/test/java/io/securecodebox/persistence/defectdojo/service/HttpClientExampleTest.java

+1-4
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,11 @@
11
package io.securecodebox.persistence.defectdojo.service;
22

3-
import com.fasterxml.jackson.core.JsonProcessingException;
43
import com.github.tomakehurst.wiremock.junit5.WireMockRuntimeInfo;
5-
import io.securecodebox.persistence.defectdojo.config.Config;
64
import org.junit.jupiter.api.Disabled;
75
import org.junit.jupiter.api.Test;
86

97
import java.io.IOException;
108
import java.net.URI;
11-
import java.net.URISyntaxException;
129
import java.net.http.HttpClient;
1310
import java.net.http.HttpRequest;
1411
import java.net.http.HttpResponse;
@@ -27,7 +24,7 @@
2724
final class HttpClientExampleTest extends WireMockBaseTestCase {
2825

2926
private URI createUri(String path) {
30-
return URI.create("http://localhost:%d/%s".formatted(PORT, path));
27+
return URI.create("http://localhost:%d/%s".formatted(8080, path));
3128
}
3229

3330
@Test

src/test/java/io/securecodebox/persistence/defectdojo/service/ProductTypeServiceTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ void search() throws URISyntaxException, IOException {
2424
get("/api/v2/product_types/?offset=0&limit=100")
2525
.willReturn(
2626
ok()
27-
.withBody(readResponseBodyFromFixture("io/securecodebox/persistence/defectdojo/service/fixture_ProductTypeService.json"))
27+
.withBody(readResponseBodyFromFixture("ProductTypeService_response_fixture.json"))
2828
)
2929
);
3030

src/test/java/io/securecodebox/persistence/defectdojo/service/UserServiceTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ void search() throws URISyntaxException, IOException {
2727
get("/api/v2/users/?offset=0&limit=100")
2828
.willReturn(
2929
ok()
30-
.withBody(readResponseBodyFromFixture("io/securecodebox/persistence/defectdojo/service/fixture_UserService.json"))
30+
.withBody(readResponseBodyFromFixture("UserService_response_fixture.json"))
3131
)
3232
);
3333

src/test/java/io/securecodebox/persistence/defectdojo/service/WireMockBaseTestCase.java

+4-1
Original file line numberDiff line numberDiff line change
@@ -17,12 +17,15 @@
1717
@WireMockTest(httpPort = WireMockBaseTestCase.PORT)
1818
abstract class WireMockBaseTestCase {
1919
static final int PORT = 8888;
20+
private static final String FIXTURE_BASE_PACKAGE = "io/securecodebox/persistence/defectdojo/service";
2021

2122
private final Config conf = new Config(
2223
String.format("http://localhost:%d/", PORT),
2324
"not-required-for-tests");
2425

25-
String readResponseBodyFromFixture(String fixtureFilePath) throws IOException {
26+
String readResponseBodyFromFixture(String fixtureFile) throws IOException {
27+
final var fixtureFilePath = FIXTURE_BASE_PACKAGE + "/" + fixtureFile;
28+
2629
try (final var input = getClass().getClassLoader().getResourceAsStream(fixtureFilePath)) {
2730
final var bytes = Objects.requireNonNull(input).readAllBytes();
2831
return new String(bytes, StandardCharsets.UTF_8);

0 commit comments

Comments
 (0)