@onekeyfe/hd-core 1.2.0-alpha.40 → 1.2.0-alpha.41

package/__tests__/protocol-v2-unlock-policy.test.ts

@@ -1,96 +1,45 @@
-import ConfluxSignMessageCIP23 from '../src/api/conflux/ConfluxSignMessageCIP23';
-import DeviceLock from '../src/api/device/DeviceLock';
-import { runMethodWithUnlockPolicy } from '../src/protocols/protocol-v2/unlockPolicyRunner';
+import * as publicMethods from '../src/api';
+import {
+  PROTOCOL_V2_RETRY_ON_LOCKED_METHODS,
+  getProtocolV2UnlockPolicy,
+} from '../src/protocols/protocol-v2/unlockPolicy';
 
 jest.mock('../src/data/config', () => ({
   DEFAULT_DOMAIN: 'https://example.com/',
   getSDKVersion: () => '0.0.0-test',
 }));
 
-describe('Protocol V2 unlock semantics', () => {
-  test('wallet business methods inherit the wallet-session unlock requirement', () => {
-    const method = new ConfluxSignMessageCIP23({
-      id: 1,
-      payload: { method: 'confluxSignMessageCIP23' },
-    });
-
-    expect(method.useDevicePassphraseState).toBe(true);
-    expect(method.unlockPolicy).toBe('none');
-    expect(method.getSupportedProtocols()).toContain('V2');
-  });
-
-  test('lock-free Protocol V2 controls explicitly opt out of wallet-session handling', () => {
-    const method = new DeviceLock({
-      id: 1,
-      payload: { method: 'deviceLock' },
-    });
-
-    method.init();
-
-    expect(method.useDevicePassphraseState).toBe(false);
-    expect(method.unlockPolicy).toBe('none');
+describe('Protocol V2 public method unlock policy', () => {
+  test('classifies the complete public method table with an explicit replay allowlist', () => {
+    const publicMethodNames = Object.keys(publicMethods);
+    const retryMethodNames = new Set<string>(PROTOCOL_V2_RETRY_ON_LOCKED_METHODS);
+
+    expect(publicMethodNames.length).toBeGreaterThan(100);
+    expect(new Set(PROTOCOL_V2_RETRY_ON_LOCKED_METHODS).size).toBe(
+      PROTOCOL_V2_RETRY_ON_LOCKED_METHODS.length
+    );
+
+    for (const methodName of PROTOCOL_V2_RETRY_ON_LOCKED_METHODS) {
+      expect(publicMethods).toHaveProperty(methodName);
+    }
+
+    for (const methodName of publicMethodNames) {
+      expect(getProtocolV2UnlockPolicy(methodName)).toBe(
+        retryMethodNames.has(methodName) ? 'retry-on-locked' : 'none'
+      );
+    }
   });
 
-  test('pre-unlocks an unregistered wallet business method', async () => {
-    const features = { unlocked: false };
-    const method = {
-      name: 'newWalletBusinessMethod',
-      unlockPolicy: 'none',
-      useDevicePassphraseState: true,
-      run: jest.fn().mockResolvedValue({ message: 'ok' }),
-    };
-    const device = {
-      features,
-      commands: {
-        typedCall: jest.fn().mockResolvedValue({ message: { unlocked: false } }),
-      },
-      isProtocolV2: () => true,
-      isBootloader: () => false,
-      isRomloader: () => false,
-      updateProtocolV2Status: jest.fn(() => features),
-      unlockDevice: jest.fn().mockImplementation(() => {
-        features.unlocked = true;
-        return Promise.resolve(features);
-      }),
-    };
-
-    await expect(runMethodWithUnlockPolicy(method as any, device as any)).resolves.toEqual({
-      message: 'ok',
-    });
-    expect(device.unlockDevice).toHaveBeenCalledTimes(1);
-    expect(method.run).toHaveBeenCalledTimes(1);
-  });
-
-  test('runs fresh-status validation before starting unlock', async () => {
-    const identityError = new Error('Unexpected device');
-    const method = {
-      name: 'walletBusinessMethod',
-      unlockPolicy: 'none',
-      useDevicePassphraseState: true,
-      run: jest.fn(),
-    };
-    const features = { unlocked: false };
-    const device = {
-      features,
-      commands: {
-        typedCall: jest.fn().mockResolvedValue({ message: { unlocked: false } }),
-      },
-      isProtocolV2: () => true,
-      isBootloader: () => false,
-      isRomloader: () => false,
-      updateProtocolV2Status: jest.fn(() => features),
-      unlockDevice: jest.fn(),
-    };
-
-    await expect(
-      runMethodWithUnlockPolicy(method as any, device as any, {
-        afterStatusBeforeUnlock: () => {
-          throw identityError;
-        },
-      })
-    ).rejects.toBe(identityError);
-
-    expect(device.unlockDevice).not.toHaveBeenCalled();
-    expect(method.run).not.toHaveBeenCalled();
+  test.each([
+    'deviceUnlock',
+    'deviceLock',
+    'deviceWipe',
+    'deviceChangePin',
+    'deviceSettings',
+    'firmwareUpdateV4',
+    'deviceUploadWallpaper',
+    'uploadPortfolio',
+  ])('does not make the state-changing method %s replayable by default', methodName => {
+    expect(getProtocolV2UnlockPolicy(methodName)).toBe('none');
   });
 });