refactor: remove string concatenation (#188)

openapi-library · Oct 11, 2020 · 38d17b8 · 38d17b8
1 parent 3171a90
commit 38d17b8
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 11 deletions.
diff --git a/packages/chai-openapi-response-validator/lib/openapi-validator/lib/openApiSpecFactory.js b/packages/chai-openapi-response-validator/lib/openapi-validator/lib/openApiSpecFactory.js
@@ -30,8 +30,7 @@ function loadSpec(arg) {
     throw new Error(`Received type '${argType}'`);
   } catch (error) {
     throw new Error(
-      'The provided argument must be either an absolute filepath or ' +
-        `an object representing an OpenAPI specification.\nError details: ${error.message}`,
+      `The provided argument must be either an absolute filepath or an object representing an OpenAPI specification.\nError details: ${error.message}`,
     );
   }
 }

diff --git a/packages/chai-openapi-response-validator/test/setup.test.js b/packages/chai-openapi-response-validator/test/setup.test.js
@@ -21,8 +21,7 @@ const chaiResponseValidator = require('..');
 
 const { expect } = chai;
 const genericArgTypeErrMsg =
-  'The provided argument must be either an absolute filepath or' +
-  ' an object representing an OpenAPI specification.\nError details: ';
+  'The provided argument must be either an absolute filepath or an object representing an OpenAPI specification.\nError details: ';
 
 describe('chaiResponseValidator(pathToApiSpec)', () => {
   describe('neither string nor object', () => {

diff --git a/packages/jest-openapi/__test__/setup.test.js b/packages/jest-openapi/__test__/setup.test.js
@@ -20,8 +20,7 @@ const fs = require('fs-extra');
 const jestOpenAPI = require('..');
 
 const genericArgTypeErrMsg =
-  'The provided argument must be either an absolute filepath or' +
-  ' an object representing an OpenAPI specification.\nError details: ';
+  'The provided argument must be either an absolute filepath or an object representing an OpenAPI specification.\nError details: ';
 
 describe('jestOpenAPI(pathToApiSpec)', () => {
   describe('neither string nor object', () => {

diff --git a/packages/jest-openapi/src/index.js b/packages/jest-openapi/src/index.js
@@ -27,9 +27,11 @@ module.exports = function (filepathOrObject) {
   } else {
     // eslint-disable-next-line no-console
     console.error(
-      "Unable to find Jest's global expect." +
-        '\nPlease check you have configured jest-openapi correctly.' +
-        '\nSee https://github.com/RuntimeTools/OpenAPIValidators/jest-openapi#usage for help.',
+      [
+        "Unable to find Jest's global expect.",
+        'Please check you have configured jest-openapi correctly.',
+        'See https://github.com/RuntimeTools/OpenAPIValidators/jest-openapi#usage for help.',
+      ].join('\n'),
     );
   }
   /* istanbul ignore next */

diff --git a/packages/jest-openapi/src/openapi-validator/lib/openApiSpecFactory.js b/packages/jest-openapi/src/openapi-validator/lib/openApiSpecFactory.js
@@ -30,8 +30,7 @@ function loadSpec(arg) {
     throw new Error(`Received type '${argType}'`);
   } catch (error) {
     throw new Error(
-      'The provided argument must be either an absolute filepath or ' +
-        `an object representing an OpenAPI specification.\nError details: ${error.message}`,
+      `The provided argument must be either an absolute filepath or an object representing an OpenAPI specification.\nError details: ${error.message}`,
     );
   }
 }