Fix some velocity related issues

cloud-core/src/main/java/com/intellectualsites/commands/CommandManager.java

@@ -199,7 +199,8 @@ public abstract class CommandManager<C> {
     }
 
     /**
-     * Check if the command sender has the required permission
+     * Check if the command sender has the required permission. If the permission node is
+     * empty, this should return {@code true}
      *
      * @param sender     Command sender
      * @param permission Permission node

cloud-minecraft/cloud-bukkit-test/src/main/java/com/intellectualsites/commands/BukkitTest.java

@@ -123,14 +123,6 @@ public final class BukkitTest extends JavaPlugin {
                                ));
                            })
                            .build())
-               .command(mgr.commandBuilder("test")
-                           .literal("one")
-                           .handler(c -> c.getSender().sendMessage("One!"))
-                           .build())
-               .command(mgr.commandBuilder("test")
-                           .literal("two")
-                           .handler(c -> c.getSender().sendMessage("Two!"))
-                           .build())
                .command(mgr.commandBuilder("uuidtest")
                            .argument(UUID.class, "uuid", builder -> builder
                                    .asRequired()

cloud-minecraft/cloud-bukkit/src/main/java/com/intellectualsites/commands/bukkit/BukkitCommandManager.java

@@ -146,6 +146,9 @@ public class BukkitCommandManager<C> extends CommandManager<C> {
 
     @Override
     public final boolean hasPermission(@Nonnull final C sender, @Nonnull final String permission) {
+        if (permission.isEmpty()) {
+            return true;
+        }
         return this.backwardsCommandSenderMapper.apply(sender).hasPermission(permission);
     }
 

cloud-minecraft/cloud-bungee/src/main/java/com/intellectualsites/commands/bungee/BungeeCommandManager.java

@@ -64,6 +64,9 @@ public class BungeeCommandManager<C> extends CommandManager<C> {
     @Override
     public final boolean hasPermission(@Nonnull final C sender,
                                        @Nonnull final String permission) {
+        if (permission.isEmpty()) {
+            return true;
+        }
         return this.backwardsCommandSenderMapper.apply(sender).hasPermission(permission);
     }
 

cloud-minecraft/cloud-velocity-test/src/main/java/com/intellectualsites/cloudvelocitytest/CloudVelocityTest.java

@@ -79,14 +79,14 @@ public class CloudVelocityTest {
     }
 
     @CommandMethod("test <num> [str]")
-    private void testCommand(@Nonnull @Argument("str") final String string,
+    private void testCommand(@Nonnull @Argument(value = "str", defaultValue = "potato") final String string,
                              @Nonnull final CommandSource source,
                              @Argument("num") @Range(min = "10", max = "33") final int num) {
         source.sendMessage(TextComponent.builder()
             .append("You wrote: ", NamedTextColor.GOLD)
-            .append(string, NamedTextColor.LIGHT_PURPLE)
+            .append(string, NamedTextColor.RED)
             .append(" and ", NamedTextColor.GOLD)
-            .append(Integer.toString(num), NamedTextColor.LIGHT_PURPLE)
+            .append(Integer.toString(num), NamedTextColor.RED)
             .append("!", NamedTextColor.GOLD)
         );
     }

cloud-minecraft/cloud-velocity/src/main/java/com/intellectualsites/commands/velocity/VelocityCommandManager.java

@@ -67,6 +67,9 @@ public class VelocityCommandManager<C> extends CommandManager<C> {
 
     @Override
     public final boolean hasPermission(@Nonnull final C sender, @Nonnull final String permission) {
+        if (permission.isEmpty()) {
+            return true;
+        }
         return this.backwardsCommandSenderMapper.apply(sender).hasPermission(permission);
     }