Skip to content

Commit 21b3f3f

Browse files
ryzokukentargos
authored andcommitted
test: rename test-regress-GH-4027
Rename the test appropriately alongside mentioning the subsystem Also, make a few basic changes to make sure the test conforms to the standard test structure Refs: #19105 Refs: https://github.com/nodejs/node/blob/master/doc/guides/writing-tests.md#test-structure PR-URL: #19161 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
1 parent bbc1bbd commit 21b3f3f

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

test/sequential/test-regress-GH-4027.js renamed to test/sequential/test-regress-fs-watch_file.js

+11
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,17 @@
2121

2222
'use strict';
2323
const common = require('../common');
24+
25+
// Make sure the deletion event gets reported in the following scenario:
26+
// 1. Watch a file.
27+
// 2. The initial stat() goes okay.
28+
// 3. Something deletes the watched file.
29+
// 4. The second stat() fails with ENOENT.
30+
31+
// The second stat() translates into the first 'change' event but a logic error
32+
// stopped it from getting emitted.
33+
// https://github.com/nodejs/node-v0.x-archive/issues/4027
34+
2435
const assert = require('assert');
2536
const path = require('path');
2637
const fs = require('fs');

0 commit comments

Comments
 (0)