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

test(jest-message-util): add test for codeframe and snapshot test #5103

Merged
merged 3 commits into from
Dec 18, 2017

Conversation

SimenB
Copy link
Member

@SimenB SimenB commented Dec 18, 2017

Summary
This is just adding a regression test. This was broken and fixed in #5094, but I would like to have a test for it so we don't regress

Test plan
Green CI

@@ -2620,6 +2620,12 @@ errno@^0.1.1, errno@~0.1.1:
dependencies:
Copy link
Member Author

Choose a reason for hiding this comment

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

All changes in here come from running yarn after the 22 update

@@ -173,8 +173,8 @@ test('my mocking test', () => {
Jest 21 was released back in September, and we unfortunately never got around to
write a blog post. So here is a quick summary of the main changes in version 21:

1. **Use expect and jest-mock in the browser:
**[Michael Jackson](https://github.com/mjackson) donated his excellent
1. **Use expect and jest-mock in the browser:**
Copy link
Member Author

Choose a reason for hiding this comment

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

ninja fix

Copy link
Collaborator

Choose a reason for hiding this comment

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

sneaky

@codecov-io
Copy link

Codecov Report

Merging #5103 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #5103   +/-   ##
======================================
  Coverage    60.6%   60.6%           
======================================
  Files         201     201           
  Lines        6688    6688           
  Branches        4       4           
======================================
  Hits         4053    4053           
  Misses       2635    2635

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 65ba1d1...e6b645a. Read the comment docs.

@cpojer cpojer merged commit 1918f6b into jestjs:master Dec 18, 2017
@cpojer
Copy link
Member

cpojer commented Dec 18, 2017

Nice!

@SimenB SimenB deleted the snapshot-codeframe branch December 18, 2017 13:45
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 13, 2021
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.

5 participants