Skip to content

Commit 47939ab

Browse files
committed
fix test mocks
1 parent d6dc5b5 commit 47939ab

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

packages/assets-controllers/src/NftController.test.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -162,6 +162,7 @@ jest.mock('uuid', () => {
162162
* @param args.bulkScanUrlsMock - Used to construct mock versions of the
163163
* `PhishingController:bulkScanUrls` action.
164164
* @param args.defaultSelectedAccount - The default selected account to use in
165+
* @param args.displayNftMedia - The default displayNftMedia to use in
165166
* @returns A collection of test controllers and mocks.
166167
*/
167168
function setupController({
@@ -178,6 +179,7 @@ function setupController({
178179
mockNetworkClientConfigurationsByNetworkClientId = {},
179180
defaultSelectedAccount = OWNER_ACCOUNT,
180181
mockGetNetworkClientIdByChainId = {},
182+
displayNftMedia = true,
181183
}: {
182184
options?: Partial<ConstructorParameters<typeof NftController>[0]>;
183185
getERC721AssetName?: jest.Mock<
@@ -222,6 +224,7 @@ function setupController({
222224
>;
223225
defaultSelectedAccount?: InternalAccount;
224226
mockGetNetworkClientIdByChainId?: Record<Hex, NetworkClientConfiguration>;
227+
displayNftMedia?: boolean;
225228
} = {}) {
226229
const messenger = new Messenger<
227230
| ExtractAvailableAction<NftControllerMessenger>
@@ -388,7 +391,7 @@ function setupController({
388391

389392
triggerPreferencesStateChange({
390393
...getDefaultPreferencesState(),
391-
displayNftMedia: true,
394+
displayNftMedia,
392395
});
393396

394397
const triggerSelectedAccountChange = (
@@ -4946,7 +4949,7 @@ describe('NftController', () => {
49464949
expect(spy).toHaveBeenCalledTimes(0);
49474950
});
49484951

4949-
it('should call update Nft metadata when preferences change is triggered and ipfsGateway changes', async () => {
4952+
it('calls update Nft metadata when preferences change is triggered and ipfsGateway changes', async () => {
49504953
const {
49514954
nftController,
49524955
mockGetAccount,
@@ -4980,6 +4983,7 @@ describe('NftController', () => {
49804983
triggerSelectedAccountChange,
49814984
} = setupController({
49824985
defaultSelectedAccount: OWNER_ACCOUNT,
4986+
displayNftMedia: false,
49834987
});
49844988
const spy = jest.spyOn(nftController, 'updateNftMetadata');
49854989
const testNetworkClientId = 'mainnet';
@@ -5006,6 +5010,7 @@ describe('NftController', () => {
50065010
triggerSelectedAccountChange,
50075011
} = setupController({
50085012
defaultSelectedAccount: OWNER_ACCOUNT,
5013+
displayNftMedia: false,
50095014
});
50105015
const spy = jest.spyOn(nftController, 'updateNftMetadata');
50115016
const testNetworkClientId = 'mainnet';

0 commit comments

Comments
 (0)