Skip to content

fs: add read(buffer[, options]) versions #42768

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
aduh95 merged 16 commits into from
May 2, 2022
Merged

fs: add read(buffer[, options]) versions #42768

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
c69abaa
fs: add read(buffer, options) signatures
LiviaMedeiros Apr 15, 2022
aec64ca
squash: make offsetOrOptions more consistent
LiviaMedeiros Apr 17, 2022
42c9c40
squash: add doc entries
LiviaMedeiros Apr 17, 2022
71f03ce
squash: fix fs.read()
LiviaMedeiros Apr 17, 2022
0bd9de7
squash: expand fs-read-optional test
LiviaMedeiros Apr 17, 2022
abd6e4f
squash: expand fs-read-promises-optional test
LiviaMedeiros Apr 17, 2022
9826309
squash: improve tests
LiviaMedeiros Apr 18, 2022
a32c914
squash: fixup
LiviaMedeiros Apr 18, 2022
6a974c2
squash: fixup
LiviaMedeiros Apr 18, 2022
0f586d2
squash: inline null coalescing
LiviaMedeiros Apr 18, 2022
6410dec
squash: revert changes in fs.readSync()
LiviaMedeiros Apr 19, 2022
ccaa4cb
squash: disallow offset === false
LiviaMedeiros Apr 19, 2022
0e6b06a
squash: fixup
LiviaMedeiros Apr 21, 2022
42b640c
squash: allow offset === null
LiviaMedeiros Apr 21, 2022
e153143
squash: fix tests being other way round
LiviaMedeiros Apr 21, 2022
ab0b534
squash: adjust tests
LiviaMedeiros Apr 22, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
squash: fixup
  • Loading branch information
@LiviaMedeiros
LiviaMedeiros committed Apr 18, 2022
commit 6a974c2d21764f95c4b88d0bc26f303f52cfe5ee
4 changes: 2 additions & 2 deletions test/parallel/test-fs-read-optional-params.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,15 +13,15 @@ const bufferAsOption = Buffer.allocUnsafe(expected.byteLength);
function testValid(message, ...options) {
const paramsMsg = `${message} (as params)`;
const paramsFilehandle = fs.openSync(filepath, 'r');
fs.read(paramsFilehandle, ...options, common.mustCall((err, bytesRead, buffer) => {
fs.read(paramsFilehandle, ...options, common.mustSucceed((bytesRead, buffer) => {
assert.strictEqual(bytesRead, expected.byteLength, paramsMsg);
assert.deepStrictEqual(defaultBufferAsync.byteLength, buffer.byteLength, paramsMsg);
fs.closeSync(paramsFilehandle);
}));

const optionsMsg = `${message} (as options)`;
const optionsFilehandle = fs.openSync(filepath, 'r');
fs.read(optionsFilehandle, bufferAsOption, ...options, common.mustCall((err, bytesRead, buffer) => {
fs.read(optionsFilehandle, bufferAsOption, ...options, common.mustSucceed((bytesRead, buffer) => {
assert.strictEqual(bytesRead, expected.byteLength, optionsMsg);
assert.deepStrictEqual(bufferAsOption.byteLength, buffer.byteLength, optionsMsg);
fs.closeSync(optionsFilehandle);
Expand Down