Skip to content

[Fix] notEqual and notDeepEqual show "expected" value on failure.#454

Merged
ljharb merged 1 commit intotape-testing:masterfrom
JoshuaGrams:master
Dec 28, 2018
Merged

[Fix] notEqual and notDeepEqual show "expected" value on failure.#454
ljharb merged 1 commit intotape-testing:masterfrom
JoshuaGrams:master

Conversation

@JoshuaGrams
Copy link
Copy Markdown
Contributor

Closes #453.

Copy link
Copy Markdown
Collaborator

@ljharb ljharb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It'd be great to include regression tests with this, so it doesn't break in the future.

@JoshuaGrams
Copy link
Copy Markdown
Contributor Author

OK, try this. But you had better check over my code: I just copied test/deep-equal-failure.js and adapted it: I'm not sure I fully understand how it works.

Copy link
Copy Markdown
Collaborator

@ljharb ljharb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks awesome, thanks!

@ljharb ljharb merged commit 9ced991 into tape-testing:master Dec 28, 2018
ljharb added a commit that referenced this pull request Dec 29, 2018
 - [Fix] notEqual and notDeepEqual show "expected" value on failure (#454)
 - [Docs] Updated readme to make test, test.only, and test.skip consistent (#452)
 - [Docs] Clarify doesNotThrow parameters (#450)
 - [Docs] Adding tap-junit (#449)
 - [readme] Change broken image to use web archive (#443)
 - [Docs] convert list of tap reporters to links (#439, #440)
 - [Dev Deps] update `eslint`, `eclint`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants