fix quality (coverage)

Author: HabibaMekay

table-inheritance/src/main/java/com/iluwatar/Main.java renamed to table-inheritance/src/main/java/com/iluwatar/table/inheritance/App.java

@@ -1,10 +1,9 @@
-package com.iluwatar;
+package com.iluwatar.table.inheritance;
 
 /**
  * The main entry point of the application demonstrating the use of vehicles.
  */
-public class Main {
-
+public class App {
   /**
    * The main method to demonstrate adding and retrieving vehicles from the database.
    *

table-inheritance/src/main/java/com/iluwatar/Car.java renamed to table-inheritance/src/main/java/com/iluwatar/table/inheritance/Car.java

@@ -1,4 +1,4 @@
-package com.iluwatar;
+package com.iluwatar.table.inheritance;
 
 /**
  * Represents a car with a specific number of doors.
@@ -17,6 +17,9 @@ public class Car extends Vehicle {
    */
   public Car(int year, String make, String model, int numDoors, int id) {
     super(year, make, model, id);
+    if (numDoors <= 0) {
+      throw new IllegalArgumentException("Number of doors must be positive.");
+    }
     this.numDoors = numDoors;
   }
 
@@ -26,6 +29,9 @@ public Car(int year, String make, String model, int numDoors, int id) {
    * @param doors the number of doors
    */
   public void setNumDoors(int doors) {
+    if (doors <= 0) {
+      throw new IllegalArgumentException("Number of doors must be positive.");
+    }
     this.numDoors = doors;
   }
 
@@ -49,4 +55,3 @@ public String toString() {
         + '}';
   }
 }
-

table-inheritance/src/main/java/com/iluwatar/Truck.java renamed to table-inheritance/src/main/java/com/iluwatar/table/inheritance/Truck.java

@@ -1,4 +1,4 @@
-package com.iluwatar;
+package com.iluwatar.table.inheritance;
 
 /**
  * Represents a truck, a type of vehicle with a specific load capacity.
@@ -17,7 +17,10 @@ public class Truck extends Vehicle {
    */
   public Truck(int year, String make, String model, double loadCapacity, int id) {
     super(year, make, model, id);
-    this.loadCapacity = loadCapacity; // Added spaces around '='
+    if (loadCapacity <= 0) {
+      throw new IllegalArgumentException("Load capacity must be positive.");
+    }
+    this.loadCapacity = loadCapacity;
   }
 
   /**
@@ -26,7 +29,10 @@ public Truck(int year, String make, String model, double loadCapacity, int id) {
    * @param capacity the new load capacity
    */
   public void setLoadCapacity(double capacity) {
-    this.loadCapacity = capacity; // Added spaces around '='
+    if (capacity <= 0) {
+      throw new IllegalArgumentException("Load capacity must be positive.");
+    }
+    this.loadCapacity = capacity;
   }
 
   /**
@@ -54,3 +60,4 @@ public String toString() {
         + '}';
   }
 }
+

table-inheritance/src/main/java/com/iluwatar/Vehicle.java renamed to table-inheritance/src/main/java/com/iluwatar/table/inheritance/Vehicle.java

@@ -1,4 +1,4 @@
-package com.iluwatar;
+package com.iluwatar.table.inheritance;
 
 /**
  * Represents a generic vehicle with basic attributes like make, model, year, and ID.

table-inheritance/src/main/java/com/iluwatar/VehicleDatabase.java renamed to table-inheritance/src/main/java/com/iluwatar/table/inheritance/VehicleDatabase.java

@@ -1,4 +1,4 @@
-package com.iluwatar;
+package com.iluwatar.table.inheritance;
 
 import java.util.HashMap;
 import java.util.Map;

table-inheritance/src/test/java/VehicleDatabaseTest.java

@@ -1,18 +1,19 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertThrows;
 
-import com.iluwatar.Car;
-import com.iluwatar.Truck;
-import com.iluwatar.Vehicle;
-import com.iluwatar.VehicleDatabase;
+import com.iluwatar.table.inheritance.Car;
+import com.iluwatar.table.inheritance.Truck;
+import com.iluwatar.table.inheritance.Vehicle;
+import com.iluwatar.table.inheritance.VehicleDatabase;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
 /**
  * Unit tests for the {@link VehicleDatabase} class.
  * Tests saving, retrieving, and printing vehicles of different types.
  */
-public class VehicleDatabaseTest {
+class VehicleDatabaseTest {
 
   private VehicleDatabase vehicleDatabase;
 
@@ -28,7 +29,7 @@ public void setUp() {
    * Tests saving a {@link Car} to the database and retrieving it.
    */
   @Test
-  public void testSaveAndRetrieveCar() {
+  void testSaveAndRetrieveCar() {
     Car car = new Car(2020, "Toyota", "Corolla", 4, 1);
     vehicleDatabase.saveVehicle(car);
 
@@ -48,7 +49,7 @@ public void testSaveAndRetrieveCar() {
    * Tests saving a {@link Truck} to the database and retrieving it.
    */
   @Test
-  public void testSaveAndRetrieveTruck() {
+  void testSaveAndRetrieveTruck() {
     Truck truck = new Truck(2018, "Ford", "F-150", 60, 2);
     vehicleDatabase.saveVehicle(truck);
 
@@ -68,7 +69,7 @@ public void testSaveAndRetrieveTruck() {
    * Tests saving multiple vehicles to the database and printing them.
    */
   @Test
-  public void testPrintAllVehicles() {
+  void testPrintAllVehicles() {
     Car car = new Car(2020, "Toyota", "Corolla", 4, 1);
     Truck truck = new Truck(2018, "Ford", "F-150", 60, 2);
     vehicleDatabase.saveVehicle(car);
@@ -82,4 +83,87 @@ public void testPrintAllVehicles() {
     assertNotNull(retrievedCar);
     assertNotNull(retrievedTruck);
   }
+
+  /**
+   * Tests the constructor of {@link Car} with valid values.
+   */
+  @Test
+  void testCarConstructor() {
+    Car car = new Car(2020, "Toyota", "Corolla", 4, 1);
+    assertEquals(2020, car.getYear());
+    assertEquals("Toyota", car.getMake());
+    assertEquals("Corolla", car.getModel());
+    assertEquals(4, car.getNumDoors());
+    assertEquals(1, car.getId()); // Assuming the ID is auto-generated in the constructor
+  }
+
+  /**
+   * Tests the constructor of {@link Car} with invalid number of doors (negative value).
+   */
+  @Test
+  void testCarConstructorWithInvalidNumDoors() {
+    IllegalArgumentException exception = assertThrows(IllegalArgumentException.class, () -> {
+      new Car(2020, "Toyota", "Corolla", -4, 1);
+    });
+    assertEquals("Number of doors must be positive.", exception.getMessage());
+  }
+
+  /**
+   * Tests the constructor of {@link Car} with zero doors.
+   */
+  @Test
+  void testCarConstructorWithZeroDoors() {
+    IllegalArgumentException exception = assertThrows(IllegalArgumentException.class, () -> {
+      new Car(2020, "Toyota", "Corolla", 0, 1);
+    });
+    assertEquals("Number of doors must be positive.", exception.getMessage());
+  }
+
+  /**
+   * Tests the constructor of {@link Truck} with invalid load capacity (negative value).
+   */
+  @Test
+  void testTruckConstructorWithInvalidLoadCapacity() {
+    IllegalArgumentException exception = assertThrows(IllegalArgumentException.class, () -> {
+      new Truck(2018, "Ford", "F-150", -60, 2);
+    });
+    assertEquals("Load capacity must be positive.", exception.getMessage());
+  }
+
+  /**
+   * Tests the constructor of {@link Truck} with zero load capacity.
+   */
+  @Test
+  void testTruckConstructorWithZeroLoadCapacity() {
+    IllegalArgumentException exception = assertThrows(IllegalArgumentException.class, () -> {
+      new Truck(2018, "Ford", "F-150", 0, 2);
+    });
+    assertEquals("Load capacity must be positive.", exception.getMessage());
+  }
+
+  /**
+   * Tests setting invalid number of doors in {@link Car} using setter (negative value).
+   */
+  @Test
+  void testSetInvalidNumDoors() {
+    Car car = new Car(2020, "Toyota", "Corolla", 4, 1);
+    IllegalArgumentException exception = assertThrows(IllegalArgumentException.class, () -> {
+      car.setNumDoors(-2);
+    });
+    assertEquals("Number of doors must be positive.", exception.getMessage());
+  }
+
+  /**
+   * Tests setting invalid load capacity in {@link Truck} using setter (negative value).
+   */
+  @Test
+  void testSetInvalidLoadCapacity() {
+    Truck truck = new Truck(2018, "Ford", "F-150", 60, 2);
+    IllegalArgumentException exception = assertThrows(IllegalArgumentException.class, () -> {
+      truck.setLoadCapacity(-10);
+    });
+    assertEquals("Load capacity must be positive.", exception.getMessage());
+  }
 }
+
+