Add a test to verify some behavior in StringUtil, remove unnecessary geb dependencies

Author: phillipuniverse

common/pom.xml

@@ -67,6 +67,17 @@
                 <groupId>org.codehaus.gmavenplus</groupId>
                 <artifactId>gmavenplus-plugin</artifactId>
             </plugin>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-surefire-plugin</artifactId>
+                <configuration>
+                    <includes>
+                        <include>**/*Spec*</include>
+                        <include>**/*Test*</include>
+                    </includes>
+                    <argLine>${surefire.argLine}</argLine>
+                </configuration>
+            </plugin>
         </plugins>
         <resources>
             <resource>
@@ -377,24 +388,9 @@
             <scope>test</scope>
         </dependency>
         <dependency>
-            <groupId>org.gebish</groupId>
-            <artifactId>geb-core</artifactId>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.seleniumhq.selenium</groupId>
-            <artifactId>selenium-support</artifactId>
+            <groupId>org.spockframework</groupId>
+            <artifactId>spock-core</artifactId>
             <scope>test</scope>
         </dependency>
-        <dependency>
-            <groupId>org.seleniumhq.selenium</groupId>
-            <artifactId>selenium-firefox-driver</artifactId>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.seleniumhq.selenium</groupId>
-            <artifactId>selenium-chrome-driver</artifactId>
-            <scope>test</scope>
-        </dependency>      
     </dependencies>
 </project>

common/src/main/java/org/broadleafcommerce/common/util/StringUtil.java

@@ -20,6 +20,9 @@
 
 package org.broadleafcommerce.common.util;
 
+import org.apache.commons.lang.StringUtils;
+import org.codehaus.jettison.json.JSONObject;
+
 import java.io.ByteArrayInputStream;
 import java.io.IOException;
 import java.io.UnsupportedEncodingException;
@@ -30,9 +33,6 @@
 import java.util.zip.Adler32;
 import java.util.zip.CheckedInputStream;
 
-import org.apache.commons.lang.StringUtils;
-import org.codehaus.jettison.json.JSONObject;
-
 public class StringUtil {
 
     public static long getChecksum(String test) {
@@ -47,7 +47,7 @@ public static long getChecksum(String test) {
             throw new RuntimeException(e);
         }
     }
-
+    
     /**
      * Checks if a string is included in the beginning of another string, but only in dot-separated segment leaps.
      * Examples:

common/src/test/groovy/org/broadleafcommerce/common/test/util/StringUtilSpec.groovy

@@ -0,0 +1,49 @@
+/*
+ * #%L
+ * BroadleafCommerce Common Libraries
+ * %%
+ * Copyright (C) 2009 - 2016 Broadleaf Commerce
+ * %%
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * 
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * #L%
+ */
+/**
+ * 
+ */
+package org.broadleafcommerce.common.test.util
+
+import org.broadleafcommerce.common.util.StringUtil
+
+import spock.lang.Specification
+
+
+/**
+ * 
+ * 
+ * @author Phillip Verheyden (phillipuniverse)
+ */
+class StringUtilSpec extends Specification {
+
+    def "Exact roperty segments"() {
+        when: 
+            def result = StringUtil.segmentInclusion("fulfillmentLocation.address.phoneFax", "fulfillmentLocation.address.phone")
+        then:
+            !result
+        
+        when:
+            result = StringUtil.segmentInclusion("fulfillmentLocation.address.phoneFax.phoneNumber", "fulfillmentLocation.address.phoneFax")
+        then:
+            result
+    }
+    
+}