ItemUtils NPE bug fix

docs/src/progress/renderertodo.md

@@ -678,6 +678,7 @@ Include jenkins dockerfile in repo
 Better model for gameobjects
 Server synchronization of sprint tree
 Fix potential bad path for item state lookup
+Fix ItemUtils NPE bug + unit test
 
 
 # TODO

src/main/java/electrosphere/entity/types/item/ItemUtils.java

@@ -290,6 +290,9 @@ public class ItemUtils {
      * @return true if it is an item, false otherwise
      */
     public static boolean isItem(Entity e){
+        if(e == null){
+            return false;
+        }
         if(!e.containsKey(EntityDataStrings.ENTITY_TYPE)){
             return false;
         }

src/test/java/electrosphere/entity/types/item/ItemUtilsUnitTests.java

@@ -0,0 +1,20 @@
+package electrosphere.entity.types.item;
+
+import org.junit.jupiter.api.Assertions;
+
+import annotations.FastTest;
+import annotations.UnitTest;
+
+/**
+ * Unit tests for item utils
+ */
+public class ItemUtilsUnitTests {
+    
+    @UnitTest
+    @FastTest
+    public void isItem_NullEntity_False(){
+        boolean result = ItemUtils.isItem(null);
+        Assertions.assertEquals(false, result);
+    }
+
+}