|
|
|
import { strict as assert } from 'assert';
|
|
|
|
import sinon from 'sinon';
|
|
|
|
import { MAINNET_CHAIN_ID } from '../../../shared/constants/network';
|
|
|
|
import PreferencesController from './preferences';
|
|
|
|
import NetworkController from './network';
|
|
|
|
|
|
|
|
describe('preferences controller', function () {
|
|
|
|
let preferencesController;
|
|
|
|
let network;
|
|
|
|
let currentChainId;
|
|
|
|
let provider;
|
|
|
|
const migrateAddressBookState = sinon.stub();
|
|
|
|
|
|
|
|
beforeEach(function () {
|
|
|
|
const sandbox = sinon.createSandbox();
|
|
|
|
currentChainId = MAINNET_CHAIN_ID;
|
|
|
|
const networkControllerProviderConfig = {
|
|
|
|
getAccounts: () => undefined,
|
|
|
|
};
|
|
|
|
network = new NetworkController();
|
|
|
|
network.setInfuraProjectId('foo');
|
|
|
|
network.initializeProvider(networkControllerProviderConfig);
|
|
|
|
provider = network.getProviderAndBlockTracker().provider;
|
|
|
|
|
|
|
|
sandbox
|
|
|
|
.stub(network, 'getLatestBlock')
|
|
|
|
.callsFake(() => Promise.resolve({}));
|
|
|
|
sandbox.stub(network, 'getCurrentChainId').callsFake(() => currentChainId);
|
|
|
|
sandbox
|
|
|
|
.stub(network, 'getProviderConfig')
|
|
|
|
.callsFake(() => ({ type: 'mainnet' }));
|
|
|
|
|
|
|
|
preferencesController = new PreferencesController({
|
|
|
|
migrateAddressBookState,
|
|
|
|
network,
|
|
|
|
provider,
|
|
|
|
});
|
|
|
|
});
|
Update address book state upon custom RPC chainId edit (#9493)
When the `chainId` for a custom RPC endpoint is edited, we now migrate
the corresponding address book entries to ensure they are not orphaned.
The address book entries are grouped by the `metamask.network` state,
which unfortunately was sometimes the `chainId`, and sometimes the
`networkId`. It was always the `networkId` for built-in Infura
networks, but for custom RPC endpoints it would be set to the user-set
`chainId` field, with a fallback to the `networkId` of the network.
A recent change will force users to enter valid `chainId`s on all
custom networks, which will be normalized to be hex-prefixed. As a
result, address book contacts will now be keyed by a different string.
The contact entries are now migrated when this edit takes place.
There are some edge cases where two separate entries share the same set
of contacts. For example, if two entries have the same `chainId`, or if
they had the same `networkId` and had no `chainId` set. When the
`chainId` is edited in such cases, the contacts are duplicated on both
networks. This is the best we can do, as we don't have any way to know
which network the contacts _should_ be on.
The `typed-message-manager` unit tests have also been updated as part
of this commit because the addition of `sinon.restore()` to the
preferences controller tests ended up clearing a test object in-between
individual tests in that file. The test object is now re-constructed
before each individual test.
4 years ago
|
|
|
|
|
|
|
afterEach(function () {
|
|
|
|
sinon.restore();
|
|
|
|
});
|
|
|
|
|
|
|
|
describe('setAddresses', function () {
|
|
|
|
it('should keep a map of addresses to names and addresses in the store', function () {
|
|
|
|
preferencesController.setAddresses(['0xda22le', '0x7e57e2']);
|
|
|
|
|
|
|
|
const { identities } = preferencesController.store.getState();
|
|
|
|
assert.deepEqual(identities, {
|
|
|
|
'0xda22le': {
|
|
|
|
name: 'Account 1',
|
|
|
|
address: '0xda22le',
|
|
|
|
},
|
|
|
|
'0x7e57e2': {
|
|
|
|
name: 'Account 2',
|
|
|
|
address: '0x7e57e2',
|
|
|
|
},
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
it('should replace its list of addresses', function () {
|
|
|
|
preferencesController.setAddresses(['0xda22le', '0x7e57e2']);
|
|
|
|
preferencesController.setAddresses(['0xda22le77', '0x7e57e277']);
|
|
|
|
|
|
|
|
const { identities } = preferencesController.store.getState();
|
|
|
|
assert.deepEqual(identities, {
|
|
|
|
'0xda22le77': {
|
|
|
|
name: 'Account 1',
|
|
|
|
address: '0xda22le77',
|
|
|
|
},
|
|
|
|
'0x7e57e277': {
|
|
|
|
name: 'Account 2',
|
|
|
|
address: '0x7e57e277',
|
|
|
|
},
|
|
|
|
});
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
describe('removeAddress', function () {
|
|
|
|
it('should remove an address from state', function () {
|
|
|
|
preferencesController.setAddresses(['0xda22le', '0x7e57e2']);
|
|
|
|
|
|
|
|
preferencesController.removeAddress('0xda22le');
|
|
|
|
|
|
|
|
assert.equal(
|
|
|
|
preferencesController.store.getState().identities['0xda22le'],
|
|
|
|
undefined,
|
|
|
|
);
|
|
|
|
});
|
|
|
|
|
|
|
|
it('should switch accounts if the selected address is removed', function () {
|
|
|
|
preferencesController.setAddresses(['0xda22le', '0x7e57e2']);
|
|
|
|
|
|
|
|
preferencesController.setSelectedAddress('0x7e57e2');
|
|
|
|
preferencesController.removeAddress('0x7e57e2');
|
|
|
|
|
|
|
|
assert.equal(preferencesController.getSelectedAddress(), '0xda22le');
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
describe('setAccountLabel', function () {
|
|
|
|
it('should update a label for the given account', function () {
|
|
|
|
preferencesController.setAddresses(['0xda22le', '0x7e57e2']);
|
|
|
|
|
|
|
|
assert.deepEqual(
|
|
|
|
preferencesController.store.getState().identities['0xda22le'],
|
|
|
|
{
|
|
|
|
name: 'Account 1',
|
|
|
|
address: '0xda22le',
|
|
|
|
},
|
|
|
|
);
|
|
|
|
|
|
|
|
preferencesController.setAccountLabel('0xda22le', 'Dazzle');
|
|
|
|
assert.deepEqual(
|
|
|
|
preferencesController.store.getState().identities['0xda22le'],
|
|
|
|
{
|
|
|
|
name: 'Dazzle',
|
|
|
|
address: '0xda22le',
|
|
|
|
},
|
|
|
|
);
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
describe('setPasswordForgotten', function () {
|
|
|
|
it('should default to false', function () {
|
|
|
|
const state = preferencesController.store.getState();
|
|
|
|
assert.equal(state.forgottenPassword, false);
|
|
|
|
});
|
|
|
|
|
|
|
|
it('should set the forgottenPassword property in state', function () {
|
|
|
|
assert.equal(
|
|
|
|
preferencesController.store.getState().forgottenPassword,
|
|
|
|
false,
|
|
|
|
);
|
|
|
|
|
|
|
|
preferencesController.setPasswordForgotten(true);
|
|
|
|
|
|
|
|
assert.equal(
|
|
|
|
preferencesController.store.getState().forgottenPassword,
|
|
|
|
true,
|
|
|
|
);
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
describe('#updateRpc', function () {
|
Update address book state upon custom RPC chainId edit (#9493)
When the `chainId` for a custom RPC endpoint is edited, we now migrate
the corresponding address book entries to ensure they are not orphaned.
The address book entries are grouped by the `metamask.network` state,
which unfortunately was sometimes the `chainId`, and sometimes the
`networkId`. It was always the `networkId` for built-in Infura
networks, but for custom RPC endpoints it would be set to the user-set
`chainId` field, with a fallback to the `networkId` of the network.
A recent change will force users to enter valid `chainId`s on all
custom networks, which will be normalized to be hex-prefixed. As a
result, address book contacts will now be keyed by a different string.
The contact entries are now migrated when this edit takes place.
There are some edge cases where two separate entries share the same set
of contacts. For example, if two entries have the same `chainId`, or if
they had the same `networkId` and had no `chainId` set. When the
`chainId` is edited in such cases, the contacts are duplicated on both
networks. This is the best we can do, as we don't have any way to know
which network the contacts _should_ be on.
The `typed-message-manager` unit tests have also been updated as part
of this commit because the addition of `sinon.restore()` to the
preferences controller tests ended up clearing a test object in-between
individual tests in that file. The test object is now re-constructed
before each individual test.
4 years ago
|
|
|
it('should update the rpcDetails properly', async function () {
|
|
|
|
preferencesController.store.updateState({
|
|
|
|
frequentRpcListDetail: [{}, { rpcUrl: 'test', chainId: '0x1' }, {}],
|
|
|
|
});
|
|
|
|
await preferencesController.updateRpc({ rpcUrl: 'test', chainId: '0x1' });
|
|
|
|
await preferencesController.updateRpc({
|
|
|
|
rpcUrl: 'test/1',
|
|
|
|
chainId: '0x1',
|
|
|
|
});
|
|
|
|
await preferencesController.updateRpc({
|
|
|
|
rpcUrl: 'test/2',
|
|
|
|
chainId: '0x1',
|
|
|
|
});
|
|
|
|
await preferencesController.updateRpc({
|
|
|
|
rpcUrl: 'test/3',
|
|
|
|
chainId: '0x1',
|
|
|
|
});
|
|
|
|
const list = preferencesController.getFrequentRpcListDetail();
|
|
|
|
assert.deepEqual(list[1], { rpcUrl: 'test', chainId: '0x1' });
|
|
|
|
});
|
Update address book state upon custom RPC chainId edit (#9493)
When the `chainId` for a custom RPC endpoint is edited, we now migrate
the corresponding address book entries to ensure they are not orphaned.
The address book entries are grouped by the `metamask.network` state,
which unfortunately was sometimes the `chainId`, and sometimes the
`networkId`. It was always the `networkId` for built-in Infura
networks, but for custom RPC endpoints it would be set to the user-set
`chainId` field, with a fallback to the `networkId` of the network.
A recent change will force users to enter valid `chainId`s on all
custom networks, which will be normalized to be hex-prefixed. As a
result, address book contacts will now be keyed by a different string.
The contact entries are now migrated when this edit takes place.
There are some edge cases where two separate entries share the same set
of contacts. For example, if two entries have the same `chainId`, or if
they had the same `networkId` and had no `chainId` set. When the
`chainId` is edited in such cases, the contacts are duplicated on both
networks. This is the best we can do, as we don't have any way to know
which network the contacts _should_ be on.
The `typed-message-manager` unit tests have also been updated as part
of this commit because the addition of `sinon.restore()` to the
preferences controller tests ended up clearing a test object in-between
individual tests in that file. The test object is now re-constructed
before each individual test.
4 years ago
|
|
|
|
|
|
|
it('should migrate address book entries if chainId changes', async function () {
|
|
|
|
preferencesController.store.updateState({
|
|
|
|
frequentRpcListDetail: [{}, { rpcUrl: 'test', chainId: '1' }, {}],
|
|
|
|
});
|
|
|
|
await preferencesController.updateRpc({ rpcUrl: 'test', chainId: '0x1' });
|
|
|
|
assert(migrateAddressBookState.calledWith('1', '0x1'));
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
describe('adding and removing from frequentRpcListDetail', function () {
|
|
|
|
it('should add custom RPC url to state', function () {
|
|
|
|
preferencesController.addToFrequentRpcList('rpc_url', '0x1');
|
|
|
|
assert.deepEqual(
|
|
|
|
preferencesController.store.getState().frequentRpcListDetail,
|
|
|
|
[
|
|
|
|
{
|
|
|
|
rpcUrl: 'rpc_url',
|
|
|
|
chainId: '0x1',
|
|
|
|
ticker: 'ETH',
|
|
|
|
nickname: '',
|
|
|
|
rpcPrefs: {},
|
|
|
|
},
|
|
|
|
],
|
|
|
|
);
|
|
|
|
preferencesController.addToFrequentRpcList('rpc_url', '0x1');
|
|
|
|
assert.deepEqual(
|
|
|
|
preferencesController.store.getState().frequentRpcListDetail,
|
|
|
|
[
|
|
|
|
{
|
|
|
|
rpcUrl: 'rpc_url',
|
|
|
|
chainId: '0x1',
|
|
|
|
ticker: 'ETH',
|
|
|
|
nickname: '',
|
|
|
|
rpcPrefs: {},
|
|
|
|
},
|
|
|
|
],
|
|
|
|
);
|
|
|
|
});
|
|
|
|
|
|
|
|
it('should throw if chainId is invalid', function () {
|
|
|
|
assert.throws(() => {
|
|
|
|
preferencesController.addToFrequentRpcList('rpc_url', '1');
|
|
|
|
}, 'should throw on invalid chainId');
|
|
|
|
});
|
|
|
|
|
|
|
|
it('should remove custom RPC url from state', function () {
|
|
|
|
preferencesController.addToFrequentRpcList('rpc_url', '0x1');
|
|
|
|
assert.deepEqual(
|
|
|
|
preferencesController.store.getState().frequentRpcListDetail,
|
|
|
|
[
|
|
|
|
{
|
|
|
|
rpcUrl: 'rpc_url',
|
|
|
|
chainId: '0x1',
|
|
|
|
ticker: 'ETH',
|
|
|
|
nickname: '',
|
|
|
|
rpcPrefs: {},
|
|
|
|
},
|
|
|
|
],
|
|
|
|
);
|
|
|
|
preferencesController.removeFromFrequentRpcList('other_rpc_url');
|
|
|
|
preferencesController.removeFromFrequentRpcList('http://localhost:8545');
|
|
|
|
preferencesController.removeFromFrequentRpcList('rpc_url');
|
|
|
|
assert.deepEqual(
|
|
|
|
preferencesController.store.getState().frequentRpcListDetail,
|
|
|
|
[],
|
|
|
|
);
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
describe('setUsePhishDetect', function () {
|
|
|
|
it('should default to true', function () {
|
|
|
|
const state = preferencesController.store.getState();
|
|
|
|
assert.equal(state.usePhishDetect, true);
|
|
|
|
});
|
|
|
|
|
|
|
|
it('should set the usePhishDetect property in state', function () {
|
|
|
|
assert.equal(preferencesController.store.getState().usePhishDetect, true);
|
|
|
|
preferencesController.setUsePhishDetect(false);
|
|
|
|
assert.equal(
|
|
|
|
preferencesController.store.getState().usePhishDetect,
|
|
|
|
false,
|
|
|
|
);
|
|
|
|
});
|
|
|
|
});
|
|
|
|
describe('setUseTokenDetection', function () {
|
|
|
|
it('should default to false', function () {
|
|
|
|
const state = preferencesController.store.getState();
|
|
|
|
assert.equal(state.useTokenDetection, false);
|
|
|
|
});
|
|
|
|
|
|
|
|
it('should set the useTokenDetection property in state', function () {
|
|
|
|
assert.equal(
|
|
|
|
preferencesController.store.getState().useTokenDetection,
|
|
|
|
false,
|
|
|
|
);
|
|
|
|
preferencesController.setUseTokenDetection(true);
|
|
|
|
assert.equal(
|
|
|
|
preferencesController.store.getState().useTokenDetection,
|
|
|
|
true,
|
|
|
|
);
|
|
|
|
});
|
|
|
|
});
|
|
|
|
|
|
|
|
describe('setAdvancedGasFee', function () {
|
|
|
|
it('should default to null', function () {
|
|
|
|
const state = preferencesController.store.getState();
|
|
|
|
assert.equal(state.advancedGasFee, null);
|
|
|
|
});
|
|
|
|
|
|
|
|
it('should set the setAdvancedGasFee property in state', function () {
|
|
|
|
const state = preferencesController.store.getState();
|
|
|
|
assert.equal(state.advancedGasFee, null);
|
|
|
|
preferencesController.setAdvancedGasFee({
|
|
|
|
maxBaseFee: '1.5',
|
|
|
|
priorityFee: '2',
|
|
|
|
});
|
|
|
|
assert.equal(
|
|
|
|
preferencesController.store.getState().advancedGasFee.maxBaseFee,
|
|
|
|
'1.5',
|
|
|
|
);
|
|
|
|
assert.equal(
|
|
|
|
preferencesController.store.getState().advancedGasFee.priorityFee,
|
|
|
|
'2',
|
|
|
|
);
|
|
|
|
});
|
|
|
|
});
|
|
|
|
});
|