feat(connect): make transportReconnect implicitly true

trezor · Nov 7, 2024 · 233939a · 233939a
1 parent e436527
commit 233939a
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 2 deletions.
diff --git a/packages/connect/e2e/common.setup.ts b/packages/connect/e2e/common.setup.ts
@@ -127,6 +127,7 @@ export const initTrezorConnect = async (
         debug: false,
         popup: false,
         pendingTransportEvent: true,
+        transportReconnect: false,
         connectSrc: process.env.TREZOR_CONNECT_SRC, // custom source for karma tests
         ...options,
     });

diff --git a/packages/connect/src/core/__tests__/Core.test.ts b/packages/connect/src/core/__tests__/Core.test.ts
@@ -7,7 +7,11 @@ import { initCoreState } from '../index';
 const { createTestTransport } = global.JestMocks;
 
 const getSettings = (partial: Partial<ConnectSettings> = {}) =>
-    parseConnectSettings({ transports: [createTestTransport()], ...partial });
+    parseConnectSettings({
+        transports: [createTestTransport()],
+        transportReconnect: false,
+        ...partial,
+    });
 
 describe('Core', () => {
     beforeAll(async () => {});

diff --git a/packages/connect/src/data/connectSettings.ts b/packages/connect/src/data/connectSettings.ts
@@ -29,6 +29,7 @@ const initialSettings: ConnectSettings = {
     interactionTimeout: 600, // 5 minutes
     sharedLogger: true,
     deeplinkUrl: `${DEFAULT_DOMAIN}deeplink/${DEEPLINK_VERSION}/`,
+    transportReconnect: true,
 };
 
 const parseManifest = (manifest?: Manifest) => {

diff --git a/packages/connect/src/device/DeviceList.ts b/packages/connect/src/device/DeviceList.ts
@@ -263,7 +263,9 @@ export class DeviceList extends TypedEmitter<DeviceListEvents> implements IDevic
         const { promise, reject } = resolveAfter(1000, initParams);
         this.rejectPending = reject;
 
-        return promise.then(this.createInitPromise.bind(this));
+        return promise.then(this.createInitPromise.bind(this)).finally(() => {
+            this.rejectPending = undefined;
+        });
     }
 
     private async selectTransport([transport, ...rest]: Transport[]): Promise<Transport> {