Skip to content

Commit

Permalink
test: remove references to unsupported AIX versions
Browse files Browse the repository at this point in the history
The `filehandle.utimes()` and `fs.futimes()` APIs work on all versions
of AIX that are supported by this version of Node.js.

PR-URL: #37826
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Stewart X Addison <sxa@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ash Cripps <acripps@redhat.com>
  • Loading branch information
richardlau authored and ruyadorno committed Mar 24, 2021
1 parent 43c3b43 commit 9bc6fe7
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 13 deletions.
6 changes: 0 additions & 6 deletions doc/api/fs.md
Original file line number Diff line number Diff line change
Expand Up @@ -420,9 +420,6 @@ added: v10.0.0
Change the file system timestamps of the object referenced by the {FileHandle}
then resolves the promise with no arguments upon success.
This function does not work on AIX versions before 7.1, it will reject the
promise with an error using code `UV_ENOSYS`.
#### `filehandle.write(buffer[, offset[, length[, position]]])`
<!-- YAML
added: v10.0.0
Expand Down Expand Up @@ -2326,9 +2323,6 @@ changes:
Change the file system timestamps of the object referenced by the supplied file
descriptor. See [`fs.utimes()`][].
This function does not work on AIX versions before 7.1, it will return the
error `UV_ENOSYS`.
### `fs.lchmod(path, mode, callback)`
<!-- YAML
deprecated: v0.4.7
Expand Down
7 changes: 0 additions & 7 deletions test/parallel/test-trace-events-fs-sync.js
Original file line number Diff line number Diff line change
Expand Up @@ -124,13 +124,6 @@ for (const tr in tests) {
'--trace-event-categories', 'node.fs.sync',
'-e', tests[tr] ],
{ cwd: tmpdir.path, encoding: 'utf8' });
// Some AIX versions don't support futimes or utimes, so skip.
if (common.isAIX && proc.status !== 0 && tr === 'fs.sync.futimes') {
continue;
}
if (common.isAIX && proc.status !== 0 && tr === 'fs.sync.utimes') {
continue;
}

// Make sure the operation is successful.
// Don't use assert with a custom message here. Otherwise the
Expand Down

0 comments on commit 9bc6fe7

Please sign in to comment.