From 3c52e8071368b38690e3c2d45abb8fb0ca28551d Mon Sep 17 00:00:00 2001 From: Mark Stacey Date: Tue, 17 Jan 2023 19:57:41 -0330 Subject: [PATCH] Remove unnecessary test (#17181) This test was testing a function that was only present in the test module. The function under test was mistakenly moved here when it was discovered that it wasn't being used elsewhere, under the assumption that it was used in these tests. I hadn't realized it was being tested directly. --- .../network/network-controller.test.js | 26 ------------------- 1 file changed, 26 deletions(-) diff --git a/app/scripts/controllers/network/network-controller.test.js b/app/scripts/controllers/network/network-controller.test.js index dd1cf8846..ffe450f2e 100644 --- a/app/scripts/controllers/network/network-controller.test.js +++ b/app/scripts/controllers/network/network-controller.test.js @@ -1,10 +1,7 @@ import nock from 'nock'; import { deferredPromise } from '../../lib/util'; -import { NETWORK_TO_NAME_MAP } from '../../../../shared/constants/network'; import NetworkController, { NETWORK_EVENTS } from './network-controller'; -const getNetworkDisplayName = (key) => NETWORK_TO_NAME_MAP[key]; - /** * Construct a successful RPC response. * @@ -198,27 +195,4 @@ describe('NetworkController', () => { }); }); }); - - describe('utils', () => { - it('getNetworkDisplayName should return the correct network name', () => { - const tests = [ - { - input: 'mainnet', - expected: 'Ethereum Mainnet', - }, - { - input: 'goerli', - expected: 'Goerli', - }, - { - input: 'sepolia', - expected: 'Sepolia', - }, - ]; - - tests.forEach(({ input, expected }) => - expect(getNetworkDisplayName(input)).toStrictEqual(expected), - ); - }); - }); });