Skip to content

Commit

Permalink
test: remove AIX guard in fs-options-immutable
Browse files Browse the repository at this point in the history
The fs watch test was not run on AIX till now because of the known
issue, #5085. Now that it is
completed, this guard can be removed.

PR-URL: #12687
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
  • Loading branch information
thefourtheye authored and gibfahn committed Apr 28, 2017
1 parent 71f22c8 commit b16869c
Showing 1 changed file with 10 additions and 14 deletions.
24 changes: 10 additions & 14 deletions test/parallel/test-fs-options-immutable.js
Original file line number Diff line number Diff line change
Expand Up @@ -58,21 +58,17 @@ if (common.canCreateSymLink()) {
);
}

if (!common.isAix) {
// TODO(thefourtheye) Remove this guard once
// https://github.com/nodejs/node/issues/5085 is fixed
{
let watch;
assert.doesNotThrow(() => {
watch = fs.watch(__filename, options, common.noop);
});
watch.close();
}
{
let watch;
assert.doesNotThrow(() => {
watch = fs.watch(__filename, options, common.noop);
});
watch.close();
}

{
assert.doesNotThrow(() => fs.watchFile(__filename, options, common.noop));
fs.unwatchFile(__filename);
}
{
assert.doesNotThrow(() => fs.watchFile(__filename, options, common.noop));
fs.unwatchFile(__filename);
}

{
Expand Down

0 comments on commit b16869c

Please sign in to comment.