fix: enable fuel-core node cleanup to work in Bun

.changeset/wise-ladybugs-nail.md

@@ -0,0 +1,5 @@
+---
+"@fuel-ts/account": patch
+---
+
+fix: enable `fuel-core` node cleanup to work in Bun

packages/account/package.json

@@ -68,16 +68,16 @@
     "ramda": "^0.30.1"
   },
   "devDependencies": {
-    "type-fest": "^4.26.1",
     "@fuel-ts/hasher": "workspace:*",
     "@fuel-ts/math": "workspace:*",
     "@fuel-ts/utils": "workspace:*",
     "@graphql-codegen/cli": "^5.0.3",
     "@graphql-codegen/typescript": "^4.0.9",
     "@graphql-codegen/typescript-generic-sdk": "^4.0.1",
     "@graphql-codegen/typescript-operations": "^4.2.3",
+    "@internal/utils": "workspace:*",
     "@types/ramda": "^0.30.2",
     "get-graphql-schema": "^2.1.2",
-    "@internal/utils": "workspace:*"
+    "type-fest": "^4.26.1"
   }
 }

packages/account/src/test-utils/launchNode.test.ts

@@ -29,6 +29,10 @@ vi.mock('fs', async () => {
  * @group node
  */
 describe('launchNode', () => {
+  beforeEach(() => {
+    vi.spyOn(console, 'log').mockImplementation(() => {});
+  });
+
   test('using ephemeral port 0 is possible', async () => {
     const { cleanup, port, url } = await launchNode({ port: '0' });
     expect(await fetch(url)).toBeTruthy();
@@ -252,4 +256,18 @@ describe('launchNode', () => {
       `fuel-core node under pid ${pid} does not exist. The node might have been killed before cleanup was called. Exiting cleanly.`
     );
   });
+
+  test('should clean up when unable to kill process with "RangeError: pid must be a positive integer" error', async () => {
+    const killSpy = vi.spyOn(process, 'kill').mockImplementationOnce(() => {
+      throw new RangeError('pid must be a positive integer');
+    });
+
+    const { pid, cleanup } = await launchNode({ loggingEnabled: false });
+
+    cleanup();
+
+    expect(killSpy).toBeCalledTimes(2);
+    expect(killSpy).toBeCalledWith(-pid);
+    expect(killSpy).toBeCalledWith(+pid);
+  });
 });

packages/account/src/test-utils/launchNode.ts

@@ -236,6 +236,10 @@ export const launchNode = async ({
             console.log(
               `fuel-core node under pid ${child.pid} does not exist. The node might have been killed before cleanup was called. Exiting cleanly.`
             );
+          } else if (error.message.includes('pid must be a positive integer')) {
+            // This is a workaround for a bug with Bun.
+            // See: https://github.com/oven-sh/bun/issues/8787
+            process.kill(+child.pid);
           } else {
             throw e;
           }

tsconfig.test.json

@@ -3,6 +3,6 @@
   "compilerOptions": {
     "noEmit": true
   },
-  "include": ["**/*.test.ts"],
+  "include": ["**/*.test.ts", "**/*.d.ts"],
   "exclude": ["node_modules", "apps/docs/src/**/*.test.ts"]
 }