DRY up some sopping wet code
This commit is contained in:
@@ -6,7 +6,9 @@ getJasmineRequireObj().toThrowError = function(j$) {
|
||||
thrown,
|
||||
errorType,
|
||||
message,
|
||||
regexp;
|
||||
regexp,
|
||||
name,
|
||||
constructorName;
|
||||
|
||||
if (typeof actual != "function") {
|
||||
throw new Error("Actual is not a Function");
|
||||
@@ -33,10 +35,12 @@ getJasmineRequireObj().toThrowError = function(j$) {
|
||||
return pass("Expected function not to throw an Error, but it threw " + thrown + ".");
|
||||
}
|
||||
|
||||
if (errorType && message) {
|
||||
var name = errorType.name || errorType.toString().match(/^\s*function\s*(\w*)\s*\(/)[1];
|
||||
var constructorName = thrown.constructor.name || thrown.constructor.toString().match(/^\s*function\s*(\w*)\s*\(/)[1];
|
||||
if (errorType) {
|
||||
name = fnNameFor(errorType);
|
||||
constructorName = fnNameFor(thrown.constructor);
|
||||
}
|
||||
|
||||
if (errorType && message) {
|
||||
if (thrown.constructor == errorType && util.equals(thrown.message, message)) {
|
||||
return pass("Expected function not to throw " + name + " with message \"" + message + "\".");
|
||||
} else {
|
||||
@@ -46,9 +50,6 @@ getJasmineRequireObj().toThrowError = function(j$) {
|
||||
}
|
||||
|
||||
if (errorType && regexp) {
|
||||
var name = errorType.name || errorType.toString().match(/^\s*function\s*(\w*)\s*\(/)[1];
|
||||
var constructorName = thrown.constructor.name || thrown.constructor.toString().match(/^\s*function\s*(\w*)\s*\(/)[1];
|
||||
|
||||
if (thrown.constructor == errorType && regexp.test(thrown.message)) {
|
||||
return pass("Expected function not to throw " + name + " with message matching " + regexp + ".");
|
||||
} else {
|
||||
@@ -58,10 +59,6 @@ getJasmineRequireObj().toThrowError = function(j$) {
|
||||
}
|
||||
|
||||
if (errorType) {
|
||||
var name = errorType.name || errorType.toString().match(/^\s*function\s*(\w*)\s*\(/)[1];
|
||||
var constructorName = thrown.constructor.name || thrown.constructor.toString().match(/^\s*function\s*(\w*)\s*\(/)[1];
|
||||
|
||||
|
||||
if (thrown.constructor == errorType) {
|
||||
return pass("Expected function not to throw " + name + ".");
|
||||
} else {
|
||||
@@ -87,6 +84,10 @@ getJasmineRequireObj().toThrowError = function(j$) {
|
||||
}
|
||||
}
|
||||
|
||||
function fnNameFor(func) {
|
||||
return func.name || func.toString().match(/^\s*function\s*(\w*)\s*\(/)[1];
|
||||
}
|
||||
|
||||
function pass(notMessage) {
|
||||
return {
|
||||
pass: true,
|
||||
@@ -151,4 +152,4 @@ getJasmineRequireObj().toThrowError = function(j$) {
|
||||
}
|
||||
|
||||
return toThrowError;
|
||||
};
|
||||
};
|
||||
|
||||
Reference in New Issue
Block a user