fix(check-password): correctly validate parameters, fix test

Author: mbroadst

lib/kerberos.js

@@ -61,10 +61,10 @@ function validateParameter(parameter, spec) {
  */
 function checkPassword(username, password, service, defaultRealm, callback) {
   if (typeof defaultRealm === 'function') (callback = defaultRealm), (defaultRealm = null);
-  validateParameter(username, { name: 'service', type: 'string' });
-  validateParameter(password, { name: 'options', type: 'string' });
-  validateParameter(service, { name: 'options', type: 'string' });
-  validateParameter(defaultRealm, { name: 'options', type: 'string' });
+  validateParameter(username, { name: 'username', type: 'string' });
+  validateParameter(password, { name: 'password', type: 'string' });
+  validateParameter(service, { name: 'service', type: 'string' });
+  validateParameter(defaultRealm, { name: 'defaultRealm', type: 'string' });
   validateParameter(callback, { name: 'callback', type: 'function' });
 
   kerberos.checkPassword(username, password, service, defaultRealm, callback);
@@ -82,7 +82,7 @@ function checkPassword(username, password, service, defaultRealm, callback) {
  */
 function principalDetails(service, hostname, callback) {
   validateParameter(service, { name: 'service', type: 'string' });
-  validateParameter(hostname, { name: 'options', type: 'string' });
+  validateParameter(hostname, { name: 'hostname', type: 'string' });
   validateParameter(callback, { name: 'callback', type: 'function' });
 
   kerberos.principalDetails(service, hostname, callback);

test/kerberos_tests.js

@@ -29,7 +29,7 @@ describe('Kerberos', function() {
     kerberos.checkPassword(username, password, service, realm.toUpperCase(), err => {
       expect(err).to.not.exist;
 
-      kerberos.checkPassword(username, 'incorrect-password', service, realm.toUpperCase, err => {
+      kerberos.checkPassword(username, 'incorrect-password', service, realm.toUpperCase(), err => {
         expect(err).to.exist;
         done();
       });