Test covergae to 55%

checkmarx-ast-eclipse-plugin-tests/pom.xml

@@ -43,9 +43,34 @@
 			                <formats>
 			                    <format>XML</format>
 			                    <format>CSV</format>
+			                    <format>HTML</format>
 			                </formats>
 			            </configuration>
 			        </execution>
+			        <execution>
+			            <id>check</id>
+			            <phase>verify</phase>
+			            <goals><goal>check</goal></goals>
+			            <configuration>
+			                <dataFile>${project.build.directory}/jacoco.exec</dataFile>
+			                <classesDirectory>${project.basedir}/../checkmarx-ast-eclipse-plugin/target/classes</classesDirectory>
+			                <excludes>
+			                    <exclude>org/eclipse/wb/swt/SWTResourceManager.class</exclude>
+			                </excludes>
+			                <rules>
+			                    <rule>
+			                        <element>BUNDLE</element>
+			                        <limits>
+			                            <limit>
+			                                <counter>INSTRUCTION</counter>
+			                                <value>COVEREDRATIO</value>
+			                                <minimum>0.30</minimum>
+			                            </limit>
+			                        </limits>
+			                    </rule>
+			                </rules>
+			            </configuration>
+			        </execution>
 				</executions>				
 			</plugin>
 			<plugin>

checkmarx-ast-eclipse-plugin-tests/src/test/java/checkmarx/ast/eclipse/plugin/tests/unit/enums/SeverityTest.java

@@ -0,0 +1,67 @@
+package checkmarx.ast.eclipse.plugin.tests.unit.enums;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+import org.junit.jupiter.api.Test;
+
+import com.checkmarx.eclipse.enums.Severity;
+
+class SeverityTest {
+
+    @Test
+    void testGetSeverity_critical() {
+        assertEquals(Severity.CRITICAL, Severity.getSeverity("CRITICAL"));
+    }
+
+    @Test
+    void testGetSeverity_high() {
+        assertEquals(Severity.HIGH, Severity.getSeverity("HIGH"));
+    }
+
+    @Test
+    void testGetSeverity_medium() {
+        assertEquals(Severity.MEDIUM, Severity.getSeverity("MEDIUM"));
+    }
+
+    @Test
+    void testGetSeverity_low() {
+        assertEquals(Severity.LOW, Severity.getSeverity("LOW"));
+    }
+
+    @Test
+    void testGetSeverity_info() {
+        assertEquals(Severity.INFO, Severity.getSeverity("INFO"));
+    }
+
+    @Test
+    void testGetSeverity_groupBySeverity() {
+        assertEquals(Severity.GROUP_BY_SEVERITY, Severity.getSeverity("GROUP_BY_SEVERITY"));
+    }
+
+    @Test
+    void testGetSeverity_groupByQueryName() {
+        assertEquals(Severity.GROUP_BY_QUERY_NAME, Severity.getSeverity("GROUP_BY_QUERY_NAME"));
+    }
+
+    @Test
+    void testGetSeverity_groupByStateName() {
+        assertEquals(Severity.GROUP_BY_STATE_NAME, Severity.getSeverity("GROUP_BY_STATE_NAME"));
+    }
+
+    @Test
+    void testGetSeverity_unknownValue_throwsIllegalArgumentException() {
+        assertThrows(IllegalArgumentException.class, () -> Severity.getSeverity("UNKNOWN_SEVERITY_XYZ"));
+    }
+
+    @Test
+    void testGetSeverity_roundTrip_allValues() {
+        for (Severity severity : Severity.values()) {
+            assertEquals(severity, Severity.getSeverity(severity.name()));
+        }
+    }
+
+    @Test
+    void testEnumValues_count() {
+        assertEquals(8, Severity.values().length);
+    }
+}

checkmarx-ast-eclipse-plugin-tests/src/test/java/checkmarx/ast/eclipse/plugin/tests/unit/enums/StateTest.java

