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: expand fs-read-optional test
  • Loading branch information
@LiviaMedeiros
LiviaMedeiros committed Apr 17, 2022
commit 0bd9de772ac40f25d8c80e7c3c0acc5e5771ff94
33 changes: 25 additions & 8 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 @@ -9,28 +9,45 @@ const fd = fs.openSync(filepath, 'r');

const expected = Buffer.from('xyz\n');
const defaultBufferAsync = Buffer.alloc(16384);
const bufferAsOption = Buffer.allocUnsafe(expected.length);
const bufferAsOption = Buffer.allocUnsafe(expected.byteLength);

// Test not passing in any options object
fs.read(fd, common.mustCall((err, bytesRead, buffer) => {
assert.strictEqual(bytesRead, expected.length);
assert.deepStrictEqual(defaultBufferAsync.length, buffer.length);
assert.strictEqual(bytesRead, expected.byteLength);
assert.deepStrictEqual(defaultBufferAsync.byteLength, buffer.byteLength);
}));
fs.read(fd, bufferAsOption, { position: 0 }, common.mustCall((err, bytesRead, buffer) => {
assert.strictEqual(bytesRead, expected.byteLength);
assert.deepStrictEqual(bufferAsOption.byteLength, buffer.byteLength);
}));

// Test passing in an empty options object
fs.read(fd, { position: 0 }, common.mustCall((err, bytesRead, buffer) => {
assert.strictEqual(bytesRead, expected.length);
assert.deepStrictEqual(defaultBufferAsync.length, buffer.length);
assert.strictEqual(bytesRead, expected.byteLength);
assert.deepStrictEqual(defaultBufferAsync.byteLength, buffer.byteLength);
}));
fs.read(fd, bufferAsOption, { position: 0 }, common.mustCall((err, bytesRead, buffer) => {
assert.strictEqual(bytesRead, expected.byteLength);
assert.deepStrictEqual(bufferAsOption.byteLength, buffer.byteLength);
}));

// Test passing in options
fs.read(fd, {
buffer: bufferAsOption,
offset: 0,
length: bufferAsOption.length,
length: bufferAsOption.byteLength,
position: 0
Comment thread
LiviaMedeiros marked this conversation as resolved.
Outdated
},
common.mustCall((err, bytesRead, buffer) => {
assert.strictEqual(bytesRead, expected.byteLength);
assert.deepStrictEqual(bufferAsOption.byteLength, buffer.byteLength);
}));
fs.read(fd, bufferAsOption, {
offset: 0,
length: bufferAsOption.byteLength,
position: 0
},
common.mustCall((err, bytesRead, buffer) => {
assert.strictEqual(bytesRead, expected.length);
assert.deepStrictEqual(bufferAsOption.length, buffer.length);
assert.strictEqual(bytesRead, expected.byteLength);
assert.deepStrictEqual(bufferAsOption.byteLength, buffer.byteLength);
}));