Skip to content

Commit 248aaad

Browse files
committed
refactor: use new literal for wallet.status
1 parent f199f28 commit 248aaad

File tree

4 files changed

+181
-238
lines changed

4 files changed

+181
-238
lines changed

packages/multichain-account-service/src/MultichainAccountWallet.test.ts

Lines changed: 8 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,7 @@ import {
1515
} from '@metamask/keyring-api';
1616
import type { InternalAccount } from '@metamask/keyring-internal-api';
1717

18-
import {
19-
MultichainAccountWallet,
20-
MultichainAccountWalletStatus,
21-
} from './MultichainAccountWallet';
18+
import { MultichainAccountWallet } from './MultichainAccountWallet';
2219
import type { MockAccountProvider } from './tests';
2320
import {
2421
MOCK_HD_ACCOUNT_1,
@@ -103,7 +100,7 @@ describe('MultichainAccountWallet', () => {
103100

104101
const expectedWalletId = toMultichainAccountWalletId(entropySource);
105102
expect(wallet.id).toStrictEqual(expectedWalletId);
106-
expect(wallet.status).toStrictEqual(MultichainAccountWalletStatus.Ready);
103+
expect(wallet.status).toBe('ready');
107104
expect(wallet.type).toBe(AccountWalletType.Entropy);
108105
expect(wallet.entropySource).toStrictEqual(entropySource);
109106
expect(wallet.getMultichainAccountGroups()).toHaveLength(1); // All internal accounts are using index 0, so it means only 1 multichain account.
@@ -402,14 +399,12 @@ describe('MultichainAccountWallet', () => {
402399
// 1. Triggered when group alignment begins.
403400
.mockImplementationOnce((walletId, status) => {
404401
expect(walletId).toBe(wallet.id);
405-
expect(status).toBe(
406-
MultichainAccountWalletStatus.AlignmentInProgress,
407-
);
402+
expect(status).toBe('in-progress:alignment');
408403
})
409404
// 2. Triggered when group alignment ends.
410405
.mockImplementationOnce((walletId, status) => {
411406
expect(walletId).toBe(wallet.id);
412-
expect(status).toBe(MultichainAccountWalletStatus.Ready);
407+
expect(status).toBe('ready');
413408
});
414409

415410
messenger.subscribe(
@@ -449,14 +444,12 @@ describe('MultichainAccountWallet', () => {
449444
// 1. Triggered when group alignment begins.
450445
.mockImplementationOnce((walletId, status) => {
451446
expect(walletId).toBe(wallet.id);
452-
expect(status).toBe(
453-
MultichainAccountWalletStatus.AlignmentInProgress,
454-
);
447+
expect(status).toBe('in-progress:alignment');
455448
})
456449
// 2. Triggered when group alignment ends.
457450
.mockImplementationOnce((walletId, status) => {
458451
expect(walletId).toBe(wallet.id);
459-
expect(status).toBe(MultichainAccountWalletStatus.Ready);
452+
expect(status).toBe('ready');
460453
});
461454

462455
messenger.subscribe(
@@ -500,14 +493,12 @@ describe('MultichainAccountWallet', () => {
500493
// 1. Triggered when group alignment begins.
501494
.mockImplementationOnce((walletId, status) => {
502495
expect(walletId).toBe(wallet.id);
503-
expect(status).toBe(
504-
MultichainAccountWalletStatus.DiscoveryInProgress,
505-
);
496+
expect(status).toBe('in-progress:discovery');
506497
})
507498
// 2. Triggered when group alignment ends.
508499
.mockImplementationOnce((walletId, status) => {
509500
expect(walletId).toBe(wallet.id);
510-
expect(status).toBe(MultichainAccountWalletStatus.Ready);
501+
expect(status).toBe('ready');
511502
});
512503

513504
messenger.subscribe(

0 commit comments

Comments
 (0)