Add more getter types for time

Author: ogesaku

src/main/java/com/coditory/quark/config/AuditableConfig.java

@@ -2,6 +2,7 @@
 
 import org.jetbrains.annotations.NotNull;
 
+import java.util.Arrays;
 import java.util.List;
 import java.util.Map;
 import java.util.Map.Entry;
@@ -10,6 +11,7 @@
 import java.util.stream.Collectors;
 
 import static com.coditory.quark.config.ConfigRemoveOptions.removeEmptyParents;
+import static com.coditory.quark.config.Preconditions.expectNonNull;
 
 public final class AuditableConfig extends ConfigDecorator {
     private static final Object USED_MARKER = new Object();

src/main/java/com/coditory/quark/config/Config.java

@@ -4,6 +4,7 @@
 import org.jetbrains.annotations.Nullable;
 
 import java.util.LinkedHashMap;
+import java.util.List;
 import java.util.Map;
 import java.util.Objects;
 import java.util.Optional;
@@ -88,13 +89,29 @@ default AuditableConfig auditable() {
         return AuditableConfig.of(this);
     }
 
+    default <T> T mapAuditableSubConfigOrNull(@NotNull String path, @NotNull Function<AuditableConfig, T> configMapper) {
+        Config subconfig =  this.getSubConfigOrNull(path);
+        if (subconfig == null) {
+            return null;
+        }
+        return subconfig.mapAuditable(configMapper);
+    }
+
+    default <T> T mapAuditableSubConfig(@NotNull String path, @NotNull Function<AuditableConfig, T> configMapper) {
+        return this.getSubConfig(path).mapAuditable(configMapper);
+    }
+
     default <T> T mapAuditable(@NotNull Function<AuditableConfig, T> configMapper) {
         AuditableConfig auditableConfig = AuditableConfig.of(this);
         T result = configMapper.apply(auditableConfig);
         auditableConfig.failOnUnusedProperties();
         return result;
     }
 
+    default void consumeAuditableSubConfig(@NotNull String path, @NotNull Consumer<AuditableConfig> configConsumer) {
+        this.getSubConfig(path).consumeAuditable(configConsumer);
+    }
+
     default void consumeAuditable(@NotNull Consumer<AuditableConfig> configConsumer) {
         AuditableConfig auditableConfig = AuditableConfig.of(this);
         configConsumer.accept(auditableConfig);

src/main/java/com/coditory/quark/config/ConfigGetters.java

@@ -13,6 +13,7 @@
 import java.util.Optional;
 
 import static com.coditory.quark.config.MissingConfigValueException.missingConfigValueForPath;
+import static com.coditory.quark.config.Preconditions.expectNonBlank;
 import static com.coditory.quark.config.Preconditions.expectNonNull;
 
 interface ConfigGetters {
@@ -121,6 +122,8 @@ default Object getObjectOrNull(@NotNull String path) {
 
     @NotNull
     default Object getObject(@NotNull String path, @NotNull Object defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getObjectAsOptional(path).orElse(defaultValue);
     }
 
@@ -147,6 +150,8 @@ default Boolean getBooleanOrNull(@NotNull String path) {
 
     @NotNull
     default Boolean getBoolean(@NotNull String path, @NotNull Boolean defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getBooleanAsOptional(path).orElse(defaultValue);
     }
 
@@ -173,6 +178,8 @@ default Short getShortOrNull(@NotNull String path) {
 
     @NotNull
     default Short getShort(@NotNull String path, @NotNull Short defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getShortAsOptional(path).orElse(defaultValue);
     }
 
@@ -199,6 +206,8 @@ default Byte getByteOrNull(@NotNull String path) {
 
     @NotNull
     default Byte getByte(@NotNull String path, @NotNull Byte defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getByteAsOptional(path).orElse(defaultValue);
     }
 
@@ -225,6 +234,8 @@ default Integer getIntegerOrNull(@NotNull String path) {
 
     @NotNull
     default Integer getInteger(@NotNull String path, @NotNull Integer defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getIntegerAsOptional(path).orElse(defaultValue);
     }
 
@@ -251,6 +262,8 @@ default Long getLongOrNull(@NotNull String path) {
 
     @NotNull
     default Long getLong(@NotNull String path, @NotNull Long defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getLongAsOptional(path).orElse(defaultValue);
     }
 
@@ -277,6 +290,8 @@ default Float getFloatOrNull(@NotNull String path) {
 
     @NotNull
     default Float getFloat(@NotNull String path, @NotNull Float defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getFloatAsOptional(path).orElse(defaultValue);
     }
 
@@ -303,6 +318,8 @@ default Double getDoubleOrNull(@NotNull String path) {
 
     @NotNull
     default Double getDouble(@NotNull String path, @NotNull Double defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getDoubleAsOptional(path).orElse(defaultValue);
     }
 
@@ -329,6 +346,8 @@ default BigDecimal getBigDecimalOrNull(@NotNull String path) {
 
     @NotNull
     default BigDecimal getBigDecimal(@NotNull String path, @NotNull BigDecimal defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getBigDecimalAsOptional(path).orElse(defaultValue);
     }
 
@@ -355,6 +374,8 @@ default Instant getInstantOrNull(@NotNull String path) {
 
     @NotNull
     default Instant getInstant(@NotNull String path, @NotNull Instant defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getInstantAsOptional(path).orElse(defaultValue);
     }
 
@@ -381,6 +402,8 @@ default ZonedDateTime getZonedDateTimeOrNull(@NotNull String path) {
 
     @NotNull
     default ZonedDateTime getZonedDateTime(@NotNull String path, @NotNull ZonedDateTime defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getZonedDateTimeAsOptional(path).orElse(defaultValue);
     }
 
@@ -407,6 +430,8 @@ default Duration getDurationOrNull(@NotNull String path) {
 
     @NotNull
     default Duration getDuration(@NotNull String path, @NotNull Duration defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getDurationAsOptional(path).orElse(defaultValue);
     }
 
@@ -433,6 +458,8 @@ default Locale getLocaleOrNull(@NotNull String path) {
 
     @NotNull
     default Locale getLocale(@NotNull String path, @NotNull Locale defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getLocaleAsOptional(path).orElse(defaultValue);
     }
 
@@ -459,6 +486,8 @@ default Currency getCurrencyOrNull(@NotNull String path) {
 
     @NotNull
     default Currency getCurrency(@NotNull String path, @NotNull Currency defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getCurrencyAsOptional(path).orElse(defaultValue);
     }
 
@@ -493,6 +522,8 @@ default List<String> getStringListOrEmpty(@NotNull String path) {
 
     @NotNull
     default List<String> getStringList(@NotNull String path, @NotNull List<String> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getStringListAsOptional(path).orElse(defaultValue);
     }
 
@@ -519,6 +550,8 @@ default List<Object> getObjectListOrNull(@NotNull String path) {
 
     @NotNull
     default List<Object> getObjectList(@NotNull String path, @NotNull List<Object> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getObjectListAsOptional(path).orElse(defaultValue);
     }
 
@@ -545,6 +578,8 @@ default List<Boolean> getBooleanListOrNull(@NotNull String path) {
 
     @NotNull
     default List<Boolean> getBooleanList(@NotNull String path, @NotNull List<Boolean> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getBooleanListAsOptional(path).orElse(defaultValue);
     }
 
@@ -571,6 +606,8 @@ default List<Short> getShortListOrNull(@NotNull String path) {
 
     @NotNull
     default List<Short> getShortList(@NotNull String path, @NotNull List<Short> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getShortListAsOptional(path).orElse(defaultValue);
     }
 
@@ -597,6 +634,8 @@ default List<Byte> getByteListOrNull(@NotNull String path) {
 
     @NotNull
     default List<Byte> getByteList(@NotNull String path, @NotNull List<Byte> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getByteListAsOptional(path).orElse(defaultValue);
     }
 
@@ -623,6 +662,8 @@ default List<Integer> getIntegerListOrNull(@NotNull String path) {
 
     @NotNull
     default List<Integer> getIntegerList(@NotNull String path, @NotNull List<Integer> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getIntegerListAsOptional(path).orElse(defaultValue);
     }
 
@@ -649,6 +690,8 @@ default List<Long> getLongListOrNull(@NotNull String path) {
 
     @NotNull
     default List<Long> getLongList(@NotNull String path, @NotNull List<Long> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getLongListAsOptional(path).orElse(defaultValue);
     }
 
@@ -675,6 +718,8 @@ default List<Float> getFloatListOrNull(@NotNull String path) {
 
     @NotNull
     default List<Float> getFloatList(@NotNull String path, @NotNull List<Float> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getFloatListAsOptional(path).orElse(defaultValue);
     }
 
@@ -701,6 +746,8 @@ default List<Double> getDoubleListOrNull(@NotNull String path) {
 
     @NotNull
     default List<Double> getDoubleList(@NotNull String path, @NotNull List<Double> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getDoubleListAsOptional(path).orElse(defaultValue);
     }
 
@@ -727,6 +774,8 @@ default List<BigDecimal> getBigDecimalListOrNull(@NotNull String path) {
 
     @NotNull
     default List<BigDecimal> getBigDecimalList(@NotNull String path, @NotNull List<BigDecimal> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getBigDecimalListAsOptional(path).orElse(defaultValue);
     }
 
@@ -753,6 +802,8 @@ default List<Instant> getInstantListOrNull(@NotNull String path) {
 
     @NotNull
     default List<Instant> getInstantList(@NotNull String path, @NotNull List<Instant> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getInstantListAsOptional(path).orElse(defaultValue);
     }
 
@@ -779,6 +830,8 @@ default List<ZonedDateTime> getZonedDateTimeListOrNull(@NotNull String path) {
 
     @NotNull
     default List<ZonedDateTime> getZonedDateTimeList(@NotNull String path, @NotNull List<ZonedDateTime> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getZonedDateTimeListAsOptional(path).orElse(defaultValue);
     }
 
@@ -805,6 +858,8 @@ default List<Duration> getDurationListOrNull(@NotNull String path) {
 
     @NotNull
     default List<Duration> getDurationList(@NotNull String path, @NotNull List<Duration> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getDurationListAsOptional(path).orElse(defaultValue);
     }
 
@@ -831,6 +886,8 @@ default List<Locale> getLocaleListOrNull(@NotNull String path) {
 
     @NotNull
     default List<Locale> getLocaleList(@NotNull String path, @NotNull List<Locale> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getLocaleListAsOptional(path).orElse(defaultValue);
     }
 
@@ -857,6 +914,42 @@ default List<Currency> getCurrencyListOrNull(@NotNull String path) {
 
     @NotNull
     default List<Currency> getCurrencyList(@NotNull String path, @NotNull List<Currency> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
         return getCurrencyListAsOptional(path).orElse(defaultValue);
     }
+
+    // SubConfig List API
+
+    @NotNull
+    default Optional<List<Config>> getSubConfigListAsOptional(@NotNull String path) {
+        expectNonBlank(path, "path");
+        return getListAsOptional(Config.class, path);
+    }
+
+    @NotNull
+    default List<Config> getSubConfigList(@NotNull String path) {
+        expectNonNull(path, "path");
+        return getSubConfigListAsOptional(path)
+                .orElseThrow(() -> missingConfigValueForPath(path));
+    }
+
+    @NotNull
+    default List<Config> getSubConfigListOrEmpty(@NotNull String path) {
+        expectNonBlank(path, "path");
+        return getSubConfigListAsOptional(path).orElse(List.of());
+    }
+
+    @Nullable
+    default List<Config> getSubConfigListOrNull(@NotNull String path) {
+        expectNonNull(path, "path");
+        return getSubConfigListAsOptional(path).orElse(null);
+    }
+
+    @NotNull
+    default List<Config> getSubConfigList(@NotNull String path, @NotNull List<Config> defaultValue) {
+        expectNonNull(path, "path");
+        expectNonNull(defaultValue, "defaultValue");
+        return getSubConfigListAsOptional(path).orElse(defaultValue);
+    }
 }

src/main/java/com/coditory/quark/config/ConfigValueParser.java

@@ -3,6 +3,9 @@
 import java.math.BigDecimal;
 import java.time.Duration;
 import java.time.Instant;
+import java.time.LocalDate;
+import java.time.LocalDateTime;
+import java.time.LocalTime;
 import java.time.ZonedDateTime;
 import java.util.ArrayList;
 import java.util.Currency;
@@ -24,6 +27,9 @@ class ConfigValueParser {
             ValueParser.forType(BigDecimal.class, BigDecimal::new),
             ValueParser.forType(ZonedDateTime.class, value -> ZonedDateTime.parse(value, ISO_OFFSET_DATE_TIME)),
             ValueParser.forType(Instant.class, Instant::parse),
+            ValueParser.forType(LocalDateTime.class, LocalDateTime::parse),
+            ValueParser.forType(LocalDate.class, LocalDate::parse),
+            ValueParser.forType(LocalTime.class, LocalTime::parse),
             ValueParser.forType(Duration.class, DurationParser::parseDuration),
             ValueParser.forType(Locale.class, LocaleParser::parseLocale),
             ValueParser.forType(Currency.class, Currency::getInstance)

src/main/java/com/coditory/quark/config/ListConfigNode.java

@@ -18,6 +18,10 @@ class ListConfigNode implements ConfigNode {
         this.values = values;
     }
 
+    List<ConfigNode> children() {
+        return List.copyOf(values);
+    }
+
     @Override
     public boolean isEmpty() {
         return values.isEmpty();