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

jest-circus feature parity with jest-jasmine #4362

Closed
11 tasks done
aaronabramov opened this issue Aug 25, 2017 · 5 comments · Fixed by #6328
Closed
11 tasks done

jest-circus feature parity with jest-jasmine #4362

aaronabramov opened this issue Aug 25, 2017 · 5 comments · Fixed by #6328

Comments

@aaronabramov
Copy link
Contributor

aaronabramov commented Aug 25, 2017

TODO items:
fix broken test.
to run all tests using jest-circus update this require line
https://github.com/facebook/jest/blob/19384094dc7ac5cd77e4eacf611d76abe5837063/packages/jest-runner/src/run_test.js#L63
with a path to the adapter:

require('jest-circus/build/legacy_code_todo_rewrite/jest_adapter.js').default

and run

./jest

failing tests:

  • integration-tests/__tests__/snapshot.test.js (@aaronabramov)
  • integration-tests/__tests__/jasmine_async.test.js (@aaronabramov)
  • integration-tests/__tests__/globals.test.js (@thymikee)
  • packages/jest-cli/src/__tests__/search_source.test.js (@aaronabramov)
  • integration-tests/__tests__/lifecycles.js (@aaronabramov)
  • integration-tests/__tests__/failures.test.js (@SimenB)
  • integration-tests/__tests__/test_name_pattern.test.js (@aaronabramov)
  • integration-tests/__tests__/location_in_results.test.js (@captbaritone)
  • integration-tests/__tests__/override-globals.test.js (@aaronabramov)
  • integration-tests/__tests__/each.test.js (@mattphillips)
  • integration-tests/__tests__/timer_use_real_timers.test.js (@thymikee)

PRs that i opened but never had a chance to merge (need to come back and fix a few things):
#3819
#3824

also this is a new feature that was added to jasmine https://github.com/facebook/jest/pull/4016/files

@suchipi
Copy link
Contributor

suchipi commented Aug 25, 2017

Thanks @aaronabramov, I'll take a look at this this weekend

@suchipi
Copy link
Contributor

suchipi commented Aug 27, 2017

I'm confused; I ran the tests with the jest-circus adapter (as far as I can tell) and I'm not seeing any failures.

I ran:

./jest --testRunner '<rootDir>/packages/jest-circus/build/legacy_code_todo_rewrite/jest_adapter.js'

In the repo root, and saw no failures except for some tests that failed because they depended on mercurial, which I didn't have installed.

After installing mercurial, I ran the same command again, and saw no failures.

I was expecting to see some failures and that those would lead me to the work that needed to be done. Am I misunderstanding the situation?

@cpojer
Copy link
Member

cpojer commented Aug 27, 2017

All the integration tests will still use Jasmine. You could probably change Jest's default config to use jest-circus and then see which integration tests fail, or go into each folder in integration_tests and run individually with circus.

@suchipi
Copy link
Contributor

suchipi commented Aug 27, 2017

Ok, changing the default for testRunner in jest-cli/src/cli/args.js and running yarn run build worked, I see the failures now. Thanks!

captbaritone added a commit to captbaritone/jest that referenced this issue May 26, 2018
captbaritone added a commit to captbaritone/jest that referenced this issue May 26, 2018
captbaritone added a commit to captbaritone/jest that referenced this issue May 26, 2018
captbaritone added a commit to captbaritone/jest that referenced this issue May 26, 2018
captbaritone added a commit to captbaritone/jest that referenced this issue May 26, 2018
aaronabramov pushed a commit that referenced this issue May 27, 2018
* Add `testLocationInResults` support to jest-circus

Part of #4362

* Hoist StackUtils construction

* Move test location config into state
@github-actions
Copy link

This issue 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 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants