larvituser 0.19.0 → 0.19.1

package/index.js

@@ -344,6 +344,9 @@ User.prototype.fromUserAndPass = function fromUserAndPass(username, password, cb
 		userUuid,
 		userObj;
 
+	if (typeof username !== 'string') return cb(new Error('Username must be a string'), false);
+	if (typeof password !== 'string') return cb(new Error('Password must be a string'), false);
+
 	username	= username.trim();
 	password	= password.trim();
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "larvituser",
-  "version": "0.19.0",
+  "version": "0.19.1",
   "author": {
     "name": "Mikael 'Lilleman' Göransson",
     "email": "lilleman@larvit.se",
@@ -19,7 +19,7 @@
   "description": "User module for node.js",
   "devDependencies": {
     "freeport": "1.0.5",
-    "mocha": "6.1.4",
+    "mocha": "6.2.2",
     "mocha-eslint": "5.0.0"
   },
   "scripts": {

package/test/00test.js

@@ -53,6 +53,11 @@ before(function (done) {
 		});
 	});
 
+	// Clear DB
+	tasks.push(function (cb) {
+		db.removeAllTables(cb);
+	});
+
 	// Check for empty db
 	tasks.push(function (cb) {
 		db.query('SHOW TABLES', function (err, rows) {
@@ -202,6 +207,23 @@ describe('User', function () {
 			});
 		});
 
+		it('should fail to log in user if username is an array', function (done) {
+			userLib.fromUserAndPass(['lill', 'eman'], 'foobar', function (err, user) {
+				assert(err.message === 'Username must be a string', 'Error should be "Username must be a string"');
+				assert(user === false, 'user should be false');
+				done();
+			});
+		});
+
+
+		it('should fail to log in user if password is an array', function (done) {
+			userLib.fromUserAndPass('lilleman', ['foo', 'bar'], function (err, user) {
+				assert(err.message === 'Password must be a string', 'Error should be "Password must be a string"');
+				assert(user === false, 'user should be false');
+				done();
+			});
+		});
+
 		it('should log in user by field', function (done) {
 			userLib.fromField('firstname', 'migal', function (err, user) {
 				if (err) throw err;