Semver fix

gthea · gthea · commit c65dbb968230 · 2026-04-21T15:06:00.000-03:00
diff --git a/client/src/test/java/io/split/engine/experiments/RuleBasedSegmentParserTest.java b/client/src/test/java/io/split/engine/experiments/RuleBasedSegmentParserTest.java
@@ -431,7 +431,7 @@ public void EqualToSemverMatcher() throws IOException {
                     assertTrue(parsedCondition.label().equals("equal to semver"));
                     for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {
                         // Check the matcher is ALL_KEYS
-                        assertTrue(matcher.matcher().toString().equals(" == semver 1\\.22\\.9"));
+                        assertTrue(matcher.matcher().toString().equals(" == semver 1.22.9"));
                         return;
                     }
                 }
@@ -453,7 +453,7 @@ public void GreaterThanOrEqualSemverMatcher() throws IOException {
                     assertTrue(parsedCondition.label().equals("greater than or equal to semver"));
                     for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {
                         // Check the matcher is ALL_KEYS
-                        assertTrue(matcher.matcher().toString().equals(" >= semver 1\\.22\\.9"));
+                        assertTrue(matcher.matcher().toString().equals(" >= semver 1.22.9"));
                         return;
                     }
                 }
@@ -475,7 +475,7 @@ public void LessThanOrEqualSemverMatcher() throws IOException {
                     assertTrue(parsedCondition.label().equals("less than or equal to semver"));
                     for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {
                         // Check the matcher is ALL_KEYS
-                        assertTrue(matcher.matcher().toString().equals(" <= semver 1\\.22\\.9"));
+                        assertTrue(matcher.matcher().toString().equals(" <= semver 1.22.9"));
                         return;
                     }
                 }
@@ -497,7 +497,7 @@ public void BetweenSemverMatcher() throws IOException {
                     assertTrue(parsedCondition.label().equals("between semver"));
                     for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {
                         // Check the matcher is ALL_KEYS
-                        assertTrue(matcher.matcher().toString().equals(" between semver 1\\.22\\.9 and 2\\.1\\.0"));
+                        assertTrue(matcher.matcher().toString().equals(" between semver 1.22.9 and 2.1.0"));
                         return;
                     }
                 }
diff --git a/client/src/test/java/io/split/engine/experiments/SplitParserTest.java b/client/src/test/java/io/split/engine/experiments/SplitParserTest.java
@@ -561,7 +561,7 @@ public void EqualToSemverMatcher() throws IOException {
                     assertTrue(parsedCondition.label().equals("equal to semver"));
                     for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {
                         // Check the matcher is ALL_KEYS
-                        assertTrue(matcher.matcher().toString().equals(" == semver 1\\.22\\.9"));
+                        assertTrue(matcher.matcher().toString().equals(" == semver 1.22.9"));
                         return;
                     }
                 }
@@ -583,7 +583,7 @@ public void GreaterThanOrEqualSemverMatcher() throws IOException {
                     assertTrue(parsedCondition.label().equals("greater than or equal to semver"));
                     for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {
                         // Check the matcher is ALL_KEYS
-                        assertTrue(matcher.matcher().toString().equals(" >= semver 1\\.22\\.9"));
+                        assertTrue(matcher.matcher().toString().equals(" >= semver 1.22.9"));
                         return;
                     }
                 }
@@ -605,7 +605,7 @@ public void LessThanOrEqualSemverMatcher() throws IOException {
                     assertTrue(parsedCondition.label().equals("less than or equal to semver"));
                     for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {
                         // Check the matcher is ALL_KEYS
-                        assertTrue(matcher.matcher().toString().equals(" <= semver 1\\.22\\.9"));
+                        assertTrue(matcher.matcher().toString().equals(" <= semver 1.22.9"));
                         return;
                     }
                 }
@@ -627,7 +627,7 @@ public void BetweenSemverMatcher() throws IOException {
                     assertTrue(parsedCondition.label().equals("between semver"));
                     for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {
                         // Check the matcher is ALL_KEYS
-                        assertTrue(matcher.matcher().toString().equals(" between semver 1\\.22\\.9 and 2\\.1\\.0"));
+                        assertTrue(matcher.matcher().toString().equals(" between semver 1.22.9 and 2.1.0"));
                         return;
                     }
                 }
