From 13e4adf18f77ed89ba62cb4c9d8ab7ec89e1ee8b Mon Sep 17 00:00:00 2001 From: Joe-Dowd Date: Fri, 10 Jul 2020 01:46:09 +0100 Subject: [PATCH] PR comments resolved --- src/databases/Mysql.js | 2 +- src/databases/Sqlite.js | 5 ++--- test/cases/getSavedTimeForUser.js | 2 +- 3 files changed, 4 insertions(+), 5 deletions(-) diff --git a/src/databases/Mysql.js b/src/databases/Mysql.js index bc6e79f..b1c8fc1 100644 --- a/src/databases/Mysql.js +++ b/src/databases/Mysql.js @@ -10,7 +10,7 @@ class Mysql { } prepare (type, query, params) { - console.log("prepare (mysql): type: " + type + ", query: " + query + ", params: " + params); + (config.mode === "development") && console.log("prepare (mysql): type: " + type + ", query: " + query + ", params: " + params); if (type === 'get') { return this.connection.query(query, params)[0]; } else if (type === 'run') { diff --git a/src/databases/Sqlite.js b/src/databases/Sqlite.js index 422b9ea..c043656 100644 --- a/src/databases/Sqlite.js +++ b/src/databases/Sqlite.js @@ -10,7 +10,6 @@ class Sqlite { } prepare(type, query, params) { - //console.log("prepare: type: " + type + ", query: " + query + ", params: " + params); if (type === 'get') { return this.connection.prepare(query).get(...params); } else if (type === 'run') { @@ -18,8 +17,8 @@ class Sqlite { } else if (type === 'all') { return this.connection.prepare(query).all(...params); } else { - console.log('returning undefined...') - console.log("prepare: type: " + type + ", query: " + query + ", params: " + params); + (config.mode === "development") && console.log('returning undefined...') + (config.mode === "development") && console.log("prepare: type: " + type + ", query: " + query + ", params: " + params); return undefined; } } diff --git a/test/cases/getSavedTimeForUser.js b/test/cases/getSavedTimeForUser.js index 45b8166..5df8a48 100644 --- a/test/cases/getSavedTimeForUser.js +++ b/test/cases/getSavedTimeForUser.js @@ -1,6 +1,6 @@ var request = require('request'); var utils = require('../utils.js'); -var db = require('../../src/databases/databases.js').db.getConnection(); +var db = require('../../src/databases/databases.js').db; var getHash = require('../../src/utils/getHash.js'); describe('getSavedTimeForUser', () => {