fix: provider connect method accept options (#1388)

Co-authored-by: Dhaiwat <[email protected]>

.changeset/khaki-years-compare.md

@@ -0,0 +1,5 @@
+---
+"@fuel-ts/providers": patch
+---
+
+You can now pass in `ProviderOptions` to `Provider.connect`

packages/providers/src/provider.ts

@@ -310,9 +310,9 @@ export default class Provider {
   /**
    * Updates the URL for the provider and fetches the consensus parameters for the new URL, if needed.
    */
-  async connect(url: string) {
+  async connect(url: string, options?: ProviderOptions) {
     this.url = url;
-    this.operations = this.createOperations(url);
+    this.operations = this.createOperations(url, options ?? this.options);
     await this.fetchChainAndNodeInfo();
   }
 

packages/providers/test/provider.test.ts

@@ -241,6 +241,38 @@ describe('Provider', () => {
     const provider = await Provider.create(providerUrl, {
       fetch: getCustomFetch('getVersion', { nodeInfo: { nodeVersion: '0.30.0' } }),
     });
+
+    expect(await provider.getVersion()).toEqual('0.30.0');
+  });
+
+  it('can accept options override in connect method', async () => {
+    const providerUrl = FUEL_NETWORK_URL;
+
+    /**
+     * Mocking and initializing Provider with an invalid fetcher just
+     * to ensure it'll be properly overriden in `connect` method below
+     */
+    const fetchChainAndNodeInfo = jest
+      .spyOn(Provider.prototype, 'fetchChainAndNodeInfo')
+      .mockImplementation();
+
+    const provider = await Provider.create(providerUrl, {
+      fetch: () => {
+        throw new Error('This should never happen');
+      },
+    });
+
+    expect(fetchChainAndNodeInfo).toHaveBeenCalledTimes(1);
+
+    /**
+     * Restore mock and call connect with a proper fetch override
+     */
+    fetchChainAndNodeInfo.mockRestore();
+
+    await provider.connect(providerUrl, {
+      fetch: getCustomFetch('getVersion', { nodeInfo: { nodeVersion: '0.30.0' } }),
+    });
+
     expect(await provider.getVersion()).toEqual('0.30.0');
   });