Added unit tests for repository

Signed-off-by: Pascal Wilbrink <pascal.wilbrink@alliander.com>

Author: pascalwilbrink

app/src/test/java/org/lfenergy/compas/sitipe/rest/v1/model/BayTypicalResponseTest.java

@@ -19,7 +19,7 @@
 class BayTypicalResponseTest {
 
     @Test
-    public void itShouldMapProperties() {
+    void itShouldMapProperties() {
         final BayTypical entity = new BayTypical();
 
         entity.setId(1);

repository/pom.xml

@@ -67,6 +67,11 @@ SPDX-License-Identifier: Apache-2.0
             <groupId>io.quarkus</groupId>
             <artifactId>quarkus-hibernate-orm-panache</artifactId>
         </dependency>
+        <dependency>
+            <groupId>com.openpojo</groupId>
+            <artifactId>openpojo</artifactId>
+            <scope>test</scope>
+        </dependency>
     </dependencies>
 
     <build>

repository/src/main/java/org/lfenergy/compas/sitipe/data/repository/BayTypicalRepository.java

@@ -17,7 +17,4 @@ public List<BayTypical> findByAccessId(final String accessId) {
         return this.list("accessId", accessId);
     }
 
-
-
-
 }

repository/src/main/java/org/lfenergy/compas/sitipe/data/repository/ImportedComponentRepository.java

@@ -15,7 +15,7 @@
 public class ImportedComponentRepository implements PanacheRepositoryBase<ImportedComponent, Integer> {
 
     public List<ImportedComponent> getByAccessId(final String accessId) {
-        return this.list("componentAccessId = ?1 AND type = 'IID'", accessId);
+        return this.list("componentAccessId = ?1 AND type = ?2", accessId, "IID");
     }
 
     public ImportedComponent getById(final Integer id) {

repository/src/test/java/org/lfenergy/compas/sitipe/data/entity/BTComponentTest.java

@@ -0,0 +1,14 @@
+// SPDX-FileCopyrightText: 2023 Alliander N.V.
+//
+// SPDX-License-Identifier: Apache-2.0
+
+package org.lfenergy.compas.sitipe.data.entity;
+
+class BTComponentTest extends BaseEntityTest {
+
+    @Override
+    protected Class<?> getClassToBeTested() {
+        return BTComponent.class;
+    }
+
+}

repository/src/test/java/org/lfenergy/compas/sitipe/data/entity/BaseEntityTest.java

@@ -0,0 +1,34 @@
+// SPDX-FileCopyrightText: 2023 Alliander N.V.
+//
+// SPDX-License-Identifier: Apache-2.0
+
+package org.lfenergy.compas.sitipe.data.entity;
+
+import com.openpojo.reflection.impl.PojoClassFactory;
+import com.openpojo.validation.ValidatorBuilder;
+import com.openpojo.validation.rule.impl.GetterMustExistRule;
+import com.openpojo.validation.rule.impl.SetterMustExistRule;
+import com.openpojo.validation.test.impl.GetterTester;
+import com.openpojo.validation.test.impl.SetterTester;
+import org.junit.jupiter.api.Test;
+
+abstract class BaseEntityTest {
+
+    @Test
+    void validateSettersAndGetters() {
+        var personPojo = PojoClassFactory.getPojoClass(getClassToBeTested());
+        var validator = ValidatorBuilder.create()
+                // Let's make sure that we have a getter and a setter for every field defined.
+                .with(new GetterMustExistRule())
+                .with(new SetterMustExistRule())
+                // Let's also validate that they are behaving as expected
+                .with(new SetterTester())
+                .with(new GetterTester())
+                .build();
+
+        // Start the Test
+        validator.validate(personPojo);
+    }
+
+    protected abstract Class<?> getClassToBeTested();
+}

repository/src/test/java/org/lfenergy/compas/sitipe/data/entity/BayTypicalTest.java

@@ -0,0 +1,14 @@
+// SPDX-FileCopyrightText: 2023 Alliander N.V.
+//
+// SPDX-License-Identifier: Apache-2.0
+
+package org.lfenergy.compas.sitipe.data.entity;
+
+class BayTypicalTest extends BaseEntityTest {
+
+    @Override
+    protected Class<?> getClassToBeTested() {
+        return BayTypical.class;
+    }
+
+}

repository/src/test/java/org/lfenergy/compas/sitipe/data/entity/ImportedComponentTest.java

@@ -0,0 +1,14 @@
+// SPDX-FileCopyrightText: 2023 Alliander N.V.
+//
+// SPDX-License-Identifier: Apache-2.0
+
+package org.lfenergy.compas.sitipe.data.entity;
+
+class ImportedComponentTest extends BaseEntityTest {
+
+    @Override
+    protected Class<?> getClassToBeTested() {
+        return ImportedComponent.class;
+    }
+
+}

repository/src/test/java/org/lfenergy/compas/sitipe/data/entity/SystemVersionTest.java

@@ -0,0 +1,14 @@
+// SPDX-FileCopyrightText: 2023 Alliander N.V.
+//
+// SPDX-License-Identifier: Apache-2.0
+
+package org.lfenergy.compas.sitipe.data.entity;
+
+class SystemVersionTest extends BaseEntityTest {
+
+    @Override
+    protected Class<?> getClassToBeTested() {
+        return SystemVersion.class;
+    }
+
+}

repository/src/test/java/org/lfenergy/compas/sitipe/data/repository/BTComponentRepositoryTest.java

@@ -0,0 +1,51 @@
+// SPDX-FileCopyrightText: 2023 Alliander N.V.
+//
+// SPDX-License-Identifier: Apache-2.0
+
+package org.lfenergy.compas.sitipe.data.repository;
+
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
+import org.lfenergy.compas.sitipe.data.entity.BTComponent;
+import org.lfenergy.compas.sitipe.data.entity.BayTypical;
+import org.mockito.ArgumentCaptor;
+import org.mockito.junit.jupiter.MockitoExtension;
+
+import java.util.List;
+import java.util.UUID;
+
+import static java.util.Collections.emptyList;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.mockito.Mockito.doReturn;
+import static org.mockito.Mockito.spy;
+
+@ExtendWith(MockitoExtension.class)
+class BTComponentRepositoryTest {
+
+    private BTComponentRepository sut;
+
+    @BeforeEach
+    public void setUp() {
+        sut = spy(BTComponentRepository.class);
+    }
+
+    @Test
+    void itShouldGetBTComponents() {
+        final ArgumentCaptor<String> causeArgumentCaptor = ArgumentCaptor.forClass(String.class);
+        final ArgumentCaptor<String> typeArgumentCaptor = ArgumentCaptor.forClass(String.class);
+        final ArgumentCaptor<String> toolNameArgumentCaptor = ArgumentCaptor.forClass(String.class);
+
+        doReturn(emptyList()).when(sut).list(causeArgumentCaptor.capture(), typeArgumentCaptor.capture(), toolNameArgumentCaptor.capture());
+
+        final String accessId = UUID.randomUUID().toString();
+
+        final List<BTComponent> result = sut.findBayTypicalComponentsByTypicalAccessId(accessId);
+
+        assertEquals("DIGSI 5", toolNameArgumentCaptor.getValue());
+
+        assertEquals("typicalAccessId = ?1 AND toolName = ?2", causeArgumentCaptor.getValue());
+        assertEquals(accessId, typeArgumentCaptor.getValue());
+        assertEquals(0, result.size());
+    }
+}