Lint files

Author: Christopher Jones

lib/connection.js

@@ -223,7 +223,7 @@ function getDbObjectClass(name, cb) {
 }
 
 // This getStatementInfo function is just a place holder to allow for easier extension later.
-function getStatementInfo(sql, getStatementInfoCb) {
+function getStatementInfo(sql, getStatementInfoCb) { //eslint-disable-line
   const self = this;
 
   nodbUtil.checkAsyncArgs(arguments, 2, 2);
@@ -232,15 +232,15 @@ function getStatementInfo(sql, getStatementInfoCb) {
 }
 
 // This commit function is just a place holder to allow for easier extension later.
-function commit(commitCb) {
+function commit(commitCb) { //eslint-disable-line
   const self = this;
 
   nodbUtil.checkAsyncArgs(arguments, 1, 1);
   self._commit.apply(self, arguments);
 }
 
 // This createLob function is just a place holder to allow for easier extension later.
-function createLob(type, createLobCb) {
+function createLob(type, createLobCb) { //eslint-disable-line
   const self = this;
 
   nodbUtil.checkAsyncArgs(arguments, 2, 2);
@@ -249,7 +249,7 @@ function createLob(type, createLobCb) {
 }
 
 // This rollback function is just a place holder to allow for easier extension later.
-function rollback(rollbackCb) {
+function rollback(rollbackCb) { //eslint-disable-line
   const self = this;
 
   nodbUtil.checkAsyncArgs(arguments, 1, 1);
@@ -292,7 +292,7 @@ function close(a1, a2) {
 
 // This break function is just a place holder to allow for easier extension later.
 // It's attached to the module as break is a reserved word.
-module.break = function(breakCb) {
+module.break = function(breakCb) { //eslint-disable-line
   const self = this;
 
   nodbUtil.checkAsyncArgs(arguments, 1, 1);
@@ -334,7 +334,7 @@ function getQueue(name, a2, a3) {
 }
 
 // This ping function is just a place holder to allow for easier extension later.
-function ping(pingCb) {
+function ping(pingCb) { //eslint-disable-line
   const self = this;
 
   nodbUtil.checkAsyncArgs(arguments, 1, 1);
@@ -374,9 +374,9 @@ function buildDbObjectClass(schema, name, fqn) {
       }
       return proxy;
     } else if (initialValue !== undefined) {
-      Object.assign(this, initialValue)
+      Object.assign(this, initialValue);
     }
-  }
+  };
   DbObject.prototype = Object.create(BaseDbObject.prototype);
   DbObject.prototype.constructor = DbObject;
   DbObject.prototype.schema = schema;

lib/util.js

@@ -154,7 +154,7 @@ module.exports.assert = assert;
 // arguments is not within the given range, an error is thrown.
 function checkArgCount(args, minArgCount, maxArgCount) {
   if (args.length < minArgCount || args.length > maxArgCount)
-      throw new Error(getErrorMessage('NJS-009'));
+    throw new Error(getErrorMessage('NJS-009'));
 }
 
 module.exports.checkArgCount = checkArgCount;

test/accessPropertiesOnClosedObjects.js

@@ -54,13 +54,13 @@ describe('223. accessPropertiesOnClosedObjects.js', () => {
 
       await Lob.close();
       should.strictEqual(Lob.type, oracledb.DB_TYPE_BLOB);
-      should.strictEqual(Lob.length, 0)
+      should.strictEqual(Lob.length, 0);
 
       await conn.close();
       should.strictEqual(Lob.type, oracledb.DB_TYPE_BLOB);
-      should.strictEqual(Lob.length, 0)
+      should.strictEqual(Lob.length, 0);
     } catch (err) {
       should.not.exist(err);
     }
   }); // 223.2
-});
+});

test/connProps.js

@@ -28,7 +28,6 @@
 const oracledb  = require('oracledb');
 const should    = require('should');
 const dbconfig  = require('./dbconfig.js');
-const testsUtil = require('./testsUtil.js');
 
 describe('193. connProps.js', function() {
 
@@ -81,7 +80,7 @@ describe('193. connProps.js', function() {
       should.strictEqual(conn.clientInfo, null);
       should.strictEqual(conn.dbOp, null);
       await conn.close();
-    } catch (error) {
+    } catch (err) {
       should.not.exist(err);
     }
   }); // 193.2

test/runCQN.js

@@ -209,7 +209,7 @@ describe('185. runCQN.js', function() {
         should.strictEqual(table.name, tableName);
         let expect = oracledb.CQN_OPCODE_INSERT | oracledb.CQN_OPCODE_ALL_ROWS;
         should.strictEqual(table.operation, expect);
-      }
+      };
 
       const options = {
         callback : myCallback,
@@ -454,4 +454,4 @@ describe('185. runCQN.js', function() {
     }
   }); // 185.9
 
-});
+});