mirror of
https://github.com/kremalicious/metamask-extension.git
synced 2024-11-29 15:50:28 +01:00
022ef13372
This migration had referred to the non-existent `TransactionsController` instead of `TransactionController`, so it effectively did nothing. Now it should work. This migration hasn't been included in any release yet, so we can fix it in-place instead of adding an additional corrected migration. The migration comment has also been updated, as it was inaccurate.
110 lines
2.8 KiB
JavaScript
110 lines
2.8 KiB
JavaScript
import { strict as assert } from 'assert'
|
|
import migration47 from '../../../app/scripts/migrations/047'
|
|
|
|
describe('migration #47', function () {
|
|
it('should update the version metadata', async function () {
|
|
const oldStorage = {
|
|
'meta': {
|
|
'version': 46,
|
|
},
|
|
'data': {},
|
|
}
|
|
|
|
const newStorage = await migration47.migrate(oldStorage)
|
|
assert.deepEqual(newStorage.meta, {
|
|
'version': 47,
|
|
})
|
|
})
|
|
|
|
it('should stringify transactions metamaskNetworkId values', async function () {
|
|
const oldStorage = {
|
|
meta: {},
|
|
data: {
|
|
TransactionController: {
|
|
transactions: [
|
|
{ foo: 'bar', metamaskNetworkId: 2 },
|
|
{ foo: 'bar' },
|
|
{ foo: 'bar', metamaskNetworkId: 0 },
|
|
{ foo: 'bar', metamaskNetworkId: 42 },
|
|
],
|
|
},
|
|
foo: 'bar',
|
|
},
|
|
}
|
|
|
|
const newStorage = await migration47.migrate(oldStorage)
|
|
assert.deepEqual(newStorage.data, {
|
|
TransactionController: {
|
|
transactions: [
|
|
{ foo: 'bar', metamaskNetworkId: '2' },
|
|
{ foo: 'bar' },
|
|
{ foo: 'bar', metamaskNetworkId: '0' },
|
|
{ foo: 'bar', metamaskNetworkId: '42' },
|
|
],
|
|
},
|
|
foo: 'bar',
|
|
})
|
|
})
|
|
|
|
it('should do nothing if transactions metamaskNetworkId values are already strings', async function () {
|
|
const oldStorage = {
|
|
meta: {},
|
|
data: {
|
|
TransactionController: {
|
|
transactions: [
|
|
{ foo: 'bar', metamaskNetworkId: '2' },
|
|
{ foo: 'bar' },
|
|
{ foo: 'bar', metamaskNetworkId: '0' },
|
|
{ foo: 'bar', metamaskNetworkId: '42' },
|
|
],
|
|
},
|
|
foo: 'bar',
|
|
},
|
|
}
|
|
|
|
const newStorage = await migration47.migrate(oldStorage)
|
|
assert.deepEqual(oldStorage.data, newStorage.data)
|
|
})
|
|
|
|
it('should do nothing if transactions state does not exist', async function () {
|
|
const oldStorage = {
|
|
meta: {},
|
|
data: {
|
|
TransactionController: {
|
|
bar: 'baz',
|
|
},
|
|
foo: 'bar',
|
|
},
|
|
}
|
|
|
|
const newStorage = await migration47.migrate(oldStorage)
|
|
assert.deepEqual(oldStorage.data, newStorage.data)
|
|
})
|
|
|
|
it('should do nothing if transactions state is empty', async function () {
|
|
const oldStorage = {
|
|
meta: {},
|
|
data: {
|
|
TransactionController: {
|
|
transactions: [],
|
|
bar: 'baz',
|
|
},
|
|
foo: 'bar',
|
|
},
|
|
}
|
|
|
|
const newStorage = await migration47.migrate(oldStorage)
|
|
assert.deepEqual(oldStorage.data, newStorage.data)
|
|
})
|
|
|
|
it('should do nothing if state is empty', async function () {
|
|
const oldStorage = {
|
|
meta: {},
|
|
data: {},
|
|
}
|
|
|
|
const newStorage = await migration47.migrate(oldStorage)
|
|
assert.deepEqual(oldStorage.data, newStorage.data)
|
|
})
|
|
})
|