default(DateTime) is actualy a conditionless condition

Author: Mpdreamz

src/Nest/QueryDsl/TermLevel/Range/DateRangeQuery .cs

@@ -43,10 +43,10 @@ public class DateRangeQuery : FieldNameQueryBase, IDateRangeQuery
 		internal override void InternalWrapInContainer(IQueryContainer c) => c.Range = this;
 
 		internal static bool IsConditionless(IDateRangeQuery q) => q.Field.IsConditionless()
-			&& (q.GreaterThanOrEqualTo == null || !q.GreaterThanOrEqualTo.IsValid)
+			|| ((q.GreaterThanOrEqualTo == null || !q.GreaterThanOrEqualTo.IsValid)
 			&& (q.LessThanOrEqualTo == null || !q.LessThanOrEqualTo.IsValid)
 			&& (q.GreaterThan == null || !q.GreaterThan.IsValid)
-			&& (q.LessThan == null || !q.LessThan.IsValid);
+			&& (q.LessThan == null || !q.LessThan.IsValid));
 	}
 
 	[JsonObject(MemberSerialization = MemberSerialization.OptIn)]

src/Tests/Tests/QueryDsl/TermLevel/Range/DateRangeQueryUsageTests.cs

@@ -19,6 +19,13 @@ public DateRangeQueryUsageTests(ReadOnlyCluster cluster, EndpointUsage usage) :
 				q.GreaterThanOrEqualTo = null;
 				q.LessThan = null;
 				q.LessThanOrEqualTo = null;
+			},
+			q =>
+			{
+				q.GreaterThan = default;
+				q.GreaterThanOrEqualTo = default;
+				q.LessThan = default;
+				q.LessThanOrEqualTo = default;
 			}
 		};
 
@@ -67,47 +74,4 @@ protected override QueryContainer QueryFluent(QueryContainerDescriptor<Project>
 			);
 	}
 
-	public class DateRangeDefaultDateTimeQueryUsageTests : QueryDslUsageTestsBase
-	{
-		public DateRangeDefaultDateTimeQueryUsageTests(ReadOnlyCluster cluster, EndpointUsage usage) : base(cluster, usage) { }
-
-		protected override ConditionlessWhen ConditionlessWhen => new ConditionlessWhen<IDateRangeQuery>(q => q.Range as IDateRangeQuery)
-			{ };
-
-		protected override QueryContainer QueryInitializer => new DateRangeQuery
-		{
-			Field = "description",
-			GreaterThan = default,
-			GreaterThanOrEqualTo = default,
-			LessThan = default,
-			LessThanOrEqualTo = default,
-			Format = "dd/MM/yyyy||yyyy",
-			TimeZone = "+01:00"
-		};
-
-		//TODO in 7.0 DateMath should make its DateTime nullable so that DateTime.MinValue is sent out as well.
-		// this is only marginally better then sending an empty string.
-		protected override object QueryJson => new
-		{
-			range = new
-			{
-				description = new
-				{
-					format = "dd/MM/yyyy||yyyy",
-					time_zone = "+01:00"
-				}
-			}
-		};
-
-		protected override QueryContainer QueryFluent(QueryContainerDescriptor<Project> q) => q
-			.DateRange(c => c
-				.Field(p => p.Description)
-				.GreaterThan(default)
-				.GreaterThanOrEquals(default)
-				.LessThan(default)
-				.LessThanOrEquals(default)
-				.Format("dd/MM/yyyy||yyyy")
-				.TimeZone("+01:00")
-			);
-	}
 }