New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade to samsam 3 #1955
Upgrade to samsam 3 #1955
Conversation
I love PRs like these! |
Pull Request Test Coverage Report for Build 2766
💛 - Coveralls |
Samsam 3 was released and this is passing all tests in all environments. Ready for review. It was required to update We could also wait for the next major version of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have verified this locally
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lib/sinon/match.js
seems like a candidate for removal.
@fearphage Thank you for reviewing. You're absolutely right. I kept the tests and |
* Upgrade to samsam 3 * Remove obsolete tests and match.js
Purpose (TL;DR) - mandatory
WORK IN PROGRESS! DO NOT MERGEThis PR integrates the changes from sinonjs/samsam#49 with Sinon.
Since samsam 3 is not yet release, the build is failing.See intructions below on how to verify locally.
Changes
lib/sinon/util/core/deep-equal.js
is replaced withsamsam.deepEqual
.lib/match.js
now simply exposessamsam.createMatcher
.lib/sinon/util/core/every.js
lib/sinon/util/core/iterable-to-string.js
lib/sinon/util/core/typeOf.js
lodash.get
type-detect
samsam.deepEqual
has changed from(expected, actual)
to(actual, expected)
How to verify - mandatory
npm install
npm test
Checklist for author
npm run lint
passes