From bd44ca5eba27d91d0bbe483f42823ceb4cef740b Mon Sep 17 00:00:00 2001 From: Francesco Soncina Date: Mon, 8 Jan 2018 17:59:18 +0100 Subject: [PATCH] refactor: format error messages with getErrorMsg() --- packages/jest-jasmine2/src/jasmine/spy_registry.js | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/packages/jest-jasmine2/src/jasmine/spy_registry.js b/packages/jest-jasmine2/src/jasmine/spy_registry.js index 5427e811d6a1..905ae7ec8590 100644 --- a/packages/jest-jasmine2/src/jasmine/spy_registry.js +++ b/packages/jest-jasmine2/src/jasmine/spy_registry.js @@ -136,12 +136,14 @@ export default function SpyRegistry(options: Object) { this._spyOnProperty = function(obj, propertyName, accessType = 'get') { if (!obj) { throw new Error( - 'spyOn could not find an object to spy upon for ' + propertyName + '', + getErrorMsg( + 'could not find an object to spy upon for ' + propertyName, + ), ); } if (!propertyName) { - throw new Error('No property name supplied'); + throw new Error(getErrorMsg('No property name supplied')); } let descriptor; @@ -152,16 +154,18 @@ export default function SpyRegistry(options: Object) { } if (!descriptor) { - throw new Error(propertyName + ' property does not exist'); + throw new Error(getErrorMsg(propertyName + ' property does not exist')); } if (!descriptor.configurable) { - throw new Error(propertyName + ' is not declared configurable'); + throw new Error(getErrorMsg(propertyName + ' is not declared configurable')); } if (!descriptor[accessType]) { throw new Error( - 'Property ' + propertyName + ' does not have access type ' + accessType, + getErrorMsg( + 'Property ' + propertyName + ' does not have access type ' + accessType, + ) ); }