fix: reduce flakiness by favoring port 0 over portfinder dependen…

…cy (#3219) * chore: remove `portfinder` in favor of port `0` * chore: changeset * chore: changeset msg
FuelLabs · Sep 27, 2024 · f02fa88 · f02fa88
1 parent 465bd5c
commit f02fa88
Show file tree

Hide file tree

Showing 4 changed files with 159 additions and 114 deletions.
diff --git a/.changeset/odd-starfishes-flash.md b/.changeset/odd-starfishes-flash.md
@@ -0,0 +1,5 @@
+---
+"@fuel-ts/account": patch
+---
+
+fix: reduce flakiness by favoring port `0` over `portfinder` dependency
diff --git a/packages/account/package.json b/packages/account/package.json
@@ -65,7 +65,6 @@
     "graphql": "^16.9.0",
     "graphql-request": "5.0.0",
     "graphql-tag": "^2.12.6",
-    "portfinder": "^1.0.32",
     "ramda": "^0.30.1"
   },
   "devDependencies": {

diff --git a/packages/account/src/test-utils/launchNode.ts b/packages/account/src/test-utils/launchNode.ts
@@ -6,7 +6,6 @@ import { defaultConsensusKey, hexlify, defaultSnapshotConfigs } from '@fuel-ts/u
 import { existsSync, mkdirSync, rmSync, writeFileSync } from 'fs';
 import os from 'os';
 import path from 'path';
-import { getPortPromise } from 'portfinder';
 
 import { Signer } from '../signer';
 
@@ -148,14 +147,7 @@ export const launchNode = async ({
 
     const ipToUse = ip || '0.0.0.0';
 
-    const portToUse =
-      port ||
-      (
-        await getPortPromise({
-          port: 4000, // tries 4000 first, then 4001, then 4002, etc.
-          stopPort: 5000, // don't try ports above 5000
-        })
-      ).toString();
+    const portToUse = port || '0';
 
     let snapshotDirToUse: string;