test: remove flaky designation for test on AIX

#5085 has been completed so presumably test-fs-watch is not flaky on AIX anymore. Remove flaky designation from sequential.status. PR-URL: #12564 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]>
nodejs · May 2, 2017 · d10eb83 · d10eb83
1 parent cfd9144
commit d10eb83
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/test/sequential/sequential.status b/test/sequential/sequential.status
@@ -17,9 +17,4 @@ test-benchmark-child-process  : PASS,FLAKY
 
 [$system==freebsd]
 
-# fs-watch currently needs special configuration on AIX and we
-# want to improve under https://github.com/nodejs/node/issues/5085.
-# Tests are disabled so CI can be green and we can spot other
-# regressions until this work is complete
 [$system==aix]
-test-fs-watch               : FAIL,PASS
diff --git a/test/sequential/test-fs-watch.js b/test/sequential/test-fs-watch.js
@@ -6,7 +6,8 @@ const fs = require('fs');
 
 const expectFilePath = common.isWindows ||
                        common.isLinux ||
-                       common.isOSX;
+                       common.isOSX ||
+                       common.isAix;
 
 let watchSeenOne = 0;
 let watchSeenTwo = 0;
@@ -80,7 +81,7 @@ const filepathThree = path.join(testsubdir, filenameThree);
 assert.doesNotThrow(
     function() {
       const watcher = fs.watch(testsubdir, function(event, filename) {
-        const renameEv = common.isSunOS ? 'change' : 'rename';
+        const renameEv = common.isSunOS || common.isAix ? 'change' : 'rename';
         assert.strictEqual(renameEv, event);
         if (expectFilePath) {
           assert.strictEqual('newfile.txt', filename);