@@ -0,0 +1,123 @@
+package checkmarx.ast.eclipse.plugin.tests.unit.enums;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+import java.util.Map;
+
+import org.junit.jupiter.api.Test;
+
+import com.checkmarx.eclipse.enums.State;
+
+class StateTest {
+
+    // ─── predefined constants ────────────────────────────────────────────────
+
+    @Test
+    void testPredefinedConstants_notNull() {
+        assertNotNull(State.TO_VERIFY);
+        assertNotNull(State.NOT_EXPLOITABLE);
+        assertNotNull(State.PROPOSED_NOT_EXPLOITABLE);
+        assertNotNull(State.CONFIRMED);
+        assertNotNull(State.NOT_IGNORED);
+        assertNotNull(State.IGNORED);
+        assertNotNull(State.URGENT);
+    }
+
+    @Test
+    void testPredefinedConstants_names() {
+        assertEquals("TO_VERIFY", State.TO_VERIFY.getName());
+        assertEquals("NOT_EXPLOITABLE", State.NOT_EXPLOITABLE.getName());
+        assertEquals("PROPOSED_NOT_EXPLOITABLE", State.PROPOSED_NOT_EXPLOITABLE.getName());
+        assertEquals("CONFIRMED", State.CONFIRMED.getName());
+        assertEquals("NOT_IGNORED", State.NOT_IGNORED.getName());
+        assertEquals("IGNORED", State.IGNORED.getName());
+        assertEquals("URGENT", State.URGENT.getName());
+    }
+
+    // ─── toString ────────────────────────────────────────────────────────────
+
+    @Test
+    void testToString_returnsName() {
+        assertEquals("TO_VERIFY", State.TO_VERIFY.toString());
+        assertEquals("CONFIRMED", State.CONFIRMED.toString());
+        assertEquals("IGNORED", State.IGNORED.toString());
+        assertEquals("URGENT", State.URGENT.toString());
+    }
+
+    // ─── getState ────────────────────────────────────────────────────────────
+
+    @Test
+    void testGetState_existingPredefined_returnsInstance() {
+        assertNotNull(State.getState("TO_VERIFY"));
+        assertEquals("TO_VERIFY", State.getState("TO_VERIFY").getName());
+    }
+
+    @Test
+    void testGetState_allPredefinedStates_found() {
+        assertNotNull(State.getState("NOT_EXPLOITABLE"));
+        assertNotNull(State.getState("PROPOSED_NOT_EXPLOITABLE"));
+        assertNotNull(State.getState("CONFIRMED"));
+        assertNotNull(State.getState("NOT_IGNORED"));
+        assertNotNull(State.getState("IGNORED"));
+        assertNotNull(State.getState("URGENT"));
+    }
+
+    @Test
+    void testGetState_nonExistent_returnsNull() {
+        assertNull(State.getState("DOES_NOT_EXIST_STATE_XYZ_123"));
+    }
+
+    // ─── of ──────────────────────────────────────────────────────────────────
+
+    @Test
+    void testOf_existingPredefined_returnsSameInstance() {
+        assertSame(State.TO_VERIFY, State.of("TO_VERIFY"));
+    }
+
+    @Test
+    void testOf_newName_createsAndRegisters() {
+        String name = "CUSTOM_OF_TEST_UNIQUE_E5";
+        State result = State.of(name);
+        assertNotNull(result);
+        assertEquals(name, result.getName());
+        assertSame(result, State.getState(name));
+    }
+
+    @Test
+    void testOf_sameName_returnsSameInstance() {
+        String name = "CUSTOM_OF_SAME_UNIQUE_F6";
+        State first = State.of(name);
+        State second = State.of(name);
+        assertSame(first, second);
+    }
+
+    @Test
+    void testOf_customState_toString() {
+        String name = "MY_CUSTOM_STATE_G7";
+        State state = State.of(name);
+        assertEquals(name, state.toString());
+    }
+
+    // ─── values ──────────────────────────────────────────────────────────────
+
+    @Test
+    void testValues_isUnmodifiable() {
+        Map<String, State> vals = State.values();
+        assertThrows(UnsupportedOperationException.class, () -> vals.put("NEW", State.TO_VERIFY));
+    }
+
+    @Test
+    void testValues_notEmpty() {
+        assertFalse(State.values().isEmpty());
+    }
+
+    @Test
+    void testValues_containsPredefinedKeys() {
+        Map<String, State> vals = State.values();
+        assertTrue(vals.containsKey("TO_VERIFY"));
+        assertTrue(vals.containsKey("CONFIRMED"));
+        assertTrue(vals.containsKey("IGNORED"));
+        assertTrue(vals.containsKey("NOT_EXPLOITABLE"));
+        assertTrue(vals.containsKey("URGENT"));
+    }
+}

checkmarx-ast-eclipse-plugin-tests/src/test/java/checkmarx/ast/eclipse/plugin/tests/unit/utils/CxLoggerTest.java

@@ -0,0 +1,25 @@
+package checkmarx.ast.eclipse.plugin.tests.unit.utils;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+import org.junit.jupiter.api.Test;
+
+import com.checkmarx.eclipse.utils.CxLogger;
+
+class CxLoggerTest {
+
+    @Test
+    void testWarning_doesNotThrow() {
+        assertDoesNotThrow(() -> CxLogger.warning("test-warning-message"));
+    }
+
+    @Test
+    void testError_withException_doesNotThrow() {
+        assertDoesNotThrow(() -> CxLogger.error("test-error-message", new RuntimeException("test")));
+    }
+
+    @Test
+    void testInfo_doesNotThrow() {
+        assertDoesNotThrow(() -> CxLogger.info("test-info-message"));
+    }
+}