ci: remove redundant type tests

eslint.config.mjs

@@ -151,6 +151,12 @@ export default [
       '@typescript-eslint/no-unsafe-function-type': 'off',
     },
   },
+  {
+    files: ['test/tsc-build/**/*.ts'],
+    rules: {
+      '@typescript-eslint/no-unused-vars': 'off',
+    },
+  },
   {
     files: ['**/*.mjs', '**/*.mts'],
     languageOptions: {

test/tsc-build/helpers.test.ts

@@ -16,22 +16,3 @@ export const isResultSetHeader = (
 
   return keys.every((key) => key in data);
 };
-
-export const isOkPacket = (
-  data: unknown
-): data is mysql.OkPacket | mysqlp.OkPacket => {
-  if (!data || typeof data !== 'object') return false;
-
-  const keys = [
-    'fieldCount',
-    'affectedRows',
-    'changedRows',
-    'insertId',
-    'serverStatus',
-    'warningCount',
-    'message',
-    'protocol41',
-  ];
-
-  return keys.every((key) => key in data);
-};

test/tsc-build/mysql/constants/CharsetToEncoding.test.ts

@@ -2,5 +2,3 @@ import { mysql } from '../../index.test.js';
 
 const charsetToEncoding: string[] = mysql.CharsetToEncoding;
 const utf8: string = charsetToEncoding[0];
-
-console.log(utf8, charsetToEncoding);

test/tsc-build/mysql/constants/Charsets.test.ts

@@ -2,5 +2,3 @@ import { mysql } from '../../index.test.js';
 
 const BIG5_CHINESE_CI: number = mysql.Charsets.BIG5_CHINESE_CI;
 const BIG5: number = mysql.Charsets.BIG5;
-
-console.log(BIG5_CHINESE_CI, BIG5);

test/tsc-build/mysql/constants/Types.test.ts

@@ -3,5 +3,3 @@ import { mysql } from '../../index.test.js';
 const BLOB: number = mysql.Types.BLOB;
 const DECIMAL: string = mysql.Types[0x00];
 const DOUBLE: string = mysql.Types[5];
-
-console.log(BLOB, DECIMAL, DOUBLE);

test/tsc-build/mysql/createPool/callbacks/connection.test.ts

@@ -3,13 +3,9 @@ import { access } from '../../baseConnection.test.js';
 
 const pool = mysql.createPool(access);
 
-pool.getConnection((_err, conn) => {
+pool.getConnection((_, conn) => {
   conn.connection;
 
-  try {
-    // @ts-expect-error: The pool can't be a connection itself
-    pool.connection;
-  } catch (err) {
-    console.log('This error is expected', err);
-  }
+  // @ts-expect-error: The pool can't be a connection itself
+  pool.connection;
 });

test/tsc-build/mysql/createPool/callbacks/createPool.test.ts

@@ -1,12 +1,5 @@
 import { mysql } from '../../../index.test.js';
-import { uriAccess, access } from '../../baseConnection.test.js';
+import { uriAccess } from '../../baseConnection.test.js';
 
-(() => {
-  let uriPool: mysql.Pool | null = null;
-  let pool: mysql.Pool | null = null;
-
-  if (uriPool === null || pool === null) return;
-
-  uriPool = mysql.createPool(uriAccess);
-  pool = mysql.createPool(access);
-})();
+// createPool(uri) overload is not covered by .mts tests
+const pool: mysql.Pool = mysql.createPool(uriAccess);

test/tsc-build/mysql/createPool/callbacks/getConnection.test.ts

@@ -3,11 +3,7 @@ import { access } from '../../baseConnection.test.js';
 
 const pool = mysql.createPool(access);
 
-pool.getConnection((_err, conn) => {
-  try {
-    // @ts-expect-error: The connection can't get another connection
-    conn.getConnection();
-  } catch (err) {
-    console.log('This error is expected', err);
-  }
+pool.getConnection((_) => {
+  // @ts-expect-error: The connection can't get another connection
+  conn.getConnection();
 });

test/tsc-build/mysql/createPool/callbacks/release.test.ts

@@ -3,13 +3,5 @@ import { access } from '../../baseConnection.test.js';
 
 const pool = mysql.createPool(access);
 
-pool.getConnection((_err, conn) => {
-  conn.release();
-
-  try {
-    // @ts-expect-error: The pool isn't a connection itself, so it doesn't have the connection methods
-    pool.release();
-  } catch (err) {
-    console.log('This error is expected', err);
-  }
-});
+// @ts-expect-error: The pool isn't a connection itself, so it doesn't have the connection methods
+pool.release();