Skip to content

Update Jest - #530

Merged
gaearon merged 7 commits into
react:masterfrom
gaearon:update-jest
Sep 1, 2016
Merged

Update Jest#530
gaearon merged 7 commits into
react:masterfrom
gaearon:update-jest

Conversation

@gaearon

Copy link
Copy Markdown
Contributor

Needs another fix scheduled for 15.0.1 but should be almost good to go.

@ghostghost added the CLA Signed label Aug 31, 2016
@vjeux

Copy link
Copy Markdown
Contributor

lgtm, the test file is very simple, I like it :)

Comment threadtasks/e2e.sh Outdated
test -e src/__tests__/__snapshots__/App-test.js.snap
npm test -- --smoke-test
# Uncomment when snapshot testing is enabled by default:
# test -e src/__tests__/__snapshots__/App-test.js.snap

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

do you mean App.spec.js here?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Since it's disabled this doesn't really matter—when we re-add it we can revisit naming and where we put it.

@gaearon
gaearon merged commit 67fe759 into react:masterSep 1, 2016
@gaearongaearon added this to the 0.3.0 milestone Sep 1, 2016
@cpojer

Copy link
Copy Markdown
Contributor

boom.

@gaearon
gaearon deleted the update-jest branch September 1, 2016 09:07
@gaearongaearon mentioned this pull request Sep 1, 2016
stayradiated pushed a commit to stayradiated/create-react-app that referenced this pull request Sep 7, 2016
* Update Jest
* Remove default snapshot test
* Fix a few things
* Add a simple default test
* App.spec.js -> App.test.js
* Fix e2e test
* Bump Jest
feiqitian pushed a commit to feiqitian/create-react-app that referenced this pull request Oct 25, 2016
* Update Jest
* Remove default snapshot test
* Fix a few things
* Add a simple default test
* App.spec.js -> App.test.js
* Fix e2e test
* Bump Jest
@locklockBot locked and limited conversation to collaborators Jan 22, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@gaearon@vjeux@cpojer@lacker