Skip to content

Commit 09cab83

Browse files
Add eiffel-commons to integration tests (#186)
* Use eiffel-commons 0.0.6 * Replace http utils with the ones in eiffel-commons
1 parent 8e017c6 commit 09cab83

File tree

4 files changed

+32
-256
lines changed

4 files changed

+32
-256
lines changed

pom.xml

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,13 @@
4646
</plugins>
4747
</reporting>
4848

49+
<repositories>
50+
<repository>
51+
<id>jitpack.io</id>
52+
<url>https://jitpack.io</url>
53+
</repository>
54+
</repositories>
55+
4956
<dependencies>
5057
<dependency>
5158
<groupId>org.springframework.boot</groupId>
@@ -164,6 +171,12 @@
164171
<version>5.4.0</version>
165172
<scope>test</scope>
166173
</dependency>
174+
<!-- Eiffel Commons -->
175+
<dependency>
176+
<groupId>com.github.eiffel-community</groupId>
177+
<artifactId>eiffel-commons</artifactId>
178+
<version>0.0.6</version>
179+
</dependency>
167180
</dependencies>
168181

169182
<build>

src/integrationtest/java/com/ericsson/ei/frontend/CommonSteps.java

Lines changed: 19 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
import java.nio.file.Paths;
88
import java.util.ArrayList;
99
import java.util.List;
10-
import java.util.concurrent.TimeUnit;
1110
import java.util.concurrent.TimeoutException;
1211

1312
import org.apache.commons.io.FileUtils;
@@ -23,16 +22,16 @@
2322
import org.springframework.boot.test.context.SpringBootTest.WebEnvironment;
2423
import org.springframework.boot.web.server.LocalServerPort;
2524
import org.springframework.http.HttpStatus;
26-
import org.springframework.http.ResponseEntity;
2725
import org.springframework.test.context.ContextConfiguration;
2826
import org.springframework.test.context.TestExecutionListeners;
2927
import org.springframework.test.context.junit4.SpringRunner;
3028
import org.springframework.test.context.support.AbstractTestExecutionListener;
3129
import org.springframework.test.context.support.DependencyInjectionTestExecutionListener;
3230

3331
import com.ericsson.ei.utils.AMQPCommunication;
34-
import com.ericsson.ei.utils.HttpRequest;
35-
import com.ericsson.ei.utils.HttpRequest.HttpMethod;
32+
import com.ericsson.eiffelcommons.utils.HttpRequest;
33+
import com.ericsson.eiffelcommons.utils.HttpRequest.HttpMethod;
34+
import com.ericsson.eiffelcommons.utils.ResponseEntity;
3635

3736
import cucumber.api.java.Before;
3837
import cucumber.api.java.en.Given;
@@ -47,8 +46,10 @@
4746
public class CommonSteps extends AbstractTestExecutionListener {
4847

4948
@LocalServerPort
50-
private int frontendPort;
51-
private String frontendHost = "localhost";
49+
private int frontEndPort;
50+
private String frontEndHost = "localhost";
51+
private String protocol = "http";
52+
private String baseURL;
5253
private String rabbitHost;
5354
private int rabbitPort;
5455
private String rabbitUsername;
@@ -58,7 +59,7 @@ public class CommonSteps extends AbstractTestExecutionListener {
5859

5960
private List<HttpRequest> httpRequestList = new ArrayList<>();
6061
private HttpRequest httpRequest;
61-
private ResponseEntity<String> response;
62+
private ResponseEntity response;
6263

6364
private static final String RESOURCE_PATH = "src/integrationtest/resources/";
6465
private static final String BODIES_PATH = "bodies/";
@@ -67,6 +68,11 @@ public class CommonSteps extends AbstractTestExecutionListener {
6768

6869
private static final Logger LOGGER = LoggerFactory.getLogger(CommonSteps.class);
6970

71+
@Before
72+
public void beforeAllScenarios() {
73+
baseURL = String.format("%s://%s:%d", protocol, frontEndHost, frontEndPort);
74+
}
75+
7076
@Before("@QueryByIdScenario or @QueryFreestyleScenario")
7177
public void beforeQueryScenario() {
7278
rabbitHost = System.getProperty("rabbit.host");
@@ -79,8 +85,8 @@ public void beforeQueryScenario() {
7985

8086
@Given("^frontend is up and running$")
8187
public void frontend_running() {
82-
LOGGER.debug("Front-end port: {}", frontendPort);
83-
assertEquals(true, frontendPort != 0);
88+
LOGGER.debug("Front-end port: {}", frontEndPort);
89+
assertEquals(true, frontEndPort != 0);
8490
}
8591

8692
@Given("^an aggregated object is created$")
@@ -99,7 +105,7 @@ public void aggregated_object_created() throws IOException, TimeoutException {
99105
public void request_to_rest_api(String method, String endpoint) throws Throwable {
100106
LOGGER.debug("Method: {}, Endpoint: {}", method, endpoint);
101107
httpRequest = new HttpRequest(HttpMethod.valueOf(method));
102-
httpRequest.setHost(frontendHost).setPort(frontendPort).setEndpoint(endpoint);
108+
httpRequest.setBaseUrl(baseURL).setEndpoint(endpoint);
103109
}
104110

105111
@When("^\'(.*)\' endpoint is set in request list at index (\\d+)$")
@@ -158,8 +164,8 @@ public void request_sent_body_not_received(int seconds, int statusCode) throws T
158164
long stopTime = System.currentTimeMillis() + (seconds * 1000);
159165
do {
160166
response = httpRequest.performRequest();
161-
} while (response.getStatusCode().value() == statusCode && stopTime > System.currentTimeMillis());
162-
assertEquals(HttpStatus.OK, response.getStatusCode());
167+
} while (response.getStatusCode() == statusCode && stopTime > System.currentTimeMillis());
168+
assertEquals(HttpStatus.OK.value(), response.getStatusCode());
163169
}
164170

165171
@Then("^request is saved to request list at index (\\d+)$")
@@ -171,7 +177,7 @@ public void request_is_saved_to_list(int index) throws Throwable {
171177
@Then("^response code (\\d+) is received$")
172178
public void get_response_code(int statusCode) throws Throwable {
173179
LOGGER.debug("Response code: {}", response.getStatusCode());
174-
assertEquals(HttpStatus.valueOf(statusCode), response.getStatusCode());
180+
assertEquals(statusCode, response.getStatusCode());
175181
}
176182

177183
@Then("^response body \'(.*)\' is received$")

src/integrationtest/java/com/ericsson/ei/utils/HttpExecutor.java

Lines changed: 0 additions & 81 deletions
This file was deleted.

src/integrationtest/java/com/ericsson/ei/utils/HttpRequest.java

Lines changed: 0 additions & 162 deletions
This file was deleted.

0 commit comments

Comments
 (0)