Citizen NPCs are not island visitors so do not protect them.

BentoBoxWorld#1518

Author: tastybento

src/main/java/world/bentobox/bentobox/listeners/flags/worldsettings/InvincibleVisitorsListener.java

@@ -103,6 +103,7 @@ public void onVisitorGetDamage(EntityDamageEvent e) {
         World world = e.getEntity().getWorld();
         if (!(e.getEntity() instanceof Player)
                 || !getIWM().inWorld(world)
+                || e.getEntity().hasMetadata("NPC")
                 || !getIWM().getIvSettings(world).contains(e.getCause().name())
                 || getIslands().userIsOnIsland(world, User.getInstance(e.getEntity()))) {
             return;

src/test/java/world/bentobox/bentobox/listeners/flags/worldsettings/InvincibleVisitorsListenerTest.java

@@ -255,6 +255,14 @@ public void testOnVisitorGetDamageNotVoid() {
         assertTrue(e.isCancelled());
         verify(player, never()).setGameMode(eq(GameMode.SPECTATOR));
     }
+    
+    @Test
+    public void testOnVisitorGetDamageNPC() {
+        when(player.hasMetadata(eq("NPC"))).thenReturn(true);
+        EntityDamageEvent e = new EntityDamageEvent(player, EntityDamageEvent.DamageCause.CRAMMING, 0D);
+        listener.onVisitorGetDamage(e);
+        assertFalse(e.isCancelled());
+    }
 
 
     @Test