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.

Sign up for GitHub

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

Jump to bottom

fix: get around port collisions with a random port #1418

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix: get around port collisions with a random port #1418

wants to merge 2 commits into from

Commits on Sep 15, 2020

  1. refactor: extracting testing types into folder 

    - updating test folder to match setup of internal ‘plugin’ library
    @haysclark
    haysclark committed Sep 15, 2020
    Configuration menu
    Copy the full SHA
    f0129d2 View commit details
    Browse the repository at this point in the history

Commits on Sep 22, 2020

  1. fix: port collisions in createTestContext (prisma-labs#1326) 

    - during testing, get-port often re-uses the same port, due to the delay between when a free-port is assigned to a test, and when the port is actually used. Simply supplying an initial random port value to get-port dramatically reduces the chance of a port being used multiple times in a test run.
- testing on CI environments should still utilize Jest’s `—runInBand` flag
    @haysclark
    haysclark committed Sep 22, 2020
    Configuration menu
    Copy the full SHA
    fac447d View commit details
    Browse the repository at this point in the history