Skip to content
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

Removes sinon and chai, Migration to Jest matchers #103

Merged
merged 2 commits into from
Mar 18, 2019

Conversation

ayusharma
Copy link
Collaborator

@ayusharma ayusharma commented Feb 23, 2019

Reference issue: #88

What changed in this PR:

  • Removes chai and sinon
  • Migrate tests to use jest matchers.

Note: I am waiting #100 to be merged to update changelog.

@ayusharma ayusharma changed the title chore: removes sinon and chai and complete migration to Jest (#88) chore: Removes sinon and chai, Migration to Jest matchers Feb 23, 2019
@ayusharma ayusharma changed the title chore: Removes sinon and chai, Migration to Jest matchers Removes sinon and chai, Migration to Jest matchers Feb 23, 2019
@pago pago requested review from pago and krakenfuss February 26, 2019 04:47
@ayusharma ayusharma force-pushed the 88-Complete-switch-to-Jest-&-Snapshots branch from 39b7490 to 7a5d6f9 Compare February 26, 2019 10:34
@pago
Copy link
Contributor

pago commented Feb 27, 2019

@ayusharma would you be ok if we merge this after the stream API? We'd then need to also update the new tests added there but I'd rather prefer to avoid a conflict on that PR.

@ayusharma
Copy link
Collaborator Author

@pago Sure 👍 Better to merge after melody-streams.

@ayusharma ayusharma force-pushed the 88-Complete-switch-to-Jest-&-Snapshots branch from 7a5d6f9 to 6b611f2 Compare March 11, 2019 10:46
@pago
Copy link
Contributor

pago commented Mar 12, 2019

@ayusharma melody-streams is now in. I think we can proceed with this change. :)

@ayusharma
Copy link
Collaborator Author

@pago Yes. It can be merged to master :)

@pago
Copy link
Contributor

pago commented Mar 14, 2019

Did you migrate the tests within the new melody-streams package? I can't seem to spot them within the PR.

@ayusharma
Copy link
Collaborator Author

@pago Yes. Tests have been migrated to use Jest matchers by @krakenfuss in melody-streams.

@pago pago merged commit 4aa5e43 into trivago:master Mar 18, 2019
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