Skip to content

Commit 04238b1

Browse files
author
Dennis Labordus
committed
Fixed warning about dependencies.
Signed-off-by: Dennis Labordus <[email protected]>
1 parent a26752d commit 04238b1

File tree

10 files changed

+70
-66
lines changed

10 files changed

+70
-66
lines changed

app/pom.xml

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -47,31 +47,23 @@ SPDX-License-Identifier: Apache-2.0
4747
</dependency>
4848
<dependency>
4949
<groupId>io.quarkus</groupId>
50-
<artifactId>quarkus-resteasy</artifactId>
50+
<artifactId>quarkus-resteasy-reactive</artifactId>
5151
</dependency>
5252
<dependency>
5353
<groupId>io.quarkus</groupId>
54-
<artifactId>quarkus-smallrye-jwt</artifactId>
55-
</dependency>
56-
<dependency>
57-
<groupId>io.quarkus</groupId>
58-
<artifactId>quarkus-smallrye-openapi</artifactId>
59-
</dependency>
60-
<dependency>
61-
<groupId>io.quarkus</groupId>
62-
<artifactId>quarkus-smallrye-health</artifactId>
54+
<artifactId>quarkus-resteasy-reactive-jaxb</artifactId>
6355
</dependency>
6456
<dependency>
6557
<groupId>io.quarkus</groupId>
66-
<artifactId>quarkus-resteasy-jaxb</artifactId>
58+
<artifactId>quarkus-smallrye-jwt</artifactId>
6759
</dependency>
6860
<dependency>
6961
<groupId>io.quarkus</groupId>
70-
<artifactId>quarkus-resteasy-mutiny</artifactId>
62+
<artifactId>quarkus-smallrye-openapi</artifactId>
7163
</dependency>
7264
<dependency>
7365
<groupId>io.quarkus</groupId>
74-
<artifactId>quarkus-vertx</artifactId>
66+
<artifactId>quarkus-smallrye-health</artifactId>
7567
</dependency>
7668
<dependency>
7769
<groupId>io.quarkus</groupId>

app/src/main/java/org/lfenergy/compas/scl/validator/rest/v1/NsdocResource.java

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,9 @@
44
package org.lfenergy.compas.scl.validator.rest.v1;
55

66
import io.quarkus.security.Authenticated;
7+
import io.smallrye.mutiny.Uni;
78
import org.lfenergy.compas.scl.validator.rest.v1.model.NsdocListResponse;
9+
import org.lfenergy.compas.scl.validator.rest.v1.model.NsdocResponse;
810
import org.lfenergy.compas.scl.validator.service.NsdocService;
911

1012
import javax.enterprise.context.RequestScoped;
@@ -27,17 +29,19 @@ public NsdocResource(NsdocService nsdocService) {
2729
@GET
2830
@Consumes(MediaType.APPLICATION_XML)
2931
@Produces(MediaType.APPLICATION_XML)
30-
public NsdocListResponse list() {
31-
NsdocListResponse response = new NsdocListResponse();
32+
public Uni<NsdocListResponse> list() {
33+
var response = new NsdocListResponse();
3234
response.setNsdocFiles(nsdocService.list());
33-
return response;
35+
return Uni.createFrom().item(response);
3436
}
3537

3638
@GET
3739
@Path("{" + ID_PARAM + "}")
3840
@Consumes(MediaType.APPLICATION_XML)
3941
@Produces(MediaType.APPLICATION_XML)
40-
public String get(@PathParam(ID_PARAM) UUID id) {
41-
return nsdocService.get(id);
42+
public Uni<NsdocResponse> get(@PathParam(ID_PARAM) UUID id) {
43+
var response = new NsdocResponse();
44+
response.setNsdocFile(nsdocService.get(id));
45+
return Uni.createFrom().item(response);
4246
}
4347
}

app/src/main/java/org/lfenergy/compas/scl/validator/rest/v1/SclValidatorResource.java

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,10 @@
55

66
import io.quarkus.security.Authenticated;
77
import io.smallrye.mutiny.Uni;
8-
import io.vertx.mutiny.core.eventbus.EventBus;
9-
import io.vertx.mutiny.core.eventbus.Message;
108
import org.lfenergy.compas.scl.extensions.model.SclFileType;
11-
import org.lfenergy.compas.scl.validator.rest.v1.event.SclValidatorEventRequest;
129
import org.lfenergy.compas.scl.validator.rest.v1.model.SclValidateRequest;
1310
import org.lfenergy.compas.scl.validator.rest.v1.model.SclValidateResponse;
11+
import org.lfenergy.compas.scl.validator.service.SclValidatorService;
1412

1513
import javax.enterprise.context.RequestScoped;
1614
import javax.inject.Inject;
@@ -24,21 +22,20 @@
2422
@RequestScoped
2523
@Path("/validate/v1/{" + TYPE_PATH_PARAM + "}")
2624
public class SclValidatorResource {
27-
private final EventBus eventBus;
25+
private final SclValidatorService sclValidatorService;
2826

2927
@Inject
30-
public SclValidatorResource(EventBus eventBus) {
31-
this.eventBus = eventBus;
28+
public SclValidatorResource(SclValidatorService sclValidatorService) {
29+
this.sclValidatorService = sclValidatorService;
3230
}
3331

3432
@POST
3533
@Consumes(MediaType.APPLICATION_XML)
3634
@Produces(MediaType.APPLICATION_XML)
3735
public Uni<SclValidateResponse> validateSCL(@PathParam(TYPE_PATH_PARAM) SclFileType type,
3836
@Valid SclValidateRequest request) {
39-
return eventBus.<SclValidateResponse>request(
40-
"validate-rest",
41-
new SclValidatorEventRequest(type, request.getSclData()))
42-
.onItem().transform(Message::body);
37+
var response = new SclValidateResponse();
38+
response.setValidationErrorList(sclValidatorService.validate(type, request.getSclData()));
39+
return Uni.createFrom().item(response);
4340
}
4441
}

app/src/main/java/org/lfenergy/compas/scl/validator/rest/v1/event/SclValidatorEventHandler.java

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44
package org.lfenergy.compas.scl.validator.rest.v1.event;
55

66
import io.quarkus.vertx.ConsumeEvent;
7-
import io.smallrye.mutiny.Uni;
87
import org.lfenergy.compas.scl.validator.rest.v1.model.SclValidateResponse;
98
import org.lfenergy.compas.scl.validator.service.SclValidatorService;
109

@@ -31,11 +30,4 @@ public void validateWebsocketsEvent(SclValidatorEventRequest request) {
3130
var session = request.getSession();
3231
session.getAsyncRemote().sendObject(response);
3332
}
34-
35-
@ConsumeEvent(value = "validate-rest", blocking = true)
36-
public Uni<SclValidateResponse> validateRestEvent(SclValidatorEventRequest request) {
37-
var response = new SclValidateResponse();
38-
response.setValidationErrorList(sclValidatorService.validate(request.getType(), request.getSclData()));
39-
return Uni.createFrom().item(response);
40-
}
4133
}
Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
// SPDX-FileCopyrightText: 2022 Alliander N.V.
2+
//
3+
// SPDX-License-Identifier: Apache-2.0
4+
package org.lfenergy.compas.scl.validator.rest.v1.model;
5+
6+
import org.eclipse.microprofile.openapi.annotations.media.Schema;
7+
8+
import javax.xml.bind.annotation.XmlAccessType;
9+
import javax.xml.bind.annotation.XmlAccessorType;
10+
import javax.xml.bind.annotation.XmlElement;
11+
import javax.xml.bind.annotation.XmlRootElement;
12+
13+
import static org.lfenergy.compas.scl.validator.SclValidatorConstants.SCL_VALIDATOR_SERVICE_V1_NS_URI;
14+
15+
@Schema(description = "The response with NSDoc File.")
16+
@XmlRootElement(name = "NsdocResponse", namespace = SCL_VALIDATOR_SERVICE_V1_NS_URI)
17+
@XmlAccessorType(XmlAccessType.FIELD)
18+
public class NsdocResponse {
19+
@Schema(description = "The NSDoc File as String")
20+
@XmlElement(name = "NsdocFile", namespace = SCL_VALIDATOR_SERVICE_V1_NS_URI)
21+
private String nsdocFile;
22+
23+
public String getNsdocFile() {
24+
return nsdocFile;
25+
}
26+
27+
public void setNsdocFile(String nsdocFile) {
28+
this.nsdocFile = nsdocFile;
29+
}
30+
}

app/src/main/resources/application.properties

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ quarkus.http.cors = false
1010
# Sets the extra context for all RestEasy API Calls, for Websockets this needs to be included in the endpoint itself.
1111
quarkus.http.root-path = /compas-scl-validator
1212
quarkus.http.limits.max-body-size = 150M
13+
quarkus.websocket.max-frame-size = 157286400
1314

1415
quarkus.log.level = INFO
1516
quarkus.log.category."org.lfenergy.compas.scl.validator".level = INFO
@@ -22,7 +23,7 @@ quarkus.log.category."org.lfenergy.compas.scl.validator".level = INFO
2223
%dev.quarkus.http.port = 9093
2324
%dev.quarkus.http.cors = true
2425

25-
%dev.quarkus.log.level = DEBUG
26+
%dev.quarkus.log.level = TRACE
2627
%dev.quarkus.log.category."org.lfenergy.compas.scl.validator".level = DEBUG
2728

2829
# Test Profile overrides.

app/src/test/java/org/lfenergy/compas/scl/validator/rest/v1/NsdocResourceTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,9 @@ void get_WhenCalled_ThenContentReturned() {
7373
.extract()
7474
.response();
7575

76-
var nsdocFile = response.asString();
76+
var xmlPath = response.xmlPath()
77+
.using(xmlPathConfig().declaredNamespace("svs", SCL_VALIDATOR_SERVICE_V1_NS_URI));
78+
var nsdocFile = xmlPath.getString("svs:NsdocResponse.svs:NsdocFile");
7779
assertNotNull(nsdocFile);
7880
assertEquals(result, nsdocFile);
7981
verify(nsdocService, times(1)).get(id);

app/src/test/java/org/lfenergy/compas/scl/validator/rest/v1/event/SclValidatorEventHandlerTest.java

Lines changed: 0 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
// SPDX-License-Identifier: Apache-2.0
44
package org.lfenergy.compas.scl.validator.rest.v1.event;
55

6-
import io.smallrye.mutiny.helpers.test.UniAssertSubscriber;
76
import org.junit.jupiter.api.Test;
87
import org.junit.jupiter.api.extension.ExtendWith;
98
import org.lfenergy.compas.scl.extensions.model.SclFileType;
@@ -32,30 +31,6 @@ class SclValidatorEventHandlerTest {
3231
@InjectMocks
3332
private SclValidatorEventHandler eventHandler;
3433

35-
@Test
36-
void validateRestEvent_WhenCalled_ThenExpectedCallsAreMade() {
37-
var veList = new ArrayList<ValidationError>();
38-
var type = SclFileType.CID;
39-
var sclData = "Some SCL Data";
40-
41-
var request = Mockito.mock(SclValidatorEventRequest.class);
42-
when(request.getType()).thenReturn(type);
43-
when(request.getSclData()).thenReturn(sclData);
44-
45-
when(service.validate(type, sclData)).thenReturn(veList);
46-
47-
var result = eventHandler.validateRestEvent(request);
48-
49-
var subscriber = result.subscribe().withSubscriber(UniAssertSubscriber.create());
50-
var response = subscriber.assertCompleted().getItem();
51-
assertEquals(veList, response.getValidationErrorList());
52-
53-
verify(service, times(1)).validate(type, sclData);
54-
verify(request, never()).getSession();
55-
verify(request, times(1)).getType();
56-
verify(request, times(1)).getSclData();
57-
}
58-
5934
@Test
6035
void validateWebsocketsEvent_WhenCalled_ThenExpectedCallsAreMade() throws IOException {
6136
var veList = new ArrayList<ValidationError>();
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
// SPDX-FileCopyrightText: 2022 Alliander N.V.
2+
//
3+
// SPDX-License-Identifier: Apache-2.0
4+
package org.lfenergy.compas.scl.validator.rest.v1.model;
5+
6+
import org.lfenergy.compas.scl.validator.model.AbstractPojoTester;
7+
8+
class NsdocResponseTest extends AbstractPojoTester {
9+
@Override
10+
protected Class<?> getClassToBeTested() {
11+
return NsdocResponse.class;
12+
}
13+
}

service/src/main/java/org/lfenergy/compas/scl/validator/service/SclValidatorService.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55

66
import org.lfenergy.compas.scl.extensions.model.SclFileType;
77
import org.lfenergy.compas.scl.validator.SclValidator;
8-
import org.lfenergy.compas.scl.validator.xsd.SclXsdValidator;
98
import org.lfenergy.compas.scl.validator.model.ValidationError;
9+
import org.lfenergy.compas.scl.validator.xsd.SclXsdValidator;
1010

1111
import javax.enterprise.context.ApplicationScoped;
1212
import javax.inject.Inject;
@@ -25,11 +25,9 @@ public SclValidatorService(SclValidator validator, SclXsdValidator xsdValidator)
2525

2626
public List<ValidationError> validate(SclFileType type, String sclData) {
2727
var errors = xsdValidator.validate(sclData);
28-
2928
if (errors.isEmpty()) {
3029
errors = validator.validate(type, sclData);
3130
}
32-
3331
return errors;
3432
}
3533
}

0 commit comments

Comments
 (0)