Home Original page

test: fix single test runner regression by apapirovski · Pull Request #15300 · nodejs/node

@apapirovski

@nodejs-github-bot added the test

Issues and PRs related to the tests.

label

Sep 9, 2017

@mscdex mscdex added the python

PRs and issues that require attention from people who are familiar with Python.

label

Sep 9, 2017

This was referenced

Sep 11, 2017

TimothyGu added a commit to TimothyGu/node that referenced this pull request

Sep 11, 2017

MylesBorins pushed a commit that referenced this pull request

Sep 12, 2017 
When ESM support was added it created a regression in the test runner
that broke the ability to run individual tests. This commit
re-introduces the use of `NormalizePath` which fixes the regression
in the test runner

Refs: #15300
PR-URL: #15329
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

MylesBorins pushed a commit that referenced this pull request

Sep 12, 2017 
When ESM support was added it created a regression in the test runner
that broke the ability to run individual tests. This commit
re-introduces the use of `NormalizePath` which fixes the regression
in the test runner

Refs: #15300
PR-URL: #15329
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

addaleax pushed a commit to addaleax/node that referenced this pull request

Sep 13, 2017 
When ESM support was added it created a regression in the test runner
that broke the ability to run individual tests. This commit
re-introduces the use of `NormalizePath` which fixes the regression
in the test runner

Refs: nodejs#15300
PR-URL: nodejs#15329
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>