Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

test(*): introduce the toEqualMinErr() custom Jasmine matcher #15216

Closed

Conversation

gkalpak
Copy link
Member

@gkalpak gkalpak commented Oct 5, 2016

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Test/Refactor.

Does this PR introduce a breaking change?
No.

Please check if the PR fulfills these requirements

Other information:
Discussed in #15213 (comment).

@mgol
Copy link
Member

mgol commented Oct 5, 2016

It seems the toThrowError to toThrowMinErr changes belong in a separate commit but otherwise LGTM.

@gkalpak
Copy link
Member Author

gkalpak commented Oct 5, 2016

True (I got lazy 😳)
I'll split them 😃

@gkalpak gkalpak force-pushed the test-add-toEqualMinErr-matcher branch from 2ad88b7 to a17ef89 Compare October 5, 2016 14:34
@gkalpak
Copy link
Member Author

gkalpak commented Oct 5, 2016

I split the commits up and added a 3rd commit with a few more replacements.

@gkalpak
Copy link
Member Author

gkalpak commented Oct 5, 2016

LGTM'd by @mgol

@gkalpak gkalpak closed this in e8aebb3 Oct 5, 2016
@gkalpak gkalpak deleted the test-add-toEqualMinErr-matcher branch October 5, 2016 16:15
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
ellimist pushed a commit to ellimist/angular.js that referenced this pull request Mar 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants