Added -Dvallang.random.partialDateTime=true option to turn off genera…

…tion of partial datetimes by default
usethesource · Sep 27, 2021 · c88bf39 · c88bf39
1 parent 6233a9a
commit c88bf39
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/pom.xml b/pom.xml
@@ -110,6 +110,7 @@
 						-Xmx2G
 						-ea
 						-Dvallang.test.seed=${buildNumber}
+						-Dvallang.random.partialDateTime=true
 					</argLine>
 				</configuration>
 			</plugin>

diff --git a/src/main/java/io/usethesource/vallang/type/DateTimeType.java b/src/main/java/io/usethesource/vallang/type/DateTimeType.java
@@ -76,9 +76,10 @@ private static long between(Random r, long a, long b) {
 	@Override
 	public IValue randomValue(Random random, IValueFactory vf, TypeStore store, Map<Type, Type> typeParameters,
 	        int maxDepth, int maxBreadth) {
+		boolean partialDateTime = "true".equals(System.getProperty("vallang.random.partialDateTime"));
 
         try {
-			if (random.nextDouble() > 0.8) {
+			if (partialDateTime && random.nextDouble() > 0.8) {
 				LocalTime result = LocalTime.ofSecondOfDay(between(random, ChronoField.SECOND_OF_DAY));
 				return vf.time(
 					result.getHour(), 
@@ -87,7 +88,7 @@ public IValue randomValue(Random random, IValueFactory vf, TypeStore store, Map<
 					(int) TimeUnit.MILLISECONDS.convert(result.getNano(), TimeUnit.NANOSECONDS)
 				);
 			}
-			if (random.nextDouble() > 0.8) {
+			if (partialDateTime && random.nextDouble() > 0.8) {
 				LocalDate result = LocalDate.ofEpochDay(between(random, 
 					LocalDate.of(0,1, 1).toEpochDay(),
 					LocalDate.of(9999,1, 1).toEpochDay()