polish

Author: Bilal Al

client/src/main/java/io/split/engine/experiments/SplitParser.java

@@ -19,7 +19,7 @@
 import io.split.engine.matchers.GreaterThanOrEqualToMatcher;
 import io.split.engine.matchers.LessThanOrEqualToMatcher;
 import io.split.engine.matchers.UserDefinedSegmentMatcher;
-import io.split.engine.matchers.EqualToMatcherSemver;
+import io.split.engine.matchers.EqualToSemverMatcher;
 import io.split.engine.matchers.collections.ContainsAllOfSetMatcher;
 import io.split.engine.matchers.collections.ContainsAnyOfSetMatcher;
 import io.split.engine.matchers.collections.EqualToSetMatcher;
@@ -198,7 +198,7 @@ private AttributeMatcher toMatcher(Matcher matcher) {
                 break;
             case EQUAL_TO_SEMVER:
                 checkNotNull(matcher.stringMatcherData);
-                delegate = new EqualToMatcherSemver(matcher.stringMatcherData);
+                delegate = new EqualToSemverMatcher(matcher.stringMatcherData);
                 break;
             case GREATER_THAN_OR_EQUAL_TO_SEMVER:
                 checkNotNull(matcher.stringMatcherData);

…ngine/matchers/EqualToMatcherSemver.java …ngine/matchers/EqualToSemverMatcher.javaclient/src/main/java/io/split/engine/matchers/EqualToMatcherSemver.java renamed to client/src/main/java/io/split/engine/matchers/EqualToSemverMatcher.java client/src/main/java/io/split/engine/matchers/EqualToMatcherSemver.java renamed to client/src/main/java/io/split/engine/matchers/EqualToSemverMatcher.java

@@ -4,11 +4,11 @@
 
 import java.util.Map;
 
-public class EqualToMatcherSemver implements Matcher {
+public class EqualToSemverMatcher implements Matcher {
 
     private final Semver _semVer;
 
-    public EqualToMatcherSemver(String semVer) {
+    public EqualToSemverMatcher(String semVer) {
         _semVer = Semver.build(semVer);
     }
 
@@ -44,9 +44,9 @@ public int hashCode() {
     public boolean equals(Object obj) {
         if (obj == null) return false;
         if (this == obj) return true;
-        if (!(obj instanceof EqualToMatcherSemver)) return false;
+        if (!(obj instanceof EqualToSemverMatcher)) return false;
 
-        EqualToMatcherSemver other = (EqualToMatcherSemver) obj;
+        EqualToSemverMatcher other = (EqualToSemverMatcher) obj;
 
         return _semVer == other._semVer;
     }

client/src/test/java/io/split/engine/matchers/EqualToSemverMatcherTest.java

@@ -0,0 +1,22 @@
+package io.split.engine.matchers;
+
+import org.junit.Test;
+
+import static org.junit.Assert.assertTrue;
+
+/**
+ * Tests for EqualToSemverMatcher
+ */
+
+public class EqualToSemverMatcherTest {
+
+    @Test
+    public void works() {
+        EqualToSemverMatcher equalToSemverMatcher = new EqualToSemverMatcher("2.1.8");
+
+        assertTrue( equalToSemverMatcher.match("2.1.8", null, null, null) == true);
+        assertTrue( equalToSemverMatcher.match("2.1.9", null, null, null) == false);
+        assertTrue( equalToSemverMatcher.match("2.1.8-rc", null, null, null) == false);
+        assertTrue( equalToSemverMatcher.match("2.1.8+build", null, null, null) == false);
+    }
+}

client/src/test/java/io/split/engine/matchers/EqualToSemverTest.java

@@ -8,7 +8,7 @@
  * Tests for EqualToSemverMatcher
  */
 
-public class GreaterThanOrEqualToSemverTest {
+public class GreaterThanOrEqualToSemverMatcherTest {
 
     @Test
     public void works() {