Merge pull request #81 from trayio/anastasiia-siniukova/CSP-6106/upda…

…te-threadneedle-err-messages [CSP-6106] /update-threadneedle-err-messages
trayio · Mar 29, 2021 · 39e7b9d · 39e7b9d
2 parents bb6eca0 + 9ad4bdf
commit 39e7b9d
Show file tree

Hide file tree

Showing 7 changed files with 222 additions and 215 deletions.
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -6,9 +6,6 @@ module.exports = function (grunt) {
 		// Configure a mochaTest task
 		mochaTest: {
 			test: {
-				options: {
-					reporter: 'mocha-unfunk-reporter'
-				},
 				src: ['tests/*_test.js']
 			}
 		},

diff --git a/lib/addMethod/validateRESTInput.js b/lib/addMethod/validateRESTInput.js
@@ -6,15 +6,15 @@ module.exports = function (methodName, config) {
 
 	// Ensure the minimum parameters have been passed
 	if (!methodName || !_.isString(methodName)) {
-		throw new Error('The first parameter passed to `addMethod` should be a string.');
+		throw new Error(`The first parameter passed to 'addMethod' should be a string. Method: ${methodName}`);
 	}
 	// If a function is inputted as the `config`, then just return - there's
 	// really not much to validate.
 	if (_.isFunction(config)) {
 		return;
 	}
 	if (!config || !_.isObject(config)) {
-		throw new Error('The `config` object should be an object.');
+		throw new Error(`The 'config' object should be an object. Method: ${methodName}`);
 	}
 
 	// Check to see if the method has already been declared
@@ -24,7 +24,7 @@ module.exports = function (methodName, config) {
 
 	// Ensure the config parameters have been specified correctly
 	if (!config.url && config.url !== '') {
-		throw new Error('The `url` config parameter should be declared.');
+		throw new Error(`The 'url' config parameter should be declared. Method: ${methodName}`);
 	}
 
 	let method = config.method;
@@ -38,7 +38,7 @@ module.exports = function (methodName, config) {
 			}
 		}
 	} else if (!_.isFunction(method)) {
-		throw new Error('The `method` parameter needs to be provided in the method configuration.');
+		throw new Error(`The 'method' parameter needs to be provided in the method configuration. Method: ${methodName}`);
 	}
 
 };
diff --git a/lib/addMethod/validateSOAPInput.js b/lib/addMethod/validateSOAPInput.js
@@ -4,15 +4,15 @@ module.exports = function (methodName, config) {
 
     // Ensure the minimum parameters have been passed
     if (!methodName || !_.isString(methodName)) {
-        throw new Error('The first parameter passed to `addMethod` should be a string.');
+        throw new Error(`The first parameter passed to 'addMethod' should be a string. Method: ${methodName}`);
     }
     // If a function is inputted as the `config`, then just return - there's
     // really not much to validate.
     if (_.isFunction(config)) {
         return;
     }
     if (!config || !_.isObject(config)) {
-        throw new Error('The `config` object should be an object.');
+        throw new Error(`The 'config' object should be an object. Method: ${methodName}`);
     }
 
     // Check to see if the method has already been declared
@@ -22,7 +22,7 @@ module.exports = function (methodName, config) {
 
     // Ensure the config parameters have been specified correctly
     if (!config.method) {
-        throw new Error('The `method` config parameter should be declared.');
+        throw new Error(`The 'method' config parameter should be declared. Method: ${methodName}`);
     }
 
     if (!_.isUndefined(config.expects) && !_.isFunction(config.expects)) {