Test coverage improvements

We're now at 100% line and conditional coverage (minus skipped lines and
conditions that represent unknown or erroneous states).

Author: txase

index.js

@@ -15,9 +15,9 @@ function getAuroraDataValue (value) {
     return value.longValue;
   } else if ('stringValue' in value) {
     return value.stringValue;
-  } else if ('booleanValue' in value) {
+  } else /* istanbul ignore else */ if ('booleanValue' in value) {
     return value.booleanValue;
-  } else /* istanbul ignore next */ {
+  } else {
     const type = Object.keys(value)[0];
     throw new Error(`Unknown value type '${type}' from row`);
   }

tests/constants.js

@@ -2122,6 +2122,12 @@ module.exports = {
   },
   PLUCK_RESPONSE_ROWS: [1, 2],
 
+  DEL_RESPONSE_DATA: {
+    generatedFields: [],
+    numberOfRecordsUpdated: 7
+  },
+  DEL_RESPONSE_ROWS: 7,
+
   INSERT_RESPONSE_DATA: {
     generatedFields: [
       {

tests/test.js

@@ -103,6 +103,27 @@ describe('SDK configuration tests', () => {
   });
 });
 
+test('Destroy functionality', async () => {
+  const knex = require('knex')({
+    client: require('..'),
+    connection: {
+      database: constants.DATABASE,
+      resourceArn: constants.AURORA_CLUSTER_ARN,
+      secretArn: constants.SECRET_ARN
+    }
+  });
+
+  mockExecuteStatementPromise.mockResolvedValue(constants.ALL_QUERY_RESPONSE_DATA);
+
+  await knex.select('*').from('foo');
+
+  expect(mockExecuteStatement).toHaveBeenCalledTimes(1);
+
+  knex.destroy();
+
+  await expect(knex.select('*').from('foo')).rejects.toThrow();
+});
+
 describe('Query statement tests', () => {
   const knex = require('knex')({
     client: require('..'),
@@ -300,6 +321,24 @@ describe('Query statement tests', () => {
     expect(rows).toEqual(constants.PLUCK_RESPONSE_ROWS);
   });
 
+  test('.del() works', async () => {
+    mockExecuteStatementPromise.mockResolvedValue(constants.DEL_RESPONSE_DATA);
+
+    const rows = await knex('test').del();
+
+    expect(mockExecuteStatement).toHaveBeenCalledTimes(1);
+    expect(mockExecuteStatement).toHaveBeenCalledWith({
+      resourceArn: constants.AURORA_CLUSTER_ARN,
+      secretArn: constants.SECRET_ARN,
+      database: constants.DATABASE,
+      sql: 'delete from `test`',
+      parameters: [],
+      includeResultMetadata: true
+    });
+
+    expect(rows).toEqual(constants.DEL_RESPONSE_ROWS);
+  });
+
   test("Insert returns first row's primary ID", async () => {
     mockExecuteStatementPromise.mockResolvedValue(constants.INSERT_RESPONSE_DATA);