Clarify the Value Object concept

Author: mperor

CleanCode/src/main/java/pl/mperor/lab/java/clean/code/ddd/value/object/DateTimeRange.java renamed to CleanCode/src/main/java/pl/mperor/lab/java/clean/code/ddd/value/object/DateRange.java

@@ -1,10 +1,10 @@
 package pl.mperor.lab.java.clean.code.ddd.value.object;
 
-import java.time.LocalDateTime;
+import java.time.LocalDate;
 
-record DateTimeRange(LocalDateTime start, LocalDateTime end) {
+record DateRange(LocalDate start, LocalDate end) {
 
-    DateTimeRange {
+    DateRange {
         if (start == null || end == null) {
             throw new IllegalArgumentException("Date range start & end cannot be null!");
         }
@@ -13,7 +13,7 @@ record DateTimeRange(LocalDateTime start, LocalDateTime end) {
         }
     }
 
-    boolean isWithinRange(LocalDateTime dateTime) {
+    boolean isWithinRange(LocalDate dateTime) {
         return !dateTime.isBefore(start) && !dateTime.isAfter(end);
     }
 }

CleanCode/src/test/java/pl/mperor/lab/java/clean/code/ddd/value/object/ValueObjectTest.java

@@ -5,9 +5,18 @@
 
 import java.math.BigDecimal;
 import java.math.RoundingMode;
-import java.time.LocalDateTime;
-
-
+import java.time.LocalDate;
+
+/**
+ * 🧩 Value Object
+ *
+ * <p>A small simple object, like money or a date range, whose equality isn't based on identity.
+ *
+ * <p><b>Key Traits:</b>
+ * - Immutable 🚫
+ * - Equality by values ✅
+ * - Reusable & shareable ♻️
+ */
 class ValueObjectTest {
 
     @Test
@@ -33,11 +42,11 @@ void shouldAllowToUseEmailAddressAsValueObject() {
 
     @Test
     void shouldAllowToUseDateTimeRangeAsValueObject() {
-        Assertions.assertTrue(new DateTimeRange(LocalDateTime.MIN, LocalDateTime.MAX)
-                .isWithinRange(LocalDateTime.now()));
-        Assertions.assertTrue(new DateTimeRange(LocalDateTime.MIN, LocalDateTime.MAX)
-                .isWithinRange(LocalDateTime.MIN));
-        Assertions.assertTrue(new DateTimeRange(LocalDateTime.MIN, LocalDateTime.MAX)
-                .isWithinRange(LocalDateTime.MAX));
+        Assertions.assertTrue(new DateRange(LocalDate.MIN, LocalDate.MAX)
+                .isWithinRange(LocalDate.now()));
+        Assertions.assertTrue(new DateRange(LocalDate.MIN, LocalDate.MAX)
+                .isWithinRange(LocalDate.MIN));
+        Assertions.assertTrue(new DateRange(LocalDate.MIN, LocalDate.MAX)
+                .isWithinRange(LocalDate.MAX));
     }
 }