[ #154578064 ] - removed unused variables and some updates

Author: denshikov-vovan

tests/connection.spec.ts

@@ -9,7 +9,7 @@ import {settings} from './settings';
 
 // OBJECT QUERY TESTS
 // ================================================================================================
-describe('Object query tests', function () {
+describe('Object query tests;', function () {
 
     it('Object query should return a single object', () => {
         return new Database(settings).connect().then((session) => {
@@ -218,7 +218,7 @@ describe('Object query tests', function () {
 
 // LIST QUERY TESTS
 // ================================================================================================
-describe('List query tests', function () {
+describe('List query tests;', function () {
 
     it('List query should return an array of objects', () => {
         return new Database(settings).connect().then((session) => {
@@ -411,7 +411,7 @@ describe('List query tests', function () {
 
 // NON-RESULT QUERY TESTS
 // ================================================================================================
-describe('Non-result query tests', function () {
+describe('Non-result query tests;', function () {
 
     it('A non-result query should produce no results', () => {
         return new Database(settings).connect().then((session) => {
@@ -442,7 +442,7 @@ describe('Non-result query tests', function () {
 
 // MIXED QUERY TESTS
 // ================================================================================================
-describe('Mixed query tests', function () {
+describe('Mixed query tests;', function () {
 
     it('Multiple mixed queries should produce a Map of results', () => {
         return new Database(settings).connect().then((session) => {
@@ -557,7 +557,7 @@ describe('Mixed query tests', function () {
 
 // PARAMETRIZED QUERY TESTS
 // ================================================================================================
-describe('Parametrized query tests', function () {
+describe('Parametrized query tests;', function () {
 
     it('Object query parametrized with number should retrive correct row', () => {
         return new Database(settings).connect().then((session) => {
@@ -728,7 +728,7 @@ describe('Parametrized query tests', function () {
 
 // SESSION LIFECYCLE TESTS
 // ================================================================================================
-describe('Session lifecycle tests', function () {
+describe('Session lifecycle tests;', function () {
 
     it('Closing a session should return a connection back to the pool', () => {
         const database = new Database(settings);
@@ -861,7 +861,7 @@ describe('Session lifecycle tests', function () {
 
 // ERROR CONDITION TESTS
 // ================================================================================================
-describe('Error condition tests', function () {
+describe('Error condition tests;', function () {
 
     it('Query execution error should close the session and release the connection back to the pool', () => {
         const database = new Database(settings);

tests/connectionTimeout.spec.ts

@@ -6,7 +6,7 @@ import {ConnectionError} from '../lib/errors';
 import {settings} from './settings';
 import {buildLogger} from '../lib/util';
 
-describe('connection timeout;', () => {
+describe('Pool connection timeout;', () => {
     before(done => {
         this.server = net.createServer();
 

tests/ending.spec.ts

@@ -1,15 +1,15 @@
 import {expect} from 'chai';
 
 import {Client} from 'pg';
-import {ConnectionPool, PoolOptions} from '../lib/Pool';
+import {ConnectionPool} from '../lib/Pool';
 import {buildLogger} from '../lib/util';
 import {settings} from './settings';
 
 const createNewPool = (): ConnectionPool => {
     return new ConnectionPool(settings.pool, settings.connection, buildLogger('test'));
 };
 
-describe('pool ending;', () => {
+describe('Pool ending;', () => {
     it('ends without being used', (done) => {
         const pool = createNewPool();
 

tests/errorHandling.spec.ts

@@ -2,7 +2,6 @@ import {expect} from 'chai';
 
 import {Client, QueryResult} from 'pg';
 import {ConnectionPool, PoolOptions} from '../lib/Pool';
-import {ConnectionError} from '../lib/errors';
 import {settings} from './settings';
 import {buildLogger} from '../lib/util';
 
@@ -12,7 +11,7 @@ const createNewPool = (options?: PoolOptions): ConnectionPool => {
     return new ConnectionPool(poolSettings, settings.connection, buildLogger('test'));
 };
 
-describe('pool error handling;', () => {
+describe('Pool error handling;', () => {
     it('Should complete these queries without dying', done => {
         const iterations = 5;
         const pool = createNewPool();
@@ -23,7 +22,7 @@ describe('pool error handling;', () => {
         const runErrorQuery = (client: Client): Promise<QueryResult> => {
             shouldGet++;
 
-            return new Promise(function (resolve, reject) {
+            return new Promise(resolve => {
                 client.query('SELECT \'asd\'+1 ')
                     .catch(err => {
                         errors++;
@@ -138,7 +137,6 @@ describe('pool error handling;', () => {
         const iterations = 20;
         const queryValue = 'brianc';
         const pool = createNewPool({maxSize: 1});
-        const promises = Promise.resolve();
         const errors = [];
 
         const errorHandler = (err, cb: Function): void => {

tests/events.spec.ts

@@ -1,17 +1,14 @@
 import {expect} from 'chai';
-import {EventEmitter} from 'events';
 
-import {Client, QueryResult} from 'pg';
-import {ConnectionPool, PoolOptions, ERROR_EVENT, CREATED_EVENT, CLOSED_EVENT} from '../lib/Pool';
-import {ConnectionError} from '../lib/errors';
+import {ConnectionPool, ERROR_EVENT, CREATED_EVENT, CLOSED_EVENT} from '../lib/Pool';
 import {settings} from './settings';
 import {buildLogger} from '../lib/util';
 
 const createNewPool = (): ConnectionPool => {
     return new ConnectionPool(settings.pool, settings.connection, buildLogger('test'));
 };
 
-describe('events;', () => {
+describe('Pool Events;', () => {
     it('emits acquire before callback', done => {
         const pool = createNewPool();
         let emittedClient;