admin/jasmine
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Removed useless passed property from ThrowUnlessFailure

Browse Source
This commit is contained in:
Steve Gravrock
2025-09-20 16:17:02 -07:00
parent 1e98a4b61b
commit d333ecb5b1
3 changed files with 0 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/jasmine-core/jasmine.js
View File

@@ -1286,10 +1286,8 @@ getJasmineRequireObj().Env = function(j$) {
* *
* @property {String} matcherName - The name of the matcher that was executed for this expectation. * @property {String} matcherName - The name of the matcher that was executed for this expectation.
* @property {String} message - The failure message for the expectation. * @property {String} message - The failure message for the expectation.
* @property {Boolean} passed - Whether the expectation passed or failed.
*/ */
const error = new Error(result.message); const error = new Error(result.message);
error.passed = result.passed;
error.message = result.message; error.message = result.message;
error.matcherName = result.matcherName; error.matcherName = result.matcherName;
throw error; throw error;

2
spec/core/integration/EnvSpec.js
View File

@@ -3699,7 +3699,6 @@ describe('Env integration', function() {
await env.execute(); await env.execute();
expect(thrown).toBeInstanceOf(Error); expect(thrown).toBeInstanceOf(Error);
expect(thrown.passed).toEqual(false);
expect(thrown.matcherName).toEqual('toEqual'); expect(thrown.matcherName).toEqual('toEqual');
expect(thrown.message).toEqual('Expected 1 to equal 2.'); expect(thrown.message).toEqual('Expected 1 to equal 2.');
}); });
@@ -3753,7 +3752,6 @@ describe('Env integration', function() {
await env.execute(); await env.execute();
expect(thrown).toBeInstanceOf(Error); expect(thrown).toBeInstanceOf(Error);
expect(thrown.passed).toEqual(false);
expect(thrown.matcherName).toEqual('toBeResolvedTo'); expect(thrown.matcherName).toEqual('toBeResolvedTo');
expect(thrown.message).toEqual( expect(thrown.message).toEqual(
"Expected a promise to be resolved to 'b' but it was resolved to 'a'." "Expected a promise to be resolved to 'b' but it was resolved to 'a'."

2
src/core/Env.js
View File

@@ -160,10 +160,8 @@ getJasmineRequireObj().Env = function(j$) {
* *
* @property {String} matcherName - The name of the matcher that was executed for this expectation. * @property {String} matcherName - The name of the matcher that was executed for this expectation.
* @property {String} message - The failure message for the expectation. * @property {String} message - The failure message for the expectation.
* @property {Boolean} passed - Whether the expectation passed or failed.
*/ */
const error = new Error(result.message); const error = new Error(result.message);
error.passed = result.passed;
error.message = result.message; error.message = result.message;
error.matcherName = result.matcherName; error.matcherName = result.matcherName;
throw error; throw error;