Fix Range._incLower default

It should include the lower bound if `_lower` is not `null` or `undefined`
edgedb · Oct 26, 2023 · c0c7210 · c0c7210
1 parent 551ed4b
commit c0c7210
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/integration-tests/lts/primitives.test.ts b/integration-tests/lts/primitives.test.ts
@@ -66,7 +66,7 @@ describe("primitives", () => {
     );
     assert.equal(
       e.std.range(upperRange).toEdgeQL(),
-      `std::range(<std::int64>{}, 8, inc_lower := true, inc_upper := false)`
+      `std::range(<std::int64>{}, 8, inc_lower := false, inc_upper := false)`
     );
     assert.equal(
       e.std.range(dateRange).toEdgeQL(),

diff --git a/packages/driver/src/datatypes/range.ts b/packages/driver/src/datatypes/range.ts
@@ -26,7 +26,7 @@ export class Range<
   constructor(
     private readonly _lower: T | null,
     private readonly _upper: T | null,
-    private readonly _incLower: boolean = true,
+    private readonly _incLower: boolean = _lower != null,
     private readonly _incUpper: boolean = false
   ) {}