test: reduce fs-write-optional-params flakiness

[LiviaMedeiros]

Fixes: #46144

When position is null or undefined, reading starts with current file position. Closing and opening the file again between writes and reads resets current position, so the chunk is read from the start.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/49039/

[nodejs-github-bot]

Commit Queue failed

- Loading data for nodejs/node/pull/46238
✔  Done loading data for nodejs/node/pull/46238
----------------------------------- PR info ------------------------------------
Title      test: reduce `fs-write-optional-params` flakiness (#46238)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     LiviaMedeiros:test-fs-write-options-deflake -> nodejs:main
Labels     test, needs-ci
Commits    2
 - test: reduce `fs-write-optional-params` flakiness
 - squash: add `bytesRead === length` assertions
Committers 1
 - LiviaMedeiros 
PR-URL: https://github.com/nodejs/node/pull/46238
Fixes: https://github.com/nodejs/node/issues/46144
Reviewed-By: Luigi Pinca 
Reviewed-By: Antoine du Hamel 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/46238
Fixes: https://github.com/nodejs/node/issues/46144
Reviewed-By: Luigi Pinca 
Reviewed-By: Antoine du Hamel 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Tue, 17 Jan 2023 17:50:14 GMT
   ✔  Approvals: 2
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/46238#pullrequestreview-1252441751
   ✔  - Antoine du Hamel (@aduh95) (TSC): https://github.com/nodejs/node/pull/46238#pullrequestreview-1253151886
   ✔  Last GitHub CI successful
   ℹ  Last Full PR CI on 2023-01-18T12:46:47Z: https://ci.nodejs.org/job/node-test-pull-request/49039/
- Querying data for job/node-test-pull-request/49039/
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  No git cherry-pick in progress
   ✔  No git am in progress
   ✔  No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
 * branch                  main       -> FETCH_HEAD
✔  origin/main is now up-to-date
- Downloading patch for 46238
From https://github.com/nodejs/node
 * branch                  refs/pull/46238/merge -> FETCH_HEAD
✔  Fetched commits as aac2dccf1ccb..e9c875dcaf53
--------------------------------------------------------------------------------
Auto-merging test/parallel/test-fs-write-optional-params.js
[main 8959026472] test: reduce `fs-write-optional-params` flakiness
 Author: LiviaMedeiros 
 Date: Wed Jan 18 01:33:55 2023 +0800
 3 files changed, 52 insertions(+), 36 deletions(-)
Auto-merging test/parallel/test-fs-write-optional-params.js
[main c6379980d8] squash: add `bytesRead === length` assertions
 Author: LiviaMedeiros 
 Date: Wed Jan 18 02:16:50 2023 +0800
 3 files changed, 3 insertions(+)
   ✔  Patches applied
There are 2 commits in the PR. Attempting autorebase.
Rebasing (2/4)
Executing: git node land --amend --yes

--------------------------------- New Message ----------------------------------

test: reduce fs-write-optional-params flakiness
PR-URL: #46238

Fixes: #46144

Reviewed-By: Luigi Pinca luigipinca@gmail.com

Reviewed-By: Antoine du Hamel duhamelantoine1995@gmail.com
[detached HEAD feba87f54c] test: reduce fs-write-optional-params flakiness

Author: LiviaMedeiros livia@cirno.name

Date: Wed Jan 18 01:33:55 2023 +0800

3 files changed, 52 insertions(+), 36 deletions(-)

Rebasing (3/4)

Rebasing (4/4)
Executing: git node land --amend --yes

--------------------------------- New Message ----------------------------------

squash: add bytesRead === length assertions
PR-URL: #46238

Fixes: #46144

Reviewed-By: Luigi Pinca luigipinca@gmail.com

Reviewed-By: Antoine du Hamel duhamelantoine1995@gmail.com
[detached HEAD 89d6b13a86] squash: add bytesRead === length assertions

Author: LiviaMedeiros livia@cirno.name

Date: Wed Jan 18 02:16:50 2023 +0800

3 files changed, 3 insertions(+)
Successfully rebased and updated refs/heads/main.
ℹ  Add commit-queue-squash label to land the PR as one commit, or commit-queue-rebase to land as separate commits.

https://github.com/nodejs/node/actions/runs/3961041112

[LiviaMedeiros]

Landed in e7e1255