Revert "Fix lint warnings"

This reverts commit cdf7e48.

Author: Soremwar

array_parser.ts

@@ -79,7 +79,7 @@ class ArrayParser {
   consumeDimensions(): void {
     if (this.source[0] === "[") {
       while (!this.isEof()) {
-        const char = this.nextCharacter();
+        let char = this.nextCharacter();
         if (char.value === "=") break;
       }
     }

decode.ts

@@ -139,16 +139,16 @@ function decodeBytea(byteaStr: string): Uint8Array {
 }
 
 function decodeByteaHex(byteaStr: string): Uint8Array {
-  const bytesStr = byteaStr.slice(2);
-  const bytes = new Uint8Array(bytesStr.length / 2);
+  let bytesStr = byteaStr.slice(2);
+  let bytes = new Uint8Array(bytesStr.length / 2);
   for (let i = 0, j = 0; i < bytesStr.length; i += 2, j++) {
     bytes[j] = parseInt(bytesStr[i] + bytesStr[i + 1], HEX);
   }
   return bytes;
 }
 
 function decodeByteaEscape(byteaStr: string): Uint8Array {
-  const bytes = [];
+  let bytes = [];
   let i = 0;
   let k = 0;
   while (i < byteaStr.length) {

encode.ts

@@ -41,7 +41,7 @@ function encodeDate(date: Date): string {
 
 function escapeArrayElement(value: unknown): string {
   // deno-lint-ignore no-explicit-any
-  const strValue = (value as any).toString();
+  let strValue = (value as any).toString();
   const escapedValue = strValue.replace(/\\/g, "\\\\").replace(/"/g, '\\"');
 
   return `"${escapedValue}"`;
@@ -73,7 +73,7 @@ function encodeArray(array: Array<unknown>): string {
 }
 
 function encodeBytes(value: Uint8Array): string {
-  const hex = Array.from(value)
+  let hex = Array.from(value)
     .map((val) => (val < 10 ? `0${val.toString(16)}` : val.toString(16)))
     .join("");
   return `\\x${hex}`;

oid.ts

@@ -14,25 +14,17 @@ export const Oid = {
   xid: 28,
   cid: 29,
   oidvector: 30,
-  // deno-lint-ignore camelcase
   pg_ddl_command: 32,
-  // deno-lint-ignore camelcase
   pg_type: 71,
-  // deno-lint-ignore camelcase
   pg_attribute: 75,
-  // deno-lint-ignore camelcase
   pg_proc: 81,
-  // deno-lint-ignore camelcase
   pg_class: 83,
   json: 114,
   xml: 142,
   _xml: 143,
-  // deno-lint-ignore camelcase
   pg_node_tree: 194,
-  // deno-lint-ignore camelcase
   json_array: 199,
   smgr: 210,
-  // deno-lint-ignore camelcase
   index_am_handler: 325,
   point: 600,
   lseg: 601,
@@ -99,7 +91,6 @@ export const Oid = {
   interval: 1186,
   _interval: 1187,
   _numeric: 1231,
-  // deno-lint-ignore camelcase
   pg_database: 1248,
   _cstring: 1263,
   timetz: 1266,
@@ -127,30 +118,21 @@ export const Oid = {
   anyarray: 2277,
   void: 2278,
   trigger: 2279,
-  // deno-lint-ignore camelcase
   language_handler: 2280,
   internal: 2281,
   opaque: 2282,
   anyelement: 2283,
   _record: 2287,
   anynonarray: 2776,
-  // deno-lint-ignore camelcase
   pg_authid: 2842,
-  // deno-lint-ignore camelcase
   pg_auth_members: 2843,
-  // deno-lint-ignore camelcase
   _txid_snapshot: 2949,
   uuid: 2950,
   _uuid: 2951,
-  // deno-lint-ignore camelcase
   txid_snapshot: 2970,
-  // deno-lint-ignore camelcase
   fdw_handler: 3115,
-  // deno-lint-ignore camelcase
   pg_lsn: 3220,
-  // deno-lint-ignore camelcase
   _pg_lsn: 3221,
-  // deno-lint-ignore camelcase
   tsm_handler: 3310,
   anyenum: 3500,
   tsvector: 3614,
@@ -164,10 +146,8 @@ export const Oid = {
   regdictionary: 3769,
   _regdictionary: 3770,
   jsonb: 3802,
-  // deno-lint-ignore camelcase
   jsonb_array: 3807,
   anyrange: 3831,
-  // deno-lint-ignore camelcase
   event_trigger: 3838,
   int4range: 3904,
   _int4range: 3905,
@@ -181,7 +161,6 @@ export const Oid = {
   _daterange: 3913,
   int8range: 3926,
   _int8range: 3927,
-  // deno-lint-ignore camelcase
   pg_shseclabel: 4066,
   regnamespace: 4089,
   _regnamespace: 4090,

tests/data_types.ts

@@ -269,7 +269,6 @@ testClient(async function bpcharNestedArray() {
 });
 
 testClient(async function jsonArray() {
-  // deno-lint-ignore camelcase
   const json_array = await CLIENT.query(
     `SELECT ARRAY_AGG(A) FROM  (
       SELECT JSON_BUILD_OBJECT( 'X', '1' ) AS A
@@ -280,7 +279,6 @@ testClient(async function jsonArray() {
 
   assertEquals(json_array.rows[0][0], [{ X: "1" }, { Y: "2" }]);
 
-  // deno-lint-ignore camelcase
   const json_array_nested = await CLIENT.query(
     `SELECT ARRAY[ARRAY[ARRAY_AGG(A), ARRAY_AGG(A)], ARRAY[ARRAY_AGG(A), ARRAY_AGG(A)]] FROM  (
       SELECT JSON_BUILD_OBJECT( 'X', '1' ) AS A

tests/encode.ts

@@ -63,9 +63,7 @@ test("encodeObject", function () {
 });
 
 test("encodeUint8Array", function () {
-  // deno-lint-ignore camelcase
   const buf_1 = new Uint8Array([1, 2, 3]);
-  // deno-lint-ignore camelcase
   const buf_2 = new Uint8Array([2, 10, 500]);
 
   assertEquals("\\x010203", encode(buf_1));

tests/helpers.ts

@@ -4,7 +4,7 @@ export function getTestClient(
   client: Client,
   defSetupQueries?: Array<string>,
 ) {
-  return function testClient(
+  return async function testClient(
     t: Deno.TestDefinition["fn"],
     setupQueries?: Array<string>,
   ) {

tests/pool.ts

@@ -3,7 +3,7 @@ import { Pool } from "../pool.ts";
 import { delay } from "../utils.ts";
 import { DEFAULT_SETUP, TEST_CONNECTION_PARAMS } from "./constants.ts";
 
-function testPool(
+async function testPool(
   t: (pool: Pool) => void | Promise<void>,
   setupQueries?: Array<string> | null,
   lazy?: boolean,
@@ -63,11 +63,9 @@ testPool(
     await p;
     assertEquals(POOL.available, 1);
 
-    // deno-lint-ignore camelcase
     const qs_thunks = [...Array(25)].map((_, i) =>
       POOL.query("SELECT pg_sleep(0.1) is null, $1::text as id;", i)
     );
-    // deno-lint-ignore camelcase
     const qs_promises = Promise.all(qs_thunks);
     await delay(1);
     assertEquals(POOL.available, 0);
@@ -103,11 +101,9 @@ testPool(async function manyQueries(POOL) {
   await p;
   assertEquals(POOL.available, 10);
 
-  // deno-lint-ignore camelcase
   const qs_thunks = [...Array(25)].map((_, i) =>
     POOL.query("SELECT pg_sleep(0.1) is null, $1::text as id;", i)
   );
-  // deno-lint-ignore camelcase
   const qs_promises = Promise.all(qs_thunks);
   await delay(1);
   assertEquals(POOL.available, 0);

utils.ts

@@ -73,7 +73,6 @@ export interface DsnResult {
 export function parseDsn(dsn: string): DsnResult {
   //URL object won't parse the URL if it doesn't recognize the protocol
   //This line replaces the protocol with http and then leaves it up to URL
-  // deno-lint-ignore camelcase
   const [protocol, stripped_url] = dsn.match(/(?:(?!:\/\/).)+/g) ?? ["", ""];
   const url = new URL(`http:${stripped_url}`);