Skip to content

Commit 3bb3d58

Browse files
MultichainRoutingController -> MultichainRouter
1 parent 1fd2bd3 commit 3bb3d58

File tree

5 files changed

+98
-156
lines changed

5 files changed

+98
-156
lines changed

packages/snaps-controllers/coverage.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,5 @@
22
"branches": 93.11,
33
"functions": 96.79,
44
"lines": 98.06,
5-
"statements": 97.77
5+
"statements": 97.76
66
}

packages/snaps-controllers/src/multichain/MultichainRoutingController.test.ts renamed to packages/snaps-controllers/src/multichain/MultichainRouter.test.ts

Lines changed: 47 additions & 62 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@ import { HandlerType } from '@metamask/snaps-utils';
22
import { getTruncatedSnap } from '@metamask/snaps-utils/test-utils';
33

44
import {
5-
getRootMultichainRoutingControllerMessenger,
6-
getRestrictedMultichainRoutingControllerMessenger,
5+
getRootMultichainRouterMessenger,
6+
getRestrictedMultichainRouterMessenger,
77
BTC_CAIP2,
88
BTC_CONNECTED_ACCOUNTS,
99
MOCK_SOLANA_SNAP_PERMISSIONS,
@@ -12,16 +12,15 @@ import {
1212
MOCK_SOLANA_ACCOUNTS,
1313
MOCK_BTC_ACCOUNTS,
1414
} from '../test-utils';
15-
import { MultichainRoutingController } from './MultichainRoutingController';
15+
import { MultichainRouter } from './MultichainRouter';
1616

17-
describe('MultichainRoutingController', () => {
17+
describe('MultichainRouter', () => {
1818
it('can route signing requests to account Snaps without address resolution', async () => {
19-
const rootMessenger = getRootMultichainRoutingControllerMessenger();
20-
const messenger =
21-
getRestrictedMultichainRoutingControllerMessenger(rootMessenger);
19+
const rootMessenger = getRootMultichainRouterMessenger();
20+
const messenger = getRestrictedMultichainRouterMessenger(rootMessenger);
2221

2322
/* eslint-disable-next-line no-new */
24-
new MultichainRoutingController({
23+
new MultichainRouter({
2524
messenger,
2625
});
2726

@@ -47,32 +46,28 @@ describe('MultichainRoutingController', () => {
4746
},
4847
);
4948

50-
const result = await messenger.call(
51-
'MultichainRoutingController:handleRequest',
52-
{
53-
connectedAddresses: BTC_CONNECTED_ACCOUNTS,
54-
scope: BTC_CAIP2,
55-
request: {
56-
method: 'btc_sendmany',
57-
params: {
58-
message: 'foo',
59-
},
49+
const result = await messenger.call('MultichainRouter:handleRequest', {
50+
connectedAddresses: BTC_CONNECTED_ACCOUNTS,
51+
scope: BTC_CAIP2,
52+
request: {
53+
method: 'btc_sendmany',
54+
params: {
55+
message: 'foo',
6056
},
6157
},
62-
);
58+
});
6359

6460
expect(result).toStrictEqual({
6561
txid: '53de51e2fa75c3cfa51132865f7d430138b1cd92a8f5267ec836ec565b422969',
6662
});
6763
});
6864

6965
it('can route signing requests to account Snaps using address resolution', async () => {
70-
const rootMessenger = getRootMultichainRoutingControllerMessenger();
71-
const messenger =
72-
getRestrictedMultichainRoutingControllerMessenger(rootMessenger);
66+
const rootMessenger = getRootMultichainRouterMessenger();
67+
const messenger = getRestrictedMultichainRouterMessenger(rootMessenger);
7368

7469
/* eslint-disable-next-line no-new */
75-
new MultichainRoutingController({
70+
new MultichainRouter({
7671
messenger,
7772
});
7873

@@ -103,30 +98,26 @@ describe('MultichainRoutingController', () => {
10398
},
10499
);
105100

106-
const result = await messenger.call(
107-
'MultichainRoutingController:handleRequest',
108-
{
109-
connectedAddresses: SOLANA_CONNECTED_ACCOUNTS,
110-
scope: SOLANA_CAIP2,
111-
request: {
112-
method: 'signAndSendTransaction',
113-
params: {
114-
message: 'foo',
115-
},
101+
const result = await messenger.call('MultichainRouter:handleRequest', {
102+
connectedAddresses: SOLANA_CONNECTED_ACCOUNTS,
103+
scope: SOLANA_CAIP2,
104+
request: {
105+
method: 'signAndSendTransaction',
106+
params: {
107+
message: 'foo',
116108
},
117109
},
118-
);
110+
});
119111

120112
expect(result).toStrictEqual({ signature: '0x' });
121113
});
122114

123115
it('can route protocol requests to procotol Snaps', async () => {
124-
const rootMessenger = getRootMultichainRoutingControllerMessenger();
125-
const messenger =
126-
getRestrictedMultichainRoutingControllerMessenger(rootMessenger);
116+
const rootMessenger = getRootMultichainRouterMessenger();
117+
const messenger = getRestrictedMultichainRouterMessenger(rootMessenger);
127118

128119
/* eslint-disable-next-line no-new */
129-
new MultichainRoutingController({
120+
new MultichainRouter({
130121
messenger,
131122
});
132123

@@ -152,16 +143,13 @@ describe('MultichainRoutingController', () => {
152143
}),
153144
);
154145

155-
const result = await messenger.call(
156-
'MultichainRoutingController:handleRequest',
157-
{
158-
connectedAddresses: [],
159-
scope: SOLANA_CAIP2,
160-
request: {
161-
method: 'getVersion',
162-
},
146+
const result = await messenger.call('MultichainRouter:handleRequest', {
147+
connectedAddresses: [],
148+
scope: SOLANA_CAIP2,
149+
request: {
150+
method: 'getVersion',
163151
},
164-
);
152+
});
165153

166154
expect(result).toStrictEqual({
167155
'feature-set': 2891131721,
@@ -170,12 +158,11 @@ describe('MultichainRoutingController', () => {
170158
});
171159

172160
it('throws if no suitable Snaps are found', async () => {
173-
const rootMessenger = getRootMultichainRoutingControllerMessenger();
174-
const messenger =
175-
getRestrictedMultichainRoutingControllerMessenger(rootMessenger);
161+
const rootMessenger = getRootMultichainRouterMessenger();
162+
const messenger = getRestrictedMultichainRouterMessenger(rootMessenger);
176163

177164
/* eslint-disable-next-line no-new */
178-
new MultichainRoutingController({
165+
new MultichainRouter({
179166
messenger,
180167
});
181168

@@ -189,7 +176,7 @@ describe('MultichainRoutingController', () => {
189176
});
190177

191178
await expect(
192-
messenger.call('MultichainRoutingController:handleRequest', {
179+
messenger.call('MultichainRouter:handleRequest', {
193180
connectedAddresses: [],
194181
scope: SOLANA_CAIP2,
195182
request: {
@@ -200,12 +187,11 @@ describe('MultichainRoutingController', () => {
200187
});
201188

202189
it('throws if address resolution fails', async () => {
203-
const rootMessenger = getRootMultichainRoutingControllerMessenger();
204-
const messenger =
205-
getRestrictedMultichainRoutingControllerMessenger(rootMessenger);
190+
const rootMessenger = getRootMultichainRouterMessenger();
191+
const messenger = getRestrictedMultichainRouterMessenger(rootMessenger);
206192

207193
/* eslint-disable-next-line no-new */
208-
new MultichainRoutingController({
194+
new MultichainRouter({
209195
messenger,
210196
});
211197

@@ -231,7 +217,7 @@ describe('MultichainRoutingController', () => {
231217
);
232218

233219
await expect(
234-
messenger.call('MultichainRoutingController:handleRequest', {
220+
messenger.call('MultichainRouter:handleRequest', {
235221
connectedAddresses: SOLANA_CONNECTED_ACCOUNTS,
236222
scope: SOLANA_CAIP2,
237223
request: {
@@ -245,12 +231,11 @@ describe('MultichainRoutingController', () => {
245231
});
246232

247233
it('throws if address resolution returns an address that isnt available', async () => {
248-
const rootMessenger = getRootMultichainRoutingControllerMessenger();
249-
const messenger =
250-
getRestrictedMultichainRoutingControllerMessenger(rootMessenger);
234+
const rootMessenger = getRootMultichainRouterMessenger();
235+
const messenger = getRestrictedMultichainRouterMessenger(rootMessenger);
251236

252237
/* eslint-disable-next-line no-new */
253-
new MultichainRoutingController({
238+
new MultichainRouter({
254239
messenger,
255240
});
256241

@@ -279,7 +264,7 @@ describe('MultichainRoutingController', () => {
279264
);
280265

281266
await expect(
282-
messenger.call('MultichainRoutingController:handleRequest', {
267+
messenger.call('MultichainRouter:handleRequest', {
283268
connectedAddresses: SOLANA_CONNECTED_ACCOUNTS,
284269
scope: SOLANA_CAIP2,
285270
request: {

0 commit comments

Comments
 (0)