Fix assertions in DependencyResolverTest

base/src/main/java/de/staropensource/sosengine/base/utility/DependencyResolver.java

@@ -177,14 +177,30 @@ public final class DependencyResolver {
             for (char character : dependency.toCharArray()) {
                 switch (character) {
                     case '=' -> {
+                        // Check for duplicate
+                        if (duplicateCheck[0])
+                            throw new IllegalStateException("The dependency listing \"" + dependency + "\" cannot include multiple '=' characters");
+
+                        // Update mode
                         mode = 1;
+
                         continue;
                     }
                     case '<' -> {
+                        // Check for duplicate
+                        if (duplicateCheck[1])
+                            throw new IllegalStateException("The dependency listing \"" + dependency + "\" cannot include multiple '<' characters");
+
+                        // Update mode
                         mode = 2;
                         continue;
                     }
                     case '>' -> {
+                        // Check for duplicate
+                        if (duplicateCheck[2])
+                            throw new IllegalStateException("The dependency listing \"" + dependency + "\" cannot include multiple '>' characters");
+
+                        // Update mode
                         mode = 3;
                         continue;
                     }
@@ -195,9 +211,6 @@ public final class DependencyResolver {
                     case 0 -> identifier.append(character);
                     // Version equal
                     case 1 -> {
-                        // Check for duplicate
-                        if (character == '=' && duplicateCheck[0])
-                            throw new IllegalStateException("The dependency listing \"" + dependency + "\" cannot include multiple '=' characters");
                         duplicateCheck[0] = true;
 
                         // Check for smaller and bigger than
@@ -208,9 +221,6 @@ public final class DependencyResolver {
                     }
                     // Version smaller
                     case 2 -> {
-                        // Check for duplicate
-                        if (character == '<' && duplicateCheck[1])
-                            throw new IllegalStateException("The dependency listing \"" + dependency + "\" cannot include multiple '<' characters");
                         duplicateCheck[1] = true;
 
                         // Check for equal
@@ -221,9 +231,6 @@ public final class DependencyResolver {
                     }
                     // Version bigger
                     case 3 -> {
-                        // Check for duplicate
-                        if (character == '>' && duplicateCheck[2])
-                            throw new IllegalStateException("The dependency listing \"" + dependency + "\" cannot include multiple '>' characters");
                         duplicateCheck[2] = true;
 
                         // Check for equal

base/src/test/java/de/staropensource/sosengine/base/srctests/utility/DependencyResolverTest.java

@@ -19,7 +19,6 @@
 
 package de.staropensource.sosengine.base.srctests.utility;
 
-import de.staropensource.sosengine.base.Engine;
 import de.staropensource.sosengine.base.exception.dependency.UnmetDependenciesException;
 import de.staropensource.sosengine.base.implementation.versioning.OneNumberVersioningSystem;
 import de.staropensource.sosengine.base.srctests.TestBase;
@@ -33,7 +32,7 @@ import org.junit.jupiter.params.provider.ValueSource;
 import java.util.HashSet;
 import java.util.Set;
 
-import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 /**
  * Tests the class {@link DependencyResolver}.
@@ -270,6 +269,8 @@ class DependencyResolverTest extends TestBase {
             getLogger().error("Dependency resolution failed: Unmet dependencies found:");
             for (DependencyVector vector : exception.getUnmetDependencies().keySet())
                 getLogger().error("-> " + vector.getIdentifier() + "=" + vector.getVersion() + ": " + exception.getUnmetDependencies().get(vector));
+
+            assertEquals("Please ignore this, this just exists to trigger an error", "", "Dependency resolution failed (layers=" + layers + "): See logs");
         }
     }
 
@@ -489,6 +490,6 @@ class DependencyResolverTest extends TestBase {
             return;
         }
 
-        assertNotNull(Engine.getInstance(), "Dependency resolution succeeded (layers=" + layers + ")");
+        assertEquals("Please ignore this, this just exists to trigger an error", "", "Dependency resolution succeeded (layers=" + layers + ")");
     }
 }