additional tests

csviri · csviri · commit 27696a7584c9 · 2025-10-11T15:30:07.000+02:00
Signed-off-by: Attila Mészáros &lt;a_meszaros@apple.com&gt;
diff --git a/operator-framework-core/src/test/java/io/javaoperatorsdk/operator/api/reconciler/PrimaryUpdateAndCacheUtilsTest.java b/operator-framework-core/src/test/java/io/javaoperatorsdk/operator/api/reconciler/PrimaryUpdateAndCacheUtilsTest.java
@@ -197,6 +197,8 @@ public void compareResourceVersionsTest() {
     assertThrows(IllegalArgumentException.class, () -> compareResourceVersions("11", ""));
     assertThrows(IllegalArgumentException.class, () -> compareResourceVersions("01", "123"));
     assertThrows(IllegalArgumentException.class, () -> compareResourceVersions("123", "01"));
+    assertThrows(IllegalArgumentException.class, () -> compareResourceVersions("3213", "123a"));
+    assertThrows(IllegalArgumentException.class, () -> compareResourceVersions("321", "123a"));
   }
 
   // naive performance that compares the works case scenario for non parsing variant

Original file line number	Diff line number	Diff line change
`@@ -197,6 +197,8 @@ public void compareResourceVersionsTest() {`
`197`	`197`	`assertThrows(IllegalArgumentException.class, () -> compareResourceVersions("11", ""));`
`198`	`198`	`assertThrows(IllegalArgumentException.class, () -> compareResourceVersions("01", "123"));`
`199`	`199`	`assertThrows(IllegalArgumentException.class, () -> compareResourceVersions("123", "01"));`
	`200`	`+ assertThrows(IllegalArgumentException.class, () -> compareResourceVersions("3213", "123a"));`
	`201`	`+ assertThrows(IllegalArgumentException.class, () -> compareResourceVersions("321", "123a"));`
`200`	`202`	`}`
`201`	`203`
`202`	`204`	`// naive performance that compares the works case scenario for non parsing variant`