migrated del command to new format, and reverted back to old loot command which works fine

src/main/java/com/comandante/creeper/command/DelCommand.java

@@ -22,14 +22,8 @@ public class DelCommand extends Command {
 
     @Override
     public void messageReceived(ChannelHandlerContext ctx, MessageEvent e) throws Exception {
-        execCommand(ctx, e, new CommandRunnable() {
-            @Override
-            public void run() {
-
-            }
-        });
-        try {
-           if (originalMessageParts.size() <= 1) {
+        execCommand(ctx, e, () -> {
+            if (originalMessageParts.size() <= 1) {
                 write(returnAllSettings());
                 return;
             }
@@ -37,9 +31,7 @@ public class DelCommand extends Command {
             String desiredSettingName = originalMessageParts.get(0);
             player.removePlayerSetting(desiredSettingName);
             write("Setting removed.\r\n");
-        } finally {
-            super.messageReceived(ctx, e);
-        }
+        });
     }
 
     private String returnAllSettings() {

src/main/java/com/comandante/creeper/command/LootCommand.java

@@ -28,7 +28,7 @@ public class LootCommand extends Command {
     public void messageReceived(ChannelHandlerContext ctx, MessageEvent e) throws Exception {
         execCommand(ctx, e, () -> {
             if (originalMessageParts.size() > 1) {
-                player.getInventory().forEach(item -> {
+                for (Item item : player.getInventory()) {
                     if (item.getItemTypeId() == Item.CORPSE_ID_RESERVED) {
                         Loot loot = item.getLoot();
                         if (loot != null) {
@@ -38,18 +38,19 @@ public class LootCommand extends Command {
                                 player.incrementGold(gold);
                             }
                             Set<Item> items = lootManager.lootItemsReturn(loot);
-                            items.forEach(i -> {
+                            for (Item i: items) {
                                 gameManager.acquireItem(player, i.getItemId());
-                                write("You looted " + i.getItemName() + " from a " + item.getItemName() + ".\r\n");
-                            });
+                                write("You looted " + i.getItemName() +  " from a " + item.getItemName() + ".\r\n");
+                            }
                             if (gold < 0 && items.size() == 0) {
                                 write("You looted nothing from " + item.getItemName() + "\r\n");
                             }
                         }
                         player.removeInventoryId(item.getItemId());
                         entityManager.removeItem(item);
+                        return;
                     }
-                });
+                }
             }
         });
     }

src/test/com/comandante/creeper/player/NpcTestHarness.java

@@ -71,7 +71,7 @@ public class NpcTestHarness {
         int npcWins = 0;
 
         totalFightRounds = 0;
-        int totalIterations = 100000;
+        int totalIterations = 1000;
         for (int i = 0; i < totalIterations; i++) {
             String username = UUID.randomUUID().toString();
             Player player = createRandomPlayer(username);