Skip to content
GitLab
    • Explore Projects Groups Snippets
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • C create-react-app
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1,547
    • Issues 1,547
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 417
    • Merge requests 417
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Meta
  • create-react-app
  • Merge requests
  • !1819

Makes end-to-end testing crash on unhandled rejections

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/dbismut/e2e-testing-to-crash-on-unhandled-rejection into master 8 years ago
  • Overview 11
  • Commits 2
  • Pipelines 0
  • Changes 5

Created by: dbismut

This makes sure that end-to-end-testing crash when scripts/*.js run into an unhandled promise rejection as explained here https://github.com/facebookincubator/create-react-app/issues/1806#issuecomment-286146425.

This code has been added at the beginning of each script:

process.on('unhandledRejection', err => {
  throw err;
});

Before

Faulty eject.js fails silently.

before before 2

After

Faulty eject.js makes end-to-end testing crash.

after

Note that the bug outlined above should be fixed by #1810.

Approval is optional

Merged by (Jul 25, 2025 8:48am UTC)

Merge details

  • Changes merged into master with eaf3a647.
  • Deleted the source branch.

Activity


  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: facebook-github-bot

    Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at cla@fb.com. Thanks!

    If you are contributing on behalf of someone else (eg your employer): the individual CLA is not sufficient - use https://developers.facebook.com/opensource/cla?type=company instead. Contact cla@fb.com if you have any questions.

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: facebook-github-bot

    Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: Timer

    We should probably land this followed by https://github.com/facebookincubator/create-react-app/pull/1810 immediately after.

    LGTM if it LG2Y @gaearon.

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: Timer

    @dbismut Would you like to rebase this to see if tests pass now?

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: dbismut

    @Timer and there you reach my limits once more... How do I rebase the PR the right way? EDIT: I tried something, I'm not sure this was a good idea...

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: Timer

    Heh, you cherry-picked.

    Basically:

    # Switch to your feature branch
    $ git checkout e2e-testing-to-crash-on-unhandled-rejection # wow this is a huge branch name
    
    # Add the upstream repository as a remote
    $ git remote add upstream git@github.com:facebookincubator/create-react-app.git # if not already
    
    # Fetch upstream, creates upstream/* references
    $ git fetch upstream
    
    # Rebase your current branch onto upstream/master
    $ git rebase upstream/master
    
    # Force push to your branch (origin/e2e-testing-to-crash-on-unhandled-rejection) since you re-wrote history
    $ git push -f
  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: dbismut

    Thanks for the explanation! I just ran it, does it work?

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: Timer

    Yup! Thanks so much for your hard work @dbismut. I'm just waiting on @gaearon to sign off (I see no reason why he wouldn't).

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: dbismut

    All done @gaearon!

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Created by: gaearon

    Review: Approved

  • Administrator
    Administrator @root · 8 years ago
    Author Contributor

    Merged by: Timer at 2017-03-14 19:32:48 UTC

Please register or sign in to reply
0 Assignees
None
Assign to
0 Reviewers
None
Request review from
Labels
2
CLA Signed tag: bug fix
2
CLA Signed tag: bug fix
    Assign labels
  • Manage project labels

Milestone
1.0.0
1.0.0
None
Time tracking
No estimate or time spent
Lock merge request
Unlocked
1
1 participant
Administrator
Reference: facebook/create-react-app!1819
Source branch: github/fork/dbismut/e2e-testing-to-crash-on-unhandled-rejection

Menu

Explore Projects Groups Snippets