diff --git a/client/src/test/java/io/split/engine/matchers/SemverTest.java b/client/src/test/java/io/split/engine/matchers/SemverTest.java
@@ -108,7 +108,7 @@ public void testCompareVersions() throws IOException {
     }
     @Test
     public void testLeadingZeros() {
-        assertTrue(Semver.build("1.01.2").version().equals("1\\.1\\.2"));
-        assertTrue(Semver.build("1.01.2-rc.01").version().equals("1\\.1\\.2-rc\\.1"));
+        assertTrue(Semver.build("1.01.2").version().equals("1.1.2"));
+        assertTrue(Semver.build("1.01.2-rc.01").version().equals("1.1.2-rc.1"));
     }
 }
diff --git a/tracker/src/test/java/io/split/client/events/EventsTaskTest.java b/tracker/src/test/java/io/split/client/events/EventsTaskTest.java
@@ -35,7 +35,7 @@ public void setup() {
     public void sendEventsDoesNothingWhenQueueEmpty() {
         when(_storage.popAll()).thenReturn(Collections.<WrappedEvent>emptyList());
         _task.sendEvents();
-        verifyZeroInteractions(_sender);
+        verifyNoInteractions(_sender);
     }
 
     @Test
diff --git a/tracker/src/test/java/io/split/client/events/InMemoryEventsStorageTest.java b/tracker/src/test/java/io/split/client/events/InMemoryEventsStorageTest.java
@@ -31,7 +31,7 @@ public void trackReturnsTrue_andNotifiesQueued() {
     @Test
     public void trackNullEventReturnsFalse() {
         assertFalse(_storage.track(null, 0));
-        verifyZeroInteractions(_stats);
+        verifyNoInteractions(_stats);
     }
 
     @Test

Original file line number	Diff line number	Diff line change
`@@ -431,7 +431,7 @@ public void EqualToSemverMatcher() throws IOException {`
`431`	`431`	`assertTrue(parsedCondition.label().equals("equal to semver"));`
`432`	`432`	`for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {`
`433`	`433`	`// Check the matcher is ALL_KEYS`
`434`		`- assertTrue(matcher.matcher().toString().equals(" == semver 1\\.22\\.9"));`
	`434`	`+ assertTrue(matcher.matcher().toString().equals(" == semver 1.22.9"));`
`435`	`435`	`return;`
`436`	`436`	`}`
`437`	`437`	`}`
`@@ -453,7 +453,7 @@ public void GreaterThanOrEqualSemverMatcher() throws IOException {`
`453`	`453`	`assertTrue(parsedCondition.label().equals("greater than or equal to semver"));`
`454`	`454`	`for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {`
`455`	`455`	`// Check the matcher is ALL_KEYS`
`456`		`- assertTrue(matcher.matcher().toString().equals(" >= semver 1\\.22\\.9"));`
	`456`	`+ assertTrue(matcher.matcher().toString().equals(" >= semver 1.22.9"));`
`457`	`457`	`return;`
`458`	`458`	`}`
`459`	`459`	`}`
`@@ -475,7 +475,7 @@ public void LessThanOrEqualSemverMatcher() throws IOException {`
`475`	`475`	`assertTrue(parsedCondition.label().equals("less than or equal to semver"));`
`476`	`476`	`for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {`
`477`	`477`	`// Check the matcher is ALL_KEYS`
`478`		`- assertTrue(matcher.matcher().toString().equals(" <= semver 1\\.22\\.9"));`
	`478`	`+ assertTrue(matcher.matcher().toString().equals(" <= semver 1.22.9"));`
`479`	`479`	`return;`
`480`	`480`	`}`
`481`	`481`	`}`
`@@ -497,7 +497,7 @@ public void BetweenSemverMatcher() throws IOException {`
`497`	`497`	`assertTrue(parsedCondition.label().equals("between semver"));`
`498`	`498`	`for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {`
`499`	`499`	`// Check the matcher is ALL_KEYS`
`500`		`- assertTrue(matcher.matcher().toString().equals(" between semver 1\\.22\\.9 and 2\\.1\\.0"));`
	`500`	`+ assertTrue(matcher.matcher().toString().equals(" between semver 1.22.9 and 2.1.0"));`
`501`	`501`	`return;`
`502`	`502`	`}`
`503`	`503`	`}`
Original file line number	Diff line number	Diff line change
`@@ -561,7 +561,7 @@ public void EqualToSemverMatcher() throws IOException {`
`561`	`561`	`assertTrue(parsedCondition.label().equals("equal to semver"));`
`562`	`562`	`for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {`
`563`	`563`	`// Check the matcher is ALL_KEYS`
`564`		`- assertTrue(matcher.matcher().toString().equals(" == semver 1\\.22\\.9"));`
	`564`	`+ assertTrue(matcher.matcher().toString().equals(" == semver 1.22.9"));`
`565`	`565`	`return;`
`566`	`566`	`}`
`567`	`567`	`}`
`@@ -583,7 +583,7 @@ public void GreaterThanOrEqualSemverMatcher() throws IOException {`
`583`	`583`	`assertTrue(parsedCondition.label().equals("greater than or equal to semver"));`
`584`	`584`	`for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {`
`585`	`585`	`// Check the matcher is ALL_KEYS`
`586`		`- assertTrue(matcher.matcher().toString().equals(" >= semver 1\\.22\\.9"));`
	`586`	`+ assertTrue(matcher.matcher().toString().equals(" >= semver 1.22.9"));`
`587`	`587`	`return;`
`588`	`588`	`}`
`589`	`589`	`}`
`@@ -605,7 +605,7 @@ public void LessThanOrEqualSemverMatcher() throws IOException {`
`605`	`605`	`assertTrue(parsedCondition.label().equals("less than or equal to semver"));`
`606`	`606`	`for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {`
`607`	`607`	`// Check the matcher is ALL_KEYS`
`608`		`- assertTrue(matcher.matcher().toString().equals(" <= semver 1\\.22\\.9"));`
	`608`	`+ assertTrue(matcher.matcher().toString().equals(" <= semver 1.22.9"));`
`609`	`609`	`return;`
`610`	`610`	`}`
`611`	`611`	`}`
`@@ -627,7 +627,7 @@ public void BetweenSemverMatcher() throws IOException {`
`627`	`627`	`assertTrue(parsedCondition.label().equals("between semver"));`
`628`	`628`	`for (AttributeMatcher matcher : parsedCondition.matcher().attributeMatchers()) {`
`629`	`629`	`// Check the matcher is ALL_KEYS`
`630`		`- assertTrue(matcher.matcher().toString().equals(" between semver 1\\.22\\.9 and 2\\.1\\.0"));`
	`630`	`+ assertTrue(matcher.matcher().toString().equals(" between semver 1.22.9 and 2.1.0"));`
`631`	`631`	`return;`
`632`	`632`	`}`
`633`	`633`	`}`
Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,7 @@ public void testCompareVersions() throws IOException {`
`108`	`108`	`}`
`109`	`109`	`@Test`
`110`	`110`	`public void testLeadingZeros() {`
`111`		`- assertTrue(Semver.build("1.01.2").version().equals("1\\.1\\.2"));`
`112`		`- assertTrue(Semver.build("1.01.2-rc.01").version().equals("1\\.1\\.2-rc\\.1"));`
	`111`	`+ assertTrue(Semver.build("1.01.2").version().equals("1.1.2"));`
	`112`	`+ assertTrue(Semver.build("1.01.2-rc.01").version().equals("1.1.2-rc.1"));`
`113`	`113`	`}`
`114`	`114`	`}`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ public void setup() {`
`35`	`35`	`public void sendEventsDoesNothingWhenQueueEmpty() {`
`36`	`36`	`when(_storage.popAll()).thenReturn(Collections.<WrappedEvent>emptyList());`
`37`	`37`	`_task.sendEvents();`
`38`		`- verifyZeroInteractions(_sender);`
	`38`	`+ verifyNoInteractions(_sender);`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`@Test`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ public void trackReturnsTrue_andNotifiesQueued() {`
`31`	`31`	`@Test`
`32`	`32`	`public void trackNullEventReturnsFalse() {`
`33`	`33`	`assertFalse(_storage.track(null, 0));`
`34`		`- verifyZeroInteractions(_stats);`
	`34`	`+ verifyNoInteractions(_stats);`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`@Test`