Matchers can have a negativeCompare
- Passing in a 'negativeCompare' will cause that function to be used when it is a 'not' assertion - Otherwise, the reversal of the compare's result will be used instead [finishes #59703824]
This commit is contained in:
@@ -56,6 +56,28 @@ describe("Custom Matchers (Integration)", function() {
|
|||||||
env.execute();
|
env.execute();
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it("uses the negative compare function for a negative comparison, if provided", function(done) {
|
||||||
|
env.addMatchers({
|
||||||
|
toBeReal: function() {
|
||||||
|
return {
|
||||||
|
compare: function() { return { pass: true }; },
|
||||||
|
negativeCompare: function() { return { pass: true }; }
|
||||||
|
};
|
||||||
|
}
|
||||||
|
});
|
||||||
|
|
||||||
|
env.it("spec with custom negative comparison matcher", function() {
|
||||||
|
env.expect(true).not.toBeReal();
|
||||||
|
});
|
||||||
|
|
||||||
|
var specExpectations = function(result) {
|
||||||
|
expect(result.status).toEqual('passed');
|
||||||
|
}
|
||||||
|
|
||||||
|
env.addReporter({ specDone: specExpectations, jasmineDone: done });
|
||||||
|
env.execute();
|
||||||
|
});
|
||||||
|
|
||||||
it("generates messages with the same rules as built in matchers absent a custom message", function(done) {
|
it("generates messages with the same rules as built in matchers absent a custom message", function(done) {
|
||||||
env.addMatchers({
|
env.addMatchers({
|
||||||
toBeReal: function() {
|
toBeReal: function() {
|
||||||
|
|||||||
@@ -317,4 +317,77 @@ describe("Expectation", function() {
|
|||||||
message: "I am a custom message"
|
message: "I am a custom message"
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it("reports a passing result to the spec when the 'not' comparison passes, given a negativeCompare", function() {
|
||||||
|
var matchers = {
|
||||||
|
toFoo: function() {
|
||||||
|
return {
|
||||||
|
compare: function() { return { pass: true }; },
|
||||||
|
negativeCompare: function() { return { pass: true }; }
|
||||||
|
};
|
||||||
|
}
|
||||||
|
},
|
||||||
|
addExpectationResult = jasmine.createSpy("addExpectationResult"),
|
||||||
|
actual = "an actual",
|
||||||
|
expectation;
|
||||||
|
|
||||||
|
j$.Expectation.addMatchers(matchers);
|
||||||
|
|
||||||
|
expectation = new j$.Expectation({
|
||||||
|
matchers: matchers,
|
||||||
|
actual: "an actual",
|
||||||
|
addExpectationResult: addExpectationResult,
|
||||||
|
isNot: true
|
||||||
|
});
|
||||||
|
|
||||||
|
expectation.toFoo("hello");
|
||||||
|
|
||||||
|
expect(addExpectationResult).toHaveBeenCalledWith(true, {
|
||||||
|
matcherName: "toFoo",
|
||||||
|
passed: true,
|
||||||
|
expected: "hello",
|
||||||
|
actual: actual,
|
||||||
|
message: ""
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
it("reports a failing result and a custom fail message to the spec when the 'not' comparison fails, given a negativeCompare", function() {
|
||||||
|
var matchers = {
|
||||||
|
toFoo: function() {
|
||||||
|
return {
|
||||||
|
compare: function() { return { pass: true }; },
|
||||||
|
negativeCompare: function() {
|
||||||
|
return {
|
||||||
|
pass: false,
|
||||||
|
message: "I'm a custom message"
|
||||||
|
};
|
||||||
|
}
|
||||||
|
};
|
||||||
|
}
|
||||||
|
},
|
||||||
|
addExpectationResult = jasmine.createSpy("addExpectationResult"),
|
||||||
|
actual = "an actual",
|
||||||
|
expectation;
|
||||||
|
|
||||||
|
j$.Expectation.addMatchers(matchers);
|
||||||
|
|
||||||
|
expectation = new j$.Expectation({
|
||||||
|
matchers: matchers,
|
||||||
|
actual: "an actual",
|
||||||
|
addExpectationResult: addExpectationResult,
|
||||||
|
isNot: true
|
||||||
|
});
|
||||||
|
|
||||||
|
expectation.toFoo("hello");
|
||||||
|
|
||||||
|
expect(addExpectationResult).toHaveBeenCalledWith(false, {
|
||||||
|
matcherName: "toFoo",
|
||||||
|
passed: false,
|
||||||
|
expected: "hello",
|
||||||
|
actual: actual,
|
||||||
|
message: "I'm a custom message"
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|||||||
@@ -22,12 +22,21 @@ getJasmineRequireObj().Expectation = function() {
|
|||||||
|
|
||||||
args.unshift(this.actual);
|
args.unshift(this.actual);
|
||||||
|
|
||||||
var result = matcherFactory(this.util, this.customEqualityTesters).compare.apply(null, args);
|
var matcher = matcherFactory(this.util, this.customEqualityTesters),
|
||||||
|
matcherCompare = matcher.compare;
|
||||||
|
|
||||||
|
function defaultNegativeCompare() {
|
||||||
|
var result = matcher.compare.apply(null, args);
|
||||||
|
result.pass = !result.pass;
|
||||||
|
return result;
|
||||||
|
}
|
||||||
|
|
||||||
if (this.isNot) {
|
if (this.isNot) {
|
||||||
result.pass = !result.pass;
|
matcherCompare = matcher.negativeCompare || defaultNegativeCompare;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
var result = matcherCompare.apply(null, args);
|
||||||
|
|
||||||
if (!result.pass) {
|
if (!result.pass) {
|
||||||
if (!result.message) {
|
if (!result.message) {
|
||||||
args.unshift(this.isNot);
|
args.unshift(this.isNot);
|
||||||
|
|||||||
Reference in New Issue
Block a user