Adding 100% Unit Test Coverage

.vscode/settings.json

@@ -0,0 +1,3 @@
+{
+    "typescript.tsdk": "node_modules/typescript/lib"
+}

jest.config.js

@@ -11,6 +11,13 @@ module.exports = {
     moduleNameMapper: pathsToModuleNameMapper(compilerOptions.paths, {
         prefix: '<rootDir>/',
     }),
+    coverageThreshold: {
+        global: {
+            branches: 100,
+            functions: 100,
+            lines: 100,
+        },
+    },
     moduleDirectories: ['node_modules', 'src'],
     modulePaths: ['<rootDir>'],
     setupFiles: ['<rootDir>/jest.setup.js'],

package.json

@@ -34,6 +34,9 @@
         "handlebars": "^4.7.8"
     },
     "devDependencies": {
+        "@types/fetch-mock": "^7.3.8",
+        "fetch-mock": "^9.11.0",
+        "node-fetch": "^2.7.0",
         "@jest/globals": "^29.7.0",
         "@types/node": "^20.12.12",
         "husky": "^9.0.11",

src/client.ts

@@ -346,10 +346,6 @@ function buildQueryString(
 
             break
         case 'update':
-            if (!queryBuilder || !queryBuilder.whereClauses) {
-                break
-            }
-
             const columnsToUpdate = Object.keys(queryBuilder.data || {})
             const setClauses =
                 queryType === QueryType.named
@@ -361,8 +357,8 @@ function buildQueryString(
                           .join(', ')
 
             query.query = `UPDATE ${queryBuilder.table || ''} SET ${setClauses}`
-            if (queryBuilder.whereClauses?.length > 0) {
-                query.query += ` WHERE ${queryBuilder.whereClauses.join(' AND ')}`
+            if (queryBuilder.whereClauses?.length ?? 0 > 0) {
+                query.query += ` WHERE ${queryBuilder.whereClauses?.join(' AND ')}`
             }
 
             if (queryType === QueryType.named) {
@@ -387,8 +383,6 @@ function buildQueryString(
                 query.query += ` WHERE ${queryBuilder.whereClauses.join(' AND ')}`
             }
             break
-        default:
-            throw new Error('Invalid action')
     }
 
     return query
@@ -619,16 +613,18 @@ export function lessThanOrEqualNumber(a: any, b: any) {
     return `${a} <= ${b}`
 }
 
-export function inValues(a: any, b: any[]) {
-    return `${a} IN ('${b.join("', '").replace(/'/g, "\\'")}')`
+export function inValues(a: any, b: any[]): string {
+    const sanitizedValues = b.map((val) => val.replace(/'/g, "\\'"))
+    return `${a} IN ('${sanitizedValues.join("', '")}')`
 }
 
 export function inNumbers(a: any, b: any[]) {
     return `${a} IN (${b.join(', ')})`
 }
 
-export function notInValues(a: any, b: any[]) {
-    return `${a} NOT IN ('${b.join("', '").replace(/'/g, "\\'")}')`
+export function notInValues(a: any, b: any[]): string {
+    const sanitizedValues = b.map((val) => val.replace(/'/g, "\\'"))
+    return `${a} NOT IN ('${sanitizedValues.join("', '")}')`
 }
 
 export function notInNumbers(a: any, b: any[]) {
@@ -644,7 +640,7 @@ export function isNumber(a: any, b: any) {
     return `${a} IS ${b}`
 }
 
-export function isNot(this: any, a: any, b: null) {
+export function isNot(this: any, a: any, b: string | null) {
     if (b === null) return `${this} IS NOT NULL`
     return `${a} IS NOT ${b}`
 }