Skip to content

Commit 381f8ed

Browse files
committed
add test for confusing filenames
The behavior was correct, but the test that covered that line was relying on a race condition to actually hit it, which is no good. This makes the test deterministic.
1 parent da10468 commit 381f8ed

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

test/run/save-file.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ t.test('with bailout, should save all untested', t => {
4444
})
4545

4646
t.test('without bailout, run untested, save failures', t => {
47-
run(['a', 'x', 'z.js', '-s', savefile], opt, (er, o, e) => {
47+
run(['a', 'x', 'a\\b\\f1.js', 'z.js', '-s', savefile], opt, (er, o, e) => {
4848
t.match(er, { code: 1 })
4949
t.matchSnapshot(o, 'stdout', { skip: winSkip || oldSkip })
5050
t.equal(clean(e), '')
@@ -53,7 +53,7 @@ t.test('without bailout, run untested, save failures', t => {
5353
})
5454
})
5555

56-
t.test('make fails pass', t => {
56+
t.test('make first fail pass', t => {
5757
fs.writeFileSync(abf1, `
5858
require(${tap}).pass('fine now')
5959
`)

0 commit comments

Comments
 (0)