Undo unintended changes.

Author: arthurschreiber

test/integration/bulk-load-test.ts

@@ -1504,7 +1504,7 @@ describe('BulkLoad', function() {
       connection.execSqlBatch(request);
     });
 
-    it('should handle validation errors during streaming bulk loads', (done) => {
+    it('should handle validation errors during streaming bulk loads', function(done) {
       const bulkLoad = connection.newBulkLoad('#stream_test', completeBulkLoad);
       bulkLoad.addColumn('value', TYPES.Date, { nullable: false });
 
@@ -1522,7 +1522,7 @@ describe('BulkLoad', function() {
       }
     });
 
-    it('should allow reusing the connection after validation errors during streaming bulk loads', (done) => {
+    it('should allow reusing the connection after validation errors during streaming bulk loads', function(done) {
       const bulkLoad = connection.newBulkLoad('#stream_test', completeBulkLoad);
       bulkLoad.addColumn('value', TYPES.Date, { nullable: false });
 
@@ -1554,7 +1554,7 @@ describe('BulkLoad', function() {
     });
   });
 
-  it('should not throw in _transform function', (done) => {
+  it('should not throw in _transform function', function(done) {
     const bulkLoad = connection.newBulkLoad(
       '#tmpTestTable',
       (err, rowCount) => {

test/integration/connection-test.ts

@@ -1428,7 +1428,7 @@ update #tab1 set name = 'a3' where name like 'a%'\
     }
   });
 
-  it('should request timeout', (done) => {
+  it('should request timeout', function(done) {
     const request = new Request(
       "select 1 as C1;waitfor delay '00:00:05';select 2 as C2",
       function(err, rowCount, rows) {

test/unit/instance-lookup-test.ts

@@ -314,30 +314,30 @@ describe('InstanceLookup', function() {
 });
 
 describe('parseBrowserResponse', function() {
-  it('should find port when one instance matches', () => {
+  it('should find port when one instance matches', function() {
     const response =
       'ServerName;WINDOWS2;InstanceName;SQLEXPRESS;IsClustered;No;Version;10.50.2500.0;tcp;1433;;';
 
     assert.strictEqual(parseBrowserResponse(response, 'sqlexpress'), 1433);
   });
 
-  it('should find port in first instance when two instances exist', () => {
+  it('should find port in first instance when two instances exist', function() {
     const response =
       'ServerName;WINDOWS2;InstanceName;SQLEXPRESS;IsClustered;No;Version;10.50.2500.0;tcp;1433;;' +
       'ServerName;WINDOWS2;InstanceName;XXXXXXXXXX;IsClustered;No;Version;10.50.2500.0;tcp;0;;';
 
     assert.strictEqual(parseBrowserResponse(response, 'sqlexpress'), 1433);
   });
 
-  it('should find port in second instance when two instances exist', () => {
+  it('should find port in second instance when two instances exist', function() {
     const response =
       'ServerName;WINDOWS2;InstanceName;XXXXXXXXXX;IsClustered;No;Version;10.50.2500.0;tcp;0;;' +
       'ServerName;WINDOWS2;InstanceName;SQLEXPRESS;IsClustered;No;Version;10.50.2500.0;tcp;1433;;';
 
     assert.strictEqual(parseBrowserResponse(response, 'sqlexpress'), 1433);
   });
 
-  it('should return undefined when no instance matches', () => {
+  it('should return undefined when no instance matches', function() {
     const response =
       'ServerName;WINDOWS2;InstanceName;XXXXXXXXXX;IsClustered;No;Version;10.50.2500.0;tcp;0;;' +
       'ServerName;WINDOWS2;InstanceName;YYYYYYYYYY;IsClustered;No;Version;10.50.2500.0;tcp;0;;';

test/unit/tedious-test.ts

@@ -3,21 +3,21 @@ import { assert } from 'chai';
 
 describe('Tedious', function() {
 
-  it('should export TYPES', () => {
+  it('should export TYPES', function() {
     assert.ok(TYPES);
     assert.ok(TYPES.VarChar);
   });
 
-  it('should export ISOLATION_LEVEL', () => {
+  it('should export ISOLATION_LEVEL', function() {
     assert.ok(ISOLATION_LEVEL);
     assert.ok(ISOLATION_LEVEL.READ_UNCOMMITTED);
   });
 
-  it('should export Connection', () => {
+  it('should export Connection', function() {
     assert.ok(Connection);
   });
 
-  it('should not modify the passed config object', () => {
+  it('should not modify the passed config object', function() {
     const config = {
       server: 'localhost',
       userName: 'sa',

test/unit/token/colmetadata-token-parser-test.ts

@@ -48,7 +48,7 @@ describe('Colmetadata Token Parser', function() {
     });
   });
 
-  it('should parse Int column metadata', async () => {
+  it('should parse Int column metadata', async function() {
     const debug = new Debug();
     const numberOfColumns = 1;
     const userType = 2;
@@ -81,7 +81,7 @@ describe('Colmetadata Token Parser', function() {
     assert.isTrue((await parser.next()).done);
   });
 
-  it('should parse VarChar column metadata', async () => {
+  it('should parse VarChar column metadata', async function() {
     const debug = new Debug();
     const numberOfColumns = 1;
     const userType = 2;

test/unit/token/done-token-parser-test.ts

@@ -24,7 +24,7 @@ function parse(status: number, curCmd: number, doneRowCount: number) {
 }
 
 describe('Done Token Parser', () => {
-  it('should parse a final done token', async () => {
+  it('should parse a final done token', async function() {
     const status = 0x0000;
     const curCmd = 1;
     const doneRowCount = 2;
@@ -40,7 +40,7 @@ describe('Done Token Parser', () => {
     assert.isUndefined(token.rowCount);
   });
 
-  it('should parse a done token with more results', async () => {
+  it('should parse a done token with more results', async function() {
     const status = 0x0001;
     const curCmd = 1;
     const doneRowCount = 2;
@@ -56,7 +56,7 @@ describe('Done Token Parser', () => {
     assert.isUndefined(token.rowCount);
   });
 
-  it('should parse a done token with row count', async () => {
+  it('should parse a done token with row count', async function() {
     const status = 0x0010;
     const curCmd = 1;
     const doneRowCount = 0x1200000034;

test/unit/token/env-change-token-parser-test.ts

@@ -7,7 +7,7 @@ import { assert } from 'chai';
 const options = { tdsVersion: '7_2', useUTC: false } as ParserOptions;
 
 describe('Env Change Token Parser', () => {
-  it('should parse database change', async () => {
+  it('should parse database change', async function() {
     const debug = new Debug();
     const oldDb = 'old';
     const newDb = 'new';
@@ -34,7 +34,7 @@ describe('Env Change Token Parser', () => {
     assert.strictEqual(token.newValue, 'new');
   });
 
-  it('should parse packet size change', async () => {
+  it('should parse packet size change', async function() {
     const debug = new Debug();
     const oldSize = '1024';
     const newSize = '2048';
@@ -61,7 +61,7 @@ describe('Env Change Token Parser', () => {
     assert.strictEqual(token.newValue, 2048);
   });
 
-  it('should skip unknown env change types', async () => {
+  it('should skip unknown env change types', async function() {
     const debug = new Debug();
     const buffer = new WritableTrackingBuffer(50, 'ucs2');
 

test/unit/token/feature-ext-parser-test.ts

@@ -7,7 +7,7 @@ import { assert } from 'chai';
 const options = { tdsVersion: '7_2', useUTC: false } as ParserOptions;
 
 describe('Feature Ext Parser', () => {
-  it('should parse federated authentication token', async () => {
+  it('should parse federated authentication token', async function() {
     const debug = new Debug();
     const buffer = new WritableTrackingBuffer(50, 'ucs2');