Fix CommandHelper#getRawCommand() misbehaving with sub-commands (Fixes …

…#9), Add corresponding test to prevent this from happening again

common/pom.xml

@@ -16,5 +16,9 @@
             <groupId>net.cubespace</groupId>
             <artifactId>Yamler-Core</artifactId>
         </dependency>
+        <dependency>
+            <groupId>junit</groupId>
+            <artifactId>junit</artifactId>
+        </dependency>
     </dependencies>
 </project>

common/src/main/java/io/github/xxyy/cmdblocker/common/util/CommandHelper.java

@@ -24,7 +24,7 @@ public static String getRawCommand(String chatMessage) {
         if (spaceIndex == -1) { //If no space found
             chatMessage = chatMessage.substring(1); //Just remove slash
         } else { //If we have a space
-            chatMessage = chatMessage.substring(1, spaceIndex - 1); //Get the first word of the message and remove slash
+            chatMessage = chatMessage.substring(1, spaceIndex); //Get the first word of the message and remove slash
         }
 
         return removeModPrefix(chatMessage); //Return the raw command name!

common/src/test/java/io/github/xxyy/cmdblocker/common/util/CommandHelperTest.java

@@ -0,0 +1,24 @@
+package io.github.xxyy.cmdblocker.common.util;
+
+import org.junit.Test;
+
+import static org.hamcrest.CoreMatchers.is;
+import static org.junit.Assert.assertThat;
+
+/**
+ * Tests the CommandHelper class for compliance with the JavaDoc declarations
+ *
+ * @author <a href="http://xxyy.github.io/">xxyy</a>
+ * @since 20/09/14
+ */
+public class CommandHelperTest {
+    @Test
+    public void testGetRawCommand() {
+        assertThat(CommandHelper.getRawCommand("/help"), is("help"));
+        assertThat(CommandHelper.getRawCommand("/help test"), is("help"));
+        assertThat(CommandHelper.getRawCommand("/minecraft:help"), is("help"));
+        assertThat(CommandHelper.getRawCommand("/minecraft:help test"), is("help"));
+        assertThat(CommandHelper.getRawCommand("/bukkit:help"), is("help"));
+        assertThat(CommandHelper.getRawCommand("/bukkit:help something multiple arguments wow"), is("help"));
+    }
+}

pom.xml

@@ -122,6 +122,12 @@
                 <artifactId>Yamler-Core</artifactId>
                 <version>2.2.3-SNAPSHOT</version>
             </dependency>
+
+            <dependency>
+                <groupId>junit</groupId>
+                <artifactId>junit</artifactId>
+                <version>4.11</version>
+            </dependency>
         </dependencies>
     </dependencyManagement>