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

@@ -177,30 +177,14 @@ 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;
                     }
@@ -211,6 +195,9 @@ 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
@@ -221,6 +208,9 @@ 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
@@ -231,6 +221,9 @@ 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,6 +19,7 @@
 
 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;
@@ -32,7 +33,7 @@ import org.junit.jupiter.params.provider.ValueSource;
 import java.util.HashSet;
 import java.util.Set;
 
-import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
 
 /**
  * Tests the class {@link DependencyResolver}.
@@ -269,8 +270,6 @@ 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");
         }
     }
 
@@ -490,6 +489,6 @@ class DependencyResolverTest extends TestBase {
             return;
         }
 
-        assertEquals("Please ignore this, this just exists to trigger an error", "", "Dependency resolution succeeded (layers=" + layers + ")");
+        assertNotNull(Engine.getInstance(), "Dependency resolution succeeded (layers=" + layers + ")");
     }
 }