From 4e4139fc7c78cc5e3d3089a81973e95d02b4ec1b Mon Sep 17 00:00:00 2001
From: "Richard T. Miles" <Richard@miles.systems>
Date: Sun, 10 Aug 2025 00:31:28 -0600
Subject: [PATCH] test: add bbox helper tests

---
 src/api/C6Constants.ts      |  1 +
 src/api/orm/queryHelpers.ts |  6 +++++
 test/queryHelpers.test.js   | 44 +++++++++++++++++++++++++++++++++++++
 test/test.js                |  1 +
 4 files changed, 52 insertions(+)
 create mode 100644 test/queryHelpers.test.js

diff --git a/src/api/C6Constants.ts b/src/api/C6Constants.ts
index 6fbc2ca..91d830b 100644
--- a/src/api/C6Constants.ts
+++ b/src/api/C6Constants.ts
@@ -120,6 +120,7 @@ export const C6Constants = {
     ST_GEOMFROMWKB: 'ST_GeomFromWKB',
     ST_INTERSECTS: 'ST_Intersects',
     ST_LENGTH: 'ST_Length',
+    ST_MAKEENVELOPE: 'ST_MakeEnvelope',
     ST_OVERLAPS: 'ST_Overlaps',
     ST_POINT: 'ST_Point',
     ST_SETSRID: 'ST_SetSRID',
diff --git a/src/api/orm/queryHelpers.ts b/src/api/orm/queryHelpers.ts
index d0904b2..04f8684 100644
--- a/src/api/orm/queryHelpers.ts
+++ b/src/api/orm/queryHelpers.ts
@@ -16,3 +16,9 @@ export const fieldEq = (leftCol: string, rightCol: string, leftAlias: string, ri
 // ST_Distance_Sphere for aliased fields
 export const distSphere = (fromCol: string, toCol: string, fromAlias: string, toAlias: string): any[] =>
     [C6C.ST_DISTANCE_SPHERE, F(fromCol, fromAlias), F(toCol, toAlias)];
+
+// Build a bounding-box expression
+export const bbox = (minLng: number, minLat: number, maxLng: number, maxLat: number): any[] =>
+    [C6C.ST_SRID, [C6C.ST_MAKEENVELOPE,
+        [C6C.ST_POINT, minLng, minLat],
+        [C6C.ST_POINT, maxLng, maxLat]], 4326];
diff --git a/test/queryHelpers.test.js b/test/queryHelpers.test.js
new file mode 100644
index 0000000..6180b62
--- /dev/null
+++ b/test/queryHelpers.test.js
@@ -0,0 +1,44 @@
+import assert from 'assert';
+import { bbox, C6C } from '../dist/index.esm.js';
+
+// Verify basic bounding box construction
+const expected = [
+  C6C.ST_SRID,
+  [
+    C6C.ST_MAKEENVELOPE,
+    [C6C.ST_POINT, 10, 20],
+    [C6C.ST_POINT, 30, 40],
+  ],
+  4326,
+];
+const actual = bbox(10, 20, 30, 40);
+assert.deepStrictEqual(actual, expected);
+console.log('\u001B[32m✓\u001B[39m bbox basic structure test passed');
+
+// Negative coordinate handling
+const expectedNegative = [
+  C6C.ST_SRID,
+  [
+    C6C.ST_MAKEENVELOPE,
+    [C6C.ST_POINT, -10, -20],
+    [C6C.ST_POINT, -30, -40],
+  ],
+  4326,
+];
+const actualNegative = bbox(-10, -20, -30, -40);
+assert.deepStrictEqual(actualNegative, expectedNegative);
+console.log('\u001B[32m✓\u001B[39m bbox negative coordinates test passed');
+
+// Swapped min/max arguments
+const expectedSwapped = [
+  C6C.ST_SRID,
+  [
+    C6C.ST_MAKEENVELOPE,
+    [C6C.ST_POINT, 30, 40],
+    [C6C.ST_POINT, 10, 20],
+  ],
+  4326,
+];
+const actualSwapped = bbox(30, 40, 10, 20);
+assert.deepStrictEqual(actualSwapped, expectedSwapped);
+console.log('\u001B[32m✓\u001B[39m bbox swapped arguments test passed');
diff --git a/test/test.js b/test/test.js
index 715cc2b..5ecdd49 100644
--- a/test/test.js
+++ b/test/test.js
@@ -1,3 +1,4 @@
+import './queryHelpers.test.js';
 import assert from 'assert';
 
 // basic sanity test to ensure test infrastructure runs