squash: add bytesRead === length assertions

nodejs · Jan 17, 2023 · e9c875d · e9c875d
1 parent d2b9865
commit e9c875d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/test/parallel/test-fs-promises-write-optional-params.js b/test/parallel/test-fs-promises-write-optional-params.js
@@ -54,6 +54,7 @@ async function testValid(dest, buffer, options) {
   assert.ok(writeResult.bytesWritten >= readResult.bytesRead);
   if (length !== undefined && length !== null) {
     assert.strictEqual(writeResult.bytesWritten, length);
+    assert.strictEqual(readResult.bytesRead, length);
   }
   if (offset === undefined || offset === 0) {
     assert.deepStrictEqual(writeBufCopy, readBufCopy);

diff --git a/test/parallel/test-fs-write-optional-params.js b/test/parallel/test-fs-write-optional-params.js
@@ -40,6 +40,7 @@ function testValidCb(buffer, options, index, callback) {
             assert.ok(bytesWritten >= bytesRead);
             if (length !== undefined && length !== null) {
               assert.strictEqual(bytesWritten, length);
+              assert.strictEqual(bytesRead, length);
             }
             if (offset === undefined || offset === 0) {
               assert.deepStrictEqual(writeBufCopy, readBufCopy);

diff --git a/test/parallel/test-fs-write-sync-optional-params.js b/test/parallel/test-fs-write-sync-optional-params.js
@@ -51,6 +51,7 @@ function testValid(dest, buffer, options) {
   assert.ok(bytesWritten >= bytesRead);
   if (length !== undefined && length !== null) {
     assert.strictEqual(bytesWritten, length);
+    assert.strictEqual(bytesRead, length);
   }
 }