adding test cases to all classes

Author: rofidakhaled

entity-component-system/src/main/java/com/iluwatar/App.java

@@ -24,12 +24,18 @@
  */
 package com.iluwatar;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 /**
  * The main entry point for the application.
  * This class simulates a game loop where entities are created, updated, and their states are modified.
  */
 public class App {
 
+  // Create a logger instance
+  public static final Logger logger = LoggerFactory.getLogger(App.class);
+
   /**
    * The main method that runs the application.
    * It creates entities, adds components, updates them over several frames,
@@ -38,7 +44,7 @@ public class App {
    * @param args Command-line arguments (not used in this application)
    */
   public static void main(String[] args) {
-    // Create example entities
+
     Entity entity1 = new Entity("Entity1");
     Entity entity2 = new Entity("Entity2");
 
@@ -70,35 +76,36 @@ public static void main(String[] args) {
 
     // Simulate for a few frames
     for (int i = 0; i < 10; i++) {
-      System.out.println("Frame: " + (i + 1));
+      logger.info("Frame: {}", i + 1);
 
       // Update all entities in the system
       gameSystem.update(deltaTime);
 
       // Apply some damage to entity1's health component at frame 6
       if (i == 5) {
         health1.applyDamage(30);
-        System.out.println("Entity1's health after damage: " + health1.getCurrentHealth());
+        logger.info("Entity1's health after damage: {}", health1.getCurrentHealth());
       }
 
       // Apply some force to entity1's velocity at frame 3
       if (i == 3) {
         velocity1.applyForce(0.5f, 0.0f, 0.0f);
-        System.out.println("Entity1's velocity after force: (" + velocity1.getVelocityX() + ", "
-            + velocity1.getVelocityY() + ", " + velocity1.getVelocityZ() + ")");
+        logger.info("Entity1's velocity after force: ({}, {}, {})", velocity1.getVelocityX(),
+            velocity1.getVelocityY(), velocity1.getVelocityZ());
       }
 
       // Render the system (optional rendering logic can be added here)
       gameSystem.renderSystem();
     }
 
     // After the simulation, check final entity states
-    System.out.println("\nFinal Entity States:");
-    System.out.println("Entity1 position: " + entity1.getTransformComponent().getPosition()[0] + ", "
-        + entity1.getTransformComponent().getPosition()[1] + ", "
-        + entity1.getTransformComponent().getPosition()[2]);
-    System.out.println("Entity1 velocity: " + velocity1.getVelocityX() + ", "
-        + velocity1.getVelocityY() + ", " + velocity1.getVelocityZ());
-    System.out.println("Entity1 health: " + health1.getCurrentHealth());
+    logger.info("\nFinal Entity States:");
+    logger.info("Entity1 position: {}, {}, {}",
+        entity1.getTransformComponent().getPosition()[0],
+        entity1.getTransformComponent().getPosition()[1],
+        entity1.getTransformComponent().getPosition()[2]);
+    logger.info("Entity1 velocity: {}, {}, {}",
+        velocity1.getVelocityX(), velocity1.getVelocityY(), velocity1.getVelocityZ());
+    logger.info("Entity1 health: {}", health1.getCurrentHealth());
   }
 }

entity-component-system/src/main/java/com/iluwatar/Entity.java

@@ -26,6 +26,7 @@
 
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Objects;
 import java.util.UUID;
 
 /**
@@ -58,6 +59,7 @@ public Entity(String entityName) {
     final UUID entityId = UUID.randomUUID(); // Generates a unique UUID for this entity
     transform = new TransformComponent(new float[] {0.0f, 0.0f, 0.0f}, new float[] {0.0f, 0.0f, 0.0f}, new float[] {1.0f, 1.0f, 1.0f});
     addComponent(transform);
+
   }
 
   /**
@@ -90,7 +92,7 @@ public void removeComponent(Component component) {
    */
   public Component getComponent(String componentName) {
     for (Component component : components) {
-      if (component.getName().equals(componentName)) {
+      if (Objects.equals(component.getName(), componentName)) {
         return component;
       }
     }
@@ -204,6 +206,7 @@ public void setTransformComponent(TransformComponent transform) {
    * @return a list of mesh render components
    */
   public List<Component> getComponents() {
+
     return components;
   }
 

entity-component-system/src/main/java/com/iluwatar/HealthComponent.java

@@ -59,7 +59,7 @@ public HealthComponent(float maxHealth) {
    */
   @Override
   public void update(float deltaTime) {
-    logger.info(String.format("Updated health component: %f", currentHealth));
+    //update health
   }
 
   /**

entity-component-system/src/test/java/com/iluwatar/ComponentTest.java

@@ -29,6 +29,21 @@
 import static org.junit.jupiter.api.Assertions.*;
 
 public class ComponentTest {
+  @Test
+  public void testGetName() {
+    Component component = new HealthComponent(100);
+    component.setName("Health");
+
+    assertEquals("Health", component.getName(), "getName should return 'TestComponent'");
+  }
+
+  @Test
+  public void testSetName() {
+    Component component = new HealthComponent(100);
+    component.setName("Health");
+
+    assertEquals("Health", component.getName(), "getName should return 'TestComponent'");
+  }
 
   @Test
   public void testComponentEnabled() {

entity-component-system/src/test/java/com/iluwatar/EntityTest.java

@@ -65,27 +65,79 @@ public void testRemoveComponent() {
     assertEquals(1, entity2.getComponents().size(), "Entity1 should have no components.");
   }
 
+  /*
   @Test
   public void testSetParent() {
     entity1 = new Entity("Entity1");
     entity2 = new Entity("Entity2");
     entity1.setParent(entity2);
     assertEquals(entity2.getEntityId(), entity1.getParent().getEntityId(), "Entity1 should have Entity2 as its parent.");
   }
-
+*/
   @Test
   public void testAddChild() {
     entity1.addChild(entity2);
 
     assertTrue(entity1.getChildren().contains(entity2), "Entity1 should have Entity2 as its child.");
   }
-
+/*
   @Test
   public void testSetEnabled() {
     entity1.setEnabled(false);
     assertFalse(entity1.getEnabled(), "Entity1 should be disabled.");
     entity1.setEnabled(true);
     assertTrue(entity1.getEnabled(), "Entity1 should be enabled.");
+  } */
+
+  @Test
+  public void testGetComponent() {
+
+    Component component = new HealthComponent(100);
+    component.setName("HealthComponent");
+    Entity entity = new Entity("Entity1");
+    entity.addComponent(component);
+    Component retrievedComponent = entity.getComponent("HealthComponent");
+
+    assertEquals(component, retrievedComponent, "The component returned should match the added component.");
+    
+    Component nonExistentComponent = entity.getComponent("NonExistentComponent");
+    assertNull(nonExistentComponent, "The component should return null if it doesn't exist.");
+  }
+
+  @Test
+  public void testSetEnabled() {
+
+    Component component = new HealthComponent(100);
+    Entity entity = new Entity("MyEntity");
+    entity.addComponent(component);
+
+    assertFalse(component.getEnabled(), "Component should be disabled initially.");
+
+    entity.setEnabled(false);
+    
+    assertFalse(component.getEnabled(), "Component should be disabled after calling setEnabled(false).");
+
+    entity.setEnabled(true);
+    
+    assertTrue(component.getEnabled(), "Component should be enabled after calling setEnabled(true).");
+  }
+
+  @Test
+  public void testSetParent() {
+
+    Entity parent = new Entity("parent");
+    Entity child = new Entity("child");
+
+    child.setParent(parent);
+    
+    assertTrue(parent.getChildren().contains(child), "Parent should contain child in its children list.");
+    
+    Entity newParent = new Entity("newParent");
+    child.setParent(newParent);
+
+    assertFalse(parent.getChildren().contains(child), "Parent should no longer contain child after setting a new parent.");
+
+    assertTrue(newParent.getChildren().contains(child), "New parent should contain child in its children list.");
   }
 
   @Test
@@ -95,4 +147,45 @@ public void testUpdate() {
 
     assertNotNull(entity1, "Entity1 should be updated.");
   }
+
+  @Test
+  public void testRenderEntity() {
+
+    Component component = new HealthComponent(100);
+    Entity entity = new Entity("MyEntity");
+    entity.addComponent(component);
+    entity.renderEntity();
+
+    assertDoesNotThrow(() -> component.update(1.0f), "render function should not throw an exception");
+  }
+
+  @Test
+  public void testGetName() {
+    Entity entity = new Entity("MyEntity");
+
+    assertEquals("MyEntity", entity.getName(), "The entity name should match the given name.");
+  }
+
+  @Test
+  public void testSetIsEnabled() {
+    Entity entity = new Entity("MyEntity");
+
+    entity.setIsEnabled(false);
+
+    assertFalse(entity.isEnabled(), "The entity should be disabled after calling setIsEnabled(false).");
+    
+    entity.setIsEnabled(true);
+    
+    assertTrue(entity.isEnabled(), "The entity should be enabled after calling setIsEnabled(true).");
+  }
+
+  @Test
+  public void testGetAndSetGameSystem() {
+ 
+    GameSystem gameSystem = new GameSystem();
+    Entity entity = new Entity("MyEntity");
+    entity.setGameSystem(gameSystem);
+    
+    assertEquals(gameSystem, entity.getGameSystem(), "The game system should match the one set.");
+  }
 }

entity-component-system/src/test/java/com/iluwatar/HealthComponentTest.java

@@ -60,7 +60,55 @@ public void testHeal() {
     healthComponent.heal(50);
     assertEquals(100, healthComponent.getCurrentHealth(), "Health should not exceed maximum (100).");
   }
+  @Test
+  public void testUpdateFunction() {
+     healthComponent = new HealthComponent(100);
+
+    assertDoesNotThrow(() -> healthComponent.update(1.0f), "update function should not throw an exception");
+  }
+
+  @Test
+  public void testGetMaxHealth() {
+    HealthComponent healthComponent = new HealthComponent(100f);
+
+    assertEquals(100f, healthComponent.getMaxHealth(), "Max health should be 100.");
+  }
+
+  @Test
+  public void testSetMaxHealth() {
+    HealthComponent healthComponent = new HealthComponent(100f);
+    healthComponent.setMaxHealth(120f);
+
+    assertEquals(120f, healthComponent.getMaxHealth(), "Max health should be updated to 120.");
+  }
+
+  @Test
+  public void testIsAlive() {
+
+    HealthComponent healthComponent = new HealthComponent(100f);
+    assertTrue(healthComponent.isAlive(), "Entity should be alive initially.");
+    healthComponent.applyDamage(100f);
+
+    assertFalse(healthComponent.isAlive(), "Entity should be dead after taking 100 damage.");
+  }
+
+  @Test
+  public void testSetAlive() {
 
+    HealthComponent healthComponent = new HealthComponent(100f);
+    assertTrue(healthComponent.isAlive(), "Entity should be alive initially.");
+    healthComponent.setAlive(false);
+
+    assertFalse(healthComponent.isAlive(), "Entity should be dead after setting alive to false.");
+  }
+
+  @Test
+  public void testSetCurrentHealth() {
+    HealthComponent healthComponent = new HealthComponent(100f);
+    healthComponent.setCurrentHealth(80f);
+
+    assertEquals(80f, healthComponent.getCurrentHealth(), "Current health should be updated to 80.");
+  }
   @Test
   public void testHealthCannotGoAboveMax() {
     healthComponent.heal(150);

entity-component-system/src/test/java/com/iluwatar/TransformComponentTest.java

@@ -39,6 +39,20 @@ public void setUp() {
         new float[]{0.0f, 0.0f, 0.0f}, new float[]{1.0f, 1.0f, 1.0f});
   }
 
+
+  @Test
+  public void testTransformComponentConstructor() {
+    TransformComponent transform = new TransformComponent();
+
+    float[] expectedPosition = new float[]{0.0f, 0.0f, 0.0f};
+    float[] expectedRotation = new float[]{0.0f, 0.0f, 0.0f};
+    float[] expectedScale = new float[]{1.0f, 1.0f, 1.0f};
+
+    assertArrayEquals(expectedPosition, transform.getPosition(), "Position should be initialized to [0.0f, 0.0f, 0.0f]");
+    assertArrayEquals(expectedRotation, transform.getRotation(), "Rotation should be initialized to [0.0f, 0.0f, 0.0f]");
+    assertArrayEquals(expectedScale, transform.getScale(), "Scale should be initialized to [1.0f, 1.0f, 1.0f]");
+  }
+
   @Test
   public void testGetPosition() {
     float[] position = transform.getPosition();
@@ -74,6 +88,13 @@ public void testGetScale() {
     assertArrayEquals(new float[]{1.0f, 1.0f, 1.0f}, scale, "The scale should be initialized correctly.");
   }
 
+  @Test
+  public void testUpdateFunction() {
+    TransformComponent transform = new TransformComponent();
+
+    assertDoesNotThrow(() -> transform.update(1.0f), "update function should not throw an exception");
+  }
+
   @Test
   public void testSetScale() {
     transform.setScale(new float[]{2.0f, 2.0f, 2.0f});

entity-component-system/src/test/java/com/iluwatar/VelocityComponentTest.java

@@ -61,4 +61,25 @@ public void testUpdateVelocity() {
 
     assertTrue(velocity.getVelocityX() > initialVelocityX, "VelocityX should increase after updating.");
   }
+
+  @Test
+  public void testSetVelocityX() {
+    
+    velocity = new VelocityComponent(1.0f, 0.0f, 0.0f);
+    velocity.setVelocityX(5.0f);
+    assertEquals(5.0f, velocity.getVelocityX(), "The velocityX should be set to 5.0f");
+    
+    velocity.setVelocityX(10.0f);
+    assertEquals(10.0f, velocity.getVelocityX(), "The velocityX should now be set to 10.0f");
+  }
+  @Test
+  public void testSetVelocityZ() {
+
+    velocity = new VelocityComponent(1.0f, 0.0f, 0.0f);
+    velocity.setVelocityZ(5.0f);
+    assertEquals(5.0f, velocity.getVelocityZ(), "The velocityZ should be set to 5.0f");
+
+    velocity.setVelocityZ(10.0f);
+    assertEquals(10.0f, velocity.getVelocityZ(), "The velocityZ should now be set to 10.0f");
+  }
 }