Added name duplication check in REST API

Signed-off-by: Stef3st <[email protected]>

Author: Stef3st

app/src/main/java/org/lfenergy/compas/scl/data/rest/v1/CompasSclDataResource.java

@@ -143,4 +143,17 @@ public Uni<Void> deleteVersion(@PathParam(TYPE_PATH_PARAM) SclFileType type,
         compasSclDataService.delete(type, id, version);
         return Uni.createFrom().nullItem();
     }
+
+    @POST
+    @Path("/checkname")
+    @Consumes(MediaType.APPLICATION_XML)
+    @Produces(MediaType.APPLICATION_XML)
+    public Uni<DuplicateNameCheckResponse> checkDuplicateName(@PathParam(TYPE_PATH_PARAM) SclFileType type,
+                                                  @Valid DuplicateNameCheckRequest request) {
+        LOGGER.info("Checking for duplicate SCL File name.");
+
+        var response = new DuplicateNameCheckResponse();
+        response.setDuplicate(compasSclDataService.hasDuplicateSclName(type, request.getName()));
+        return Uni.createFrom().item(response);
+    }
 }

app/src/main/java/org/lfenergy/compas/scl/data/rest/v1/model/DuplicateNameCheckRequest.java

@@ -0,0 +1,30 @@
+package org.lfenergy.compas.scl.data.rest.v1.model;
+
+import org.eclipse.microprofile.openapi.annotations.media.Schema;
+import org.lfenergy.compas.core.commons.constraint.FilenameValid;
+
+import javax.xml.bind.annotation.XmlAccessType;
+import javax.xml.bind.annotation.XmlAccessorType;
+import javax.xml.bind.annotation.XmlElement;
+import javax.xml.bind.annotation.XmlRootElement;
+
+import static org.lfenergy.compas.scl.data.SclDataServiceConstants.SCL_DATA_SERVICE_V1_NS_URI;
+
+@Schema(description = "Request to check for duplicate names.")
+@XmlRootElement(name = "DuplicateNameCheckRequest", namespace = SCL_DATA_SERVICE_V1_NS_URI)
+@XmlAccessorType(XmlAccessType.FIELD)
+public class DuplicateNameCheckRequest {
+    @FilenameValid
+    @Schema(description = "The name that will be stored as Private Element and can later be used to determine the filename.",
+            example = "STATION-0012312")
+    @XmlElement(name = "Name", namespace = SCL_DATA_SERVICE_V1_NS_URI, required = true)
+    private String name;
+
+    public String getName() {
+        return name;
+    }
+
+    public void setName(String name) {
+        this.name = name;
+    }
+}

app/src/main/java/org/lfenergy/compas/scl/data/rest/v1/model/DuplicateNameCheckResponse.java

@@ -0,0 +1,27 @@
+package org.lfenergy.compas.scl.data.rest.v1.model;
+
+import org.eclipse.microprofile.openapi.annotations.media.Schema;
+
+import javax.xml.bind.annotation.XmlAccessType;
+import javax.xml.bind.annotation.XmlAccessorType;
+import javax.xml.bind.annotation.XmlElement;
+import javax.xml.bind.annotation.XmlRootElement;
+
+import static org.lfenergy.compas.scl.data.SclDataServiceConstants.SCL_DATA_SERVICE_V1_NS_URI;
+
+@Schema(description = "Response from duplicate name check in the database.")
+@XmlRootElement(name = "DuplicateNameCheckResponse", namespace = SCL_DATA_SERVICE_V1_NS_URI)
+@XmlAccessorType(XmlAccessType.FIELD)
+public class DuplicateNameCheckResponse {
+    @Schema(description = "Boolean result for duplicate name check.")
+    @XmlElement(name = "Duplicate", namespace = SCL_DATA_SERVICE_V1_NS_URI)
+    private boolean duplicate;
+
+    public boolean isDuplicate() {
+        return duplicate;
+    }
+
+    public void setDuplicate(boolean duplicate) {
+        this.duplicate = duplicate;
+    }
+}

app/src/main/resources/application.properties

@@ -48,7 +48,7 @@ quarkus.http.auth.permission.SSD_READ_GET.paths=/compas-scl-data-service/scl/v1/
 quarkus.http.auth.permission.SSD_READ_GET.policy=SSD_READ
 quarkus.http.auth.permission.SSD_READ_GET.methods=GET
 quarkus.http.auth.policy.SSD_CREATE.roles-allowed=SSD_CREATE
-quarkus.http.auth.permission.SSD_CREATE_POST.paths=/compas-scl-data-service/scl/v1/SSD
+quarkus.http.auth.permission.SSD_CREATE_POST.paths=/compas-scl-data-service/scl/v1/SSD/*
 quarkus.http.auth.permission.SSD_CREATE_POST.policy=SSD_CREATE
 quarkus.http.auth.permission.SSD_CREATE_POST.methods=POST
 quarkus.http.auth.policy.SSD_UPDATE.roles-allowed=SSD_UPDATE
@@ -74,7 +74,7 @@ quarkus.http.auth.permission.IID_READ_GET.paths=/compas-scl-data-service/scl/v1/
 quarkus.http.auth.permission.IID_READ_GET.policy=IID_READ
 quarkus.http.auth.permission.IID_READ_GET.methods=GET
 quarkus.http.auth.policy.IID_CREATE.roles-allowed=IID_CREATE
-quarkus.http.auth.permission.IID_CREATE_POST.paths=/compas-scl-data-service/scl/v1/IID
+quarkus.http.auth.permission.IID_CREATE_POST.paths=/compas-scl-data-service/scl/v1/IID/*
 quarkus.http.auth.permission.IID_CREATE_POST.policy=IID_CREATE
 quarkus.http.auth.permission.IID_CREATE_POST.methods=POST
 quarkus.http.auth.policy.IID_UPDATE.roles-allowed=IID_UPDATE
@@ -100,7 +100,7 @@ quarkus.http.auth.permission.ICD_READ_GET.paths=/compas-scl-data-service/scl/v1/
 quarkus.http.auth.permission.ICD_READ_GET.policy=ICD_READ
 quarkus.http.auth.permission.ICD_READ_GET.methods=GET
 quarkus.http.auth.policy.ICD_CREATE.roles-allowed=ICD_CREATE
-quarkus.http.auth.permission.ICD_CREATE_POST.paths=/compas-scl-data-service/scl/v1/ICD
+quarkus.http.auth.permission.ICD_CREATE_POST.paths=/compas-scl-data-service/scl/v1/ICD/*
 quarkus.http.auth.permission.ICD_CREATE_POST.policy=ICD_CREATE
 quarkus.http.auth.permission.ICD_CREATE_POST.methods=POST
 quarkus.http.auth.policy.ICD_UPDATE.roles-allowed=ICD_UPDATE
@@ -126,7 +126,7 @@ quarkus.http.auth.permission.SCD_READ_GET.paths=/compas-scl-data-service/scl/v1/
 quarkus.http.auth.permission.SCD_READ_GET.policy=SCD_READ
 quarkus.http.auth.permission.SCD_READ_GET.methods=GET
 quarkus.http.auth.policy.SCD_CREATE.roles-allowed=SCD_CREATE
-quarkus.http.auth.permission.SCD_CREATE_POST.paths=/compas-scl-data-service/scl/v1/SCD
+quarkus.http.auth.permission.SCD_CREATE_POST.paths=/compas-scl-data-service/scl/v1/SCD/*
 quarkus.http.auth.permission.SCD_CREATE_POST.policy=SCD_CREATE
 quarkus.http.auth.permission.SCD_CREATE_POST.methods=POST
 quarkus.http.auth.policy.SCD_UPDATE.roles-allowed=SCD_UPDATE
@@ -152,7 +152,7 @@ quarkus.http.auth.permission.CID_READ_GET.paths=/compas-scl-data-service/scl/v1/
 quarkus.http.auth.permission.CID_READ_GET.policy=CID_READ
 quarkus.http.auth.permission.CID_READ_GET.methods=GET
 quarkus.http.auth.policy.CID_CREATE.roles-allowed=CID_CREATE
-quarkus.http.auth.permission.CID_CREATE_POST.paths=/compas-scl-data-service/scl/v1/CID
+quarkus.http.auth.permission.CID_CREATE_POST.paths=/compas-scl-data-service/scl/v1/CID/*
 quarkus.http.auth.permission.CID_CREATE_POST.policy=CID_CREATE
 quarkus.http.auth.permission.CID_CREATE_POST.methods=POST
 quarkus.http.auth.policy.CID_UPDATE.roles-allowed=CID_UPDATE
@@ -178,7 +178,7 @@ quarkus.http.auth.permission.SED_READ_GET.paths=/compas-scl-data-service/scl/v1/
 quarkus.http.auth.permission.SED_READ_GET.policy=SED_READ
 quarkus.http.auth.permission.SED_READ_GET.methods=GET
 quarkus.http.auth.policy.SED_CREATE.roles-allowed=SED_CREATE
-quarkus.http.auth.permission.SED_CREATE_POST.paths=/compas-scl-data-service/scl/v1/SED
+quarkus.http.auth.permission.SED_CREATE_POST.paths=/compas-scl-data-service/scl/v1/SED/*
 quarkus.http.auth.permission.SED_CREATE_POST.policy=SED_CREATE
 quarkus.http.auth.permission.SED_CREATE_POST.methods=POST
 quarkus.http.auth.policy.SED_UPDATE.roles-allowed=SED_UPDATE
@@ -204,7 +204,7 @@ quarkus.http.auth.permission.ISD_READ_GET.paths=/compas-scl-data-service/scl/v1/
 quarkus.http.auth.permission.ISD_READ_GET.policy=ISD_READ
 quarkus.http.auth.permission.ISD_READ_GET.methods=GET
 quarkus.http.auth.policy.ISD_CREATE.roles-allowed=ISD_CREATE
-quarkus.http.auth.permission.ISD_CREATE_POST.paths=/compas-scl-data-service/scl/v1/ISD
+quarkus.http.auth.permission.ISD_CREATE_POST.paths=/compas-scl-data-service/scl/v1/ISD/*
 quarkus.http.auth.permission.ISD_CREATE_POST.policy=ISD_CREATE
 quarkus.http.auth.permission.ISD_CREATE_POST.methods=POST
 quarkus.http.auth.policy.ISD_UPDATE.roles-allowed=ISD_UPDATE
@@ -230,7 +230,7 @@ quarkus.http.auth.permission.STD_READ_GET.paths=/compas-scl-data-service/scl/v1/
 quarkus.http.auth.permission.STD_READ_GET.policy=STD_READ
 quarkus.http.auth.permission.STD_READ_GET.methods=GET
 quarkus.http.auth.policy.STD_CREATE.roles-allowed=STD_CREATE
-quarkus.http.auth.permission.STD_CREATE_POST.paths=/compas-scl-data-service/scl/v1/STD
+quarkus.http.auth.permission.STD_CREATE_POST.paths=/compas-scl-data-service/scl/v1/STD/*
 quarkus.http.auth.permission.STD_CREATE_POST.policy=STD_CREATE
 quarkus.http.auth.permission.STD_CREATE_POST.methods=POST
 quarkus.http.auth.policy.STD_UPDATE.roles-allowed=STD_UPDATE
@@ -249,4 +249,4 @@ quarkus.http.auth.permission.STD_READ_GET_VERSION_WS.policy=STD_READ
 quarkus.http.auth.permission.STD_CREATE_POST_WS.paths=/compas-scl-data-service/scl-ws/v1/STD/create
 quarkus.http.auth.permission.STD_CREATE_POST_WS.policy=STD_CREATE
 quarkus.http.auth.permission.STD_UPDATE_PUT_WS.paths=/compas-scl-data-service/scl-ws/v1/STD/update
-quarkus.http.auth.permission.STD_UPDATE_PUT_WS.policy=STD_UPDATE
+quarkus.http.auth.permission.STD_UPDATE_PUT_WS.policy=STD_UPDATE

app/src/test/java/org/lfenergy/compas/scl/data/rest/v1/CompasSclDataResourceAsEditorTest.java

@@ -16,6 +16,7 @@
 import org.lfenergy.compas.scl.data.model.Item;
 import org.lfenergy.compas.scl.data.model.Version;
 import org.lfenergy.compas.scl.data.rest.v1.model.CreateRequest;
+import org.lfenergy.compas.scl.data.rest.v1.model.DuplicateNameCheckRequest;
 import org.lfenergy.compas.scl.data.rest.v1.model.UpdateRequest;
 import org.lfenergy.compas.scl.data.service.CompasSclDataService;
 import org.lfenergy.compas.scl.extensions.model.SclFileType;
@@ -27,7 +28,7 @@
 
 import static io.restassured.RestAssured.given;
 import static io.restassured.path.xml.config.XmlPathConfig.xmlPathConfig;
-import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.*;
 import static org.lfenergy.compas.scl.data.SclDataServiceConstants.SCL_NS_URI;
 import static org.lfenergy.compas.scl.data.rest.Constants.*;
 import static org.mockito.Mockito.*;
@@ -267,6 +268,54 @@ void deleteVersion_WhenCalled_ThenServiceCalled() {
         verify(compasSclDataService).delete(type, uuid, version);
     }
 
+    @Test
+    void checkNameForDuplication_WhenCalled_WithDuplicateName_ThenServiceCalled() {
+        var type = SclFileType.SCD;
+        var name = "STATION-0012312";
+
+        when(compasSclDataService.hasDuplicateSclName(type, name)).thenReturn(true);
+
+        var request = new DuplicateNameCheckRequest();
+        request.setName(name);
+
+        var response = given()
+                .pathParam(TYPE_PATH_PARAM, type)
+                .contentType(ContentType.XML)
+                .body(request)
+                .when().post("/checkname")
+                .then()
+                .statusCode(200)
+                .extract()
+                .response();
+
+        verify(compasSclDataService).hasDuplicateSclName(type, name);
+        assertTrue(response.xmlPath().getBoolean("DuplicateNameCheckResponse.Duplicate"));
+    }
+
+    @Test
+    void checkNameForDuplication_WhenCalled_WithUniqueName_ThenServiceCalled() {
+        var type = SclFileType.SCD;
+        var name = "STATION-0012312";
+
+        when(compasSclDataService.hasDuplicateSclName(type, name)).thenReturn(false);
+
+        var request = new DuplicateNameCheckRequest();
+        request.setName(name);
+
+        var response = given()
+                .pathParam(TYPE_PATH_PARAM, type)
+                .contentType(ContentType.XML)
+                .body(request)
+                .when().post("/checkname")
+                .then()
+                .statusCode(200)
+                .extract()
+                .response();
+
+        verify(compasSclDataService).hasDuplicateSclName(type, name);
+        assertFalse(response.xmlPath().getBoolean("DuplicateNameCheckResponse.Duplicate"));
+    }
+
     private String readSCL() throws IOException {
         try (var inputStream = getClass().getResourceAsStream("/scl/icd_import_ied_test.scd")) {
             assert inputStream != null;

app/src/test/java/org/lfenergy/compas/scl/data/rest/v1/model/DuplicateNameCheckRequestTest.java

@@ -0,0 +1,8 @@
+package org.lfenergy.compas.scl.data.rest.v1.model;
+
+class DuplicateNameCheckRequestTest extends AbstractPojoTester {
+    @Override
+    protected Class<?> getClassToBeTested() {
+        return DuplicateNameCheckRequest.class;
+    }
+}

app/src/test/java/org/lfenergy/compas/scl/data/rest/v1/model/DuplicateNameCheckResponseTest.java

@@ -0,0 +1,9 @@
+package org.lfenergy.compas.scl.data.rest.v1.model;
+
+class DuplicateNameCheckResponseTest extends AbstractPojoTester{
+
+    @Override
+    protected Class<?> getClassToBeTested() {
+        return DuplicateNameCheckResponse.class;
+    }
+}

repository-postgresql/src/main/java/org/lfenergy/compas/scl/data/repository/postgresql/CompasSclDataPostgreSQLRepository.java

@@ -191,7 +191,7 @@ public String findByUUID(SclFileType type, UUID id, Version version) {
     @Transactional(SUPPORTS)
     public boolean hasDuplicateSclName(SclFileType type, String name) {
         var sql = """
-                select distinct on (scl_file.id) *
+                select distinct on (scl_file.id) scl_file.name
                   from scl_file
                  where scl_file.type = ?
                  order by scl_file.id

repository-postgresql/src/test/java/org/lfenergy/compas/scl/data/repository/postgresql/CompasSclDataPostgreSQLRepositoryTest.java

@@ -29,9 +29,6 @@ void beforeEach() {
         repository = new CompasSclDataPostgreSQLRepository(PostgreSQLServerJUnitExtension.getDataSource());
     }
 
-    /*
-     * TODO: Method beneath needs to be moved to AbstractCompasSclDataRepositoryTest
-     * when hasDuplicateSclName has been implemented by CompasSclDataBaseXRepository. */
     @Test
     void hasDuplicateSclName_WhenUsingSclNameThatHasBeenUsedYet_ThenDuplicateIsFound() {
         var expectedVersion = new Version(1, 0, 0);

service/src/main/java/org/lfenergy/compas/scl/data/service/CompasSclDataService.java

@@ -122,7 +122,7 @@ public String create(SclFileType type, String name, String who, String comment,
             throw new CompasException(NO_SCL_ELEMENT_FOUND_ERROR_CODE, "No valid SCL found in the passed SCL Data.");
         }
 
-        if (repository.hasDuplicateSclName(type, name)) {
+        if (hasDuplicateSclName(type, name)) {
             throw new CompasException(DUPLICATE_SCL_NAME_ERROR_CODE, "Given name of SCL File already used.");
         }
 
@@ -147,6 +147,10 @@ public String create(SclFileType type, String name, String who, String comment,
         return newSclData;
     }
 
+    public boolean hasDuplicateSclName(SclFileType type, String name){
+        return repository.hasDuplicateSclName(type, name);
+    }
+
     /**
      * Create a new version of a specific SCL XML File. The content will be the passed SCL XML File.
      * The UUID and new version (depending on the passed ChangeSetType) are set and