fs: fix failing fs.writeFile AbortController test

Moves the AbortSignal check to the end of the `do { } while();`
nodejs · Aug 22, 2021 · 81f457d · 81f457d
1 parent 6f22200
commit 81f457d
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/internal/fs/promises.js b/lib/internal/fs/promises.js
@@ -255,9 +255,6 @@ async function writeFileHandle(filehandle, data, signal) {
   let remaining = data.length;
   if (remaining === 0) return;
   do {
-    if (signal?.aborted) {
-      throw new lazyDOMException('The operation was aborted', 'AbortError');
-    }
     const { bytesWritten } =
       await write(filehandle, data, 0,
                   MathMin(kWriteFileMaxChunkSize, data.length));
@@ -267,6 +264,9 @@ async function writeFileHandle(filehandle, data, signal) {
       data.byteOffset + bytesWritten,
       data.byteLength - bytesWritten
     );
+    if (signal?.aborted) {
+      throw new lazyDOMException('The operation was aborted', 'AbortError');
+    }
   } while (remaining > 0);
 }