feat(remix-node): add getFilePath& remove methods to `NodeOnDiskF…

…ile` (#4408)

* feat(node): add new methods to NodeOnDiskFile class

* chore: add name to contributors for CLA signature

* added changeset

* small changeset rewording

* reference the class instead of the file in changeset

.changeset/flat-cobras-beam.md

@@ -0,0 +1,5 @@
+---
+"@remix-run/node": minor
+---
+
+Add `remove` and `getFilePath` methods to `NodeOnDiskFile`

contributors.yml

@@ -224,6 +224,7 @@
 - juhanakristian
 - JulesBlm
 - juliaqiuxy
+- julienmonnard
 - justinnoel
 - justinsalasdev
 - justinwaite

packages/remix-node/__tests__/fileUploadHandler-test.ts

@@ -104,4 +104,18 @@ describe("NodeOnDiskFile", () => {
     expect(sliced.size).toBe(slicedRes.length);
     expect(await sliced.text()).toBe(slicedRes);
   });
+
+  it("returns the file path properly", async () => {
+    expect(file.getFilePath()).toEqual(filepath);
+  });
+
+  it("removes the file properly", async () => {
+    let newFilePath = `${filepath}-copy`;
+    fs.copyFileSync(filepath, newFilePath);
+
+    let copiedFile = (file = new NodeOnDiskFile(newFilePath, "text/plain"));
+    expect(fs.existsSync(copiedFile.getFilePath())).toBe(true);
+    await copiedFile.remove();
+    expect(fs.existsSync(copiedFile.getFilePath())).toBe(false);
+  });
 });

packages/remix-node/upload/fileUploadHandler.ts

@@ -1,6 +1,6 @@
 import { randomBytes } from "crypto";
 import { createReadStream, createWriteStream, statSync } from "fs";
-import { rm, mkdir, stat as statAsync } from "fs/promises";
+import { rm, mkdir, stat as statAsync, unlink } from "fs/promises";
 import { tmpdir } from "os";
 import { basename, dirname, extname, resolve as resolvePath } from "path";
 import type { Readable } from "stream";
@@ -231,4 +231,11 @@ export class NodeOnDiskFile implements File {
   public get [Symbol.toStringTag]() {
     return "File";
   }
+
+  remove(): Promise<void> {
+    return unlink(this.filepath);
+  }
+  getFilePath(): string {
+    return this.filepath;
+  }
 }