test: update functional tests to support new api

Author: bernard-ng

CHANGELOG.md

@@ -6,6 +6,7 @@
 
 - Refactored `fetchNumeric()`, `fetchAssociative()`, and `fetchOne()` across `Result`, `Connection`, `QueryBuilder`, portability wrappers, and all bundled drivers to return `undefined` instead of `false` when no row is available, and aligned unit tests and metadata-provider integrations with the new contract.
 - Normalized bound query parameter values so `undefined` is treated as SQL `NULL` (`null`) before hitting driver statements, preventing mysql2 bind errors and aligning positional and named parameter flows.
+- Updated remaining functional tests to assert `undefined` (instead of `false`) for empty single-row `fetch*` results and no-row guards.
 
 # 1.1.1
 

src/__tests__/functional/auto-increment-column.test.ts

@@ -64,7 +64,7 @@ describe("Functional/AutoIncrementColumnTest", () => {
 
 async function maxId(connection: Connection): Promise<number> {
   const value = await connection.fetchOne("SELECT MAX(id) FROM auto_increment_table");
-  expect(value).not.toBe(false);
+  expect(value).toBeDefined();
 
   return Number(value);
 }

src/__tests__/functional/binary-data-access.test.ts

@@ -54,7 +54,7 @@ describe("Functional/BinaryDataAccessTest", () => {
     stmt.bindValue(2, Buffer.from("c0def00d", "hex"), ParameterType.BINARY);
 
     const row = lowerCaseKeys((await stmt.executeQuery()).fetchAssociative());
-    expect(row).not.toBe(false);
+    expect(row).toBeDefined();
     expect(Object.keys(row as Record<string, unknown>)).toEqual(["test_int", "test_binary"]);
     expect((row as Record<string, unknown>).test_int).toBe(1);
     expect(toBinaryBuffer((row as Record<string, unknown>).test_binary)).toEqual(
@@ -130,7 +130,7 @@ describe("Functional/BinaryDataAccessTest", () => {
         { 1: ParameterType.BINARY },
       );
 
-    expect(row).not.toBe(false);
+    expect(row).toBeDefined();
     const normalized = lowerCaseKeys(row);
     expect(normalized.test_int).toBe(1);
     expect(toBinaryBuffer(normalized.test_binary)).toEqual(Buffer.from("c0def00d", "hex"));
@@ -145,7 +145,7 @@ describe("Functional/BinaryDataAccessTest", () => {
         [ParameterType.STRING, Types.BINARY],
       );
 
-    expect(row).not.toBe(false);
+    expect(row).toBeDefined();
     const normalized = lowerCaseKeys(row);
     expect(normalized.test_int).toBe(1);
     expect(toBinaryBuffer(normalized.test_binary)).toEqual(Buffer.from("c0def00d", "hex"));
@@ -160,7 +160,7 @@ describe("Functional/BinaryDataAccessTest", () => {
         { 1: ParameterType.BINARY },
       );
 
-    expect(row).not.toBe(false);
+    expect(row).toBeDefined();
     expect(row?.[0]).toBe(1);
     expect(toBinaryBuffer(row?.[1])).toEqual(Buffer.from("c0def00d", "hex"));
   });
@@ -174,7 +174,7 @@ describe("Functional/BinaryDataAccessTest", () => {
         [ParameterType.STRING, Types.BINARY],
       );
 
-    expect(row).not.toBe(false);
+    expect(row).toBeDefined();
     expect(row?.[0]).toBe(1);
     expect(toBinaryBuffer(row?.[1])).toEqual(Buffer.from("c0def00d", "hex"));
   });
@@ -255,8 +255,8 @@ describe("Functional/BinaryDataAccessTest", () => {
   });
 });
 
-function lowerCaseKeys(row: false | Record<string, unknown> | undefined): Record<string, unknown> {
-  if (row === false || row === undefined) {
+function lowerCaseKeys(row: Record<string, unknown> | undefined): Record<string, unknown> {
+  if (row === undefined) {
     throw new Error("Expected a row.");
   }
 

src/__tests__/functional/blob.test.ts

@@ -148,7 +148,7 @@ describe("Functional/BlobTest", () => {
     const blobs = await functional
       .connection()
       .fetchNumeric("SELECT blobcolumn1, blobcolumn2 FROM blob_table");
-    expect(blobs).not.toBe(false);
+    expect(blobs).toBeDefined();
 
     const actual = (blobs ?? []).map((blob) => toText(blob));
     expect(actual).toEqual(["test1", "test2"]);

src/__tests__/functional/data-access.test.ts

@@ -129,7 +129,7 @@ describe("Functional/DataAccessTest", () => {
         [1, "foo"],
       );
 
-    expect(row).not.toBe(false);
+    expect(row).toBeDefined();
     const normalized = lowerCaseKeys(row);
     expect(normalized.test_int).toBe(1);
     expect(normalized.test_string).toBe("foo");
@@ -146,7 +146,7 @@ describe("Functional/DataAccessTest", () => {
         [ParameterType.STRING, Types.DATETIME_MUTABLE],
       );
 
-    expect(row).not.toBe(false);
+    expect(row).toBeDefined();
     const normalized = lowerCaseKeys(row);
     expect(normalized.test_int).toBe(1);
     expect(normalizeDateTimeSecondPrecision(normalized.test_datetime)).toBe("2010-01-01 10:10:10");
@@ -160,7 +160,7 @@ describe("Functional/DataAccessTest", () => {
         [1, "foo"],
       );
 
-    expect(row).not.toBe(false);
+    expect(row).toBeDefined();
     expect(row?.[0]).toBe(1);
     expect(row?.[1]).toBe("foo");
   });
@@ -176,7 +176,7 @@ describe("Functional/DataAccessTest", () => {
         [ParameterType.STRING, Types.DATETIME_MUTABLE],
       );
 
-    expect(row).not.toBe(false);
+    expect(row).toBeDefined();
     expect(row?.[0]).toBe(1);
     expect(normalizeDateTimeSecondPrecision(row?.[1])).toBe("2010-01-01 10:10:10");
   });
@@ -420,12 +420,12 @@ async function assertDateExpression(
   bindParams(stmt, interval);
 
   const date = (await stmt.executeQuery()).fetchOne();
-  expect(date).not.toBe(false);
+  expect(date).toBeDefined();
   expect(normalizeDateTimeSecondPrecision(date)).toBe(expected);
 }
 
-function lowerCaseKeys(row: false | Record<string, unknown> | undefined): Record<string, unknown> {
-  if (row === false || row === undefined) {
+function lowerCaseKeys(row: Record<string, unknown> | undefined): Record<string, unknown> {
+  if (row === undefined) {
     throw new Error("Expected a row.");
   }
 

src/__tests__/functional/platform/alter-column.test.ts

@@ -81,7 +81,7 @@ describe("Functional/Platform/AlterColumnTest", () => {
     const hasIcuCollations =
       (await functional
         .connection()
-        .fetchOne("SELECT 1 FROM pg_collation WHERE collprovider = 'icu'")) !== false;
+        .fetchOne("SELECT 1 FROM pg_collation WHERE collprovider = 'icu'")) !== undefined;
     if (!hasIcuCollations) {
       skip();
     }

src/__tests__/functional/platform/default-expression.test.ts

@@ -67,8 +67,8 @@ async function assertDefaultExpression(
   const row = await connection.fetchNumeric<[unknown, unknown]>(
     "SELECT default_value, actual_value FROM default_expr_test",
   );
-  expect(row).not.toBe(false);
-  if (row === false) {
+  expect(row).toBeDefined();
+  if (row === undefined) {
     return;
   }
 

src/__tests__/functional/platform/quoting.test.ts

@@ -28,8 +28,8 @@ describe("Functional/Platform/QuotingTest", () => {
       );
       const row = await functional.connection().fetchAssociative(query);
 
-      expect(row).not.toBe(false);
-      if (row === false) {
+      expect(row).toBeDefined();
+      if (row === undefined) {
         return;
       }
 

src/__tests__/functional/result.test.ts

@@ -15,17 +15,17 @@ describe("Functional/ResultTest", () => {
     [
       "fetchNumeric",
       (result: Awaited<ReturnType<typeof connection.executeQuery>>) => result.fetchNumeric(),
-      false,
+      undefined,
     ],
     [
       "fetchAssociative",
       (result: Awaited<ReturnType<typeof connection.executeQuery>>) => result.fetchAssociative(),
-      false,
+      undefined,
     ],
     [
       "fetchOne",
       (result: Awaited<ReturnType<typeof connection.executeQuery>>) => result.fetchOne(),
-      false,
+      undefined,
     ],
     [
       "fetchAllNumeric",

src/__tests__/functional/schema/postgre-sql/schema.test.ts

@@ -44,8 +44,8 @@ describe("Functional/Schema/PostgreSQL/SchemaTest", () => {
       ["my_table"],
     );
 
-    expect(row).not.toBe(false);
-    if (row === false) {
+    expect(row).toBeDefined();
+    if (row === undefined) {
       return;
     }