Skip to content
This repository was archived by the owner on Jan 11, 2023. It is now read-only.

[tests] Fix requiring workers in jest#2431

Merged
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
jasonLaster:jest-workers
Mar 24, 2017
Merged

[tests] Fix requiring workers in jest#2431
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
jasonLaster:jest-workers

Conversation

@jasonLaster

@jasonLaster jasonLaster commented Mar 23, 2017

Copy link
Copy Markdown
Contributor

Associated Issue: #2427

Summary of Changes

we're currently seeing issues running the unit tests that touch workers like parser, source maps. The reason for this is that we're no longer pre-bundling the workers. The solution I want to go with is to stop building the workers and instead use workerjs to point to the worker entry file directly.

  • update bootstrap to provide worker paths
  • convert pretty print to the same directory format
  • update worker URLs to a new config structure

TO DO:

  • add workerTask to devtools-modules
  • update source map worker to use the new config.

@jasonLaster jasonLaster force-pushed the jest-workers branch 5 times, most recently from f102f74 to e8d7818 Compare March 24, 2017 16:10
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 this pull request may close these issues.

1 participant