|
|
|
const assert = require('assert')
|
|
|
|
const EventEmitter = require('events')
|
|
|
|
const ethUtil = require('ethereumjs-util')
|
|
|
|
const EthTx = require('ethereumjs-tx')
|
|
|
|
const EthQuery = require('eth-query')
|
|
|
|
const ObservableStore = require('obs-store')
|
|
|
|
const clone = require('clone')
|
|
|
|
const sinon = require('sinon')
|
|
|
|
const TransactionController = require('../../app/scripts/controllers/transactions')
|
|
|
|
const noop = () => true
|
|
|
|
const currentNetworkId = 42
|
|
|
|
const otherNetworkId = 36
|
|
|
|
const privKey = new Buffer('8718b9618a37d1fc78c436511fc6df3c8258d3250635bba617f33003270ec03e', 'hex')
|
|
|
|
|
|
|
|
describe('Transaction Controller', function () {
|
|
|
|
let txController
|
|
|
|
|
|
|
|
beforeEach(function () {
|
|
|
|
txController = new TransactionController({
|
|
|
|
networkStore: new ObservableStore(currentNetworkId),
|
|
|
|
txHistoryLimit: 10,
|
|
|
|
provider: { _blockTracker: new EventEmitter()},
|
|
|
|
blockTracker: new EventEmitter(),
|
|
|
|
ethQuery: new EthQuery(new EventEmitter()),
|
|
|
|
signTransaction: (ethTx) => new Promise((resolve) => {
|
|
|
|
ethTx.sign(privKey)
|
|
|
|
resolve()
|
|
|
|
}),
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('#validateTxParams', function () {
|
|
|
|
it('returns null for positive values', function () {
|
|
|
|
var sample = {
|
|
|
|
value: '0x01',
|
|
|
|
}
|
|
|
|
txController.txProviderUtils.validateTxParams(sample, (err) => {
|
|
|
|
assert.equal(err, null, 'no error')
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
it('returns error for negative values', function () {
|
|
|
|
var sample = {
|
|
|
|
value: '-0x01',
|
|
|
|
}
|
|
|
|
txController.txProviderUtils.validateTxParams(sample, (err) => {
|
|
|
|
assert.ok(err, 'error')
|
|
|
|
})
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('#getTxList', function () {
|
|
|
|
it('when new should return empty array', function () {
|
|
|
|
var result = txController.getTxList()
|
|
|
|
assert.ok(Array.isArray(result))
|
|
|
|
assert.equal(result.length, 0)
|
|
|
|
})
|
|
|
|
it('should also return transactions from local storage if any', function () {
|
|
|
|
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('#addTx', function () {
|
|
|
|
it('adds a tx returned in getTxList', function () {
|
|
|
|
var tx = { id: 1, status: 'confirmed', metamaskNetworkId: currentNetworkId, txParams: {} }
|
|
|
|
txController.addTx(tx, noop)
|
|
|
|
var result = txController.getTxList()
|
|
|
|
assert.ok(Array.isArray(result))
|
|
|
|
assert.equal(result.length, 1)
|
|
|
|
assert.equal(result[0].id, 1)
|
|
|
|
})
|
|
|
|
|
|
|
|
it('does not override txs from other networks', function () {
|
|
|
|
var tx = { id: 1, status: 'confirmed', metamaskNetworkId: currentNetworkId, txParams: {} }
|
|
|
|
var tx2 = { id: 2, status: 'confirmed', metamaskNetworkId: otherNetworkId, txParams: {} }
|
|
|
|
txController.addTx(tx, noop)
|
|
|
|
txController.addTx(tx2, noop)
|
|
|
|
var result = txController.getFullTxList()
|
|
|
|
var result2 = txController.getTxList()
|
|
|
|
assert.equal(result.length, 2, 'txs were deleted')
|
|
|
|
assert.equal(result2.length, 1, 'incorrect number of txs on network.')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('cuts off early txs beyond a limit', function () {
|
|
|
|
const limit = txController.txHistoryLimit
|
|
|
|
for (let i = 0; i < limit + 1; i++) {
|
|
|
|
const tx = { id: i, time: new Date(), status: 'confirmed', metamaskNetworkId: currentNetworkId, txParams: {} }
|
|
|
|
txController.addTx(tx, noop)
|
|
|
|
}
|
|
|
|
var result = txController.getTxList()
|
|
|
|
assert.equal(result.length, limit, `limit of ${limit} txs enforced`)
|
|
|
|
assert.equal(result[0].id, 1, 'early txs truncted')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('cuts off early txs beyond a limit whether or not it is confirmed or rejected', function () {
|
|
|
|
const limit = txController.txHistoryLimit
|
|
|
|
for (let i = 0; i < limit + 1; i++) {
|
|
|
|
const tx = { id: i, time: new Date(), status: 'rejected', metamaskNetworkId: currentNetworkId, txParams: {} }
|
|
|
|
txController.addTx(tx, noop)
|
|
|
|
}
|
|
|
|
var result = txController.getTxList()
|
|
|
|
assert.equal(result.length, limit, `limit of ${limit} txs enforced`)
|
|
|
|
assert.equal(result[0].id, 1, 'early txs truncted')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('cuts off early txs beyond a limit but does not cut unapproved txs', function () {
|
|
|
|
var unconfirmedTx = { id: 0, time: new Date(), status: 'unapproved', metamaskNetworkId: currentNetworkId, txParams: {} }
|
|
|
|
txController.addTx(unconfirmedTx, noop)
|
|
|
|
const limit = txController.txHistoryLimit
|
|
|
|
for (let i = 1; i < limit + 1; i++) {
|
|
|
|
const tx = { id: i, time: new Date(), status: 'confirmed', metamaskNetworkId: currentNetworkId, txParams: {} }
|
|
|
|
txController.addTx(tx, noop)
|
|
|
|
}
|
|
|
|
var result = txController.getTxList()
|
|
|
|
assert.equal(result.length, limit, `limit of ${limit} txs enforced`)
|
|
|
|
assert.equal(result[0].id, 0, 'first tx should still be there')
|
|
|
|
assert.equal(result[0].status, 'unapproved', 'first tx should be unapproved')
|
|
|
|
assert.equal(result[1].id, 2, 'early txs truncted')
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('#setTxStatusSigned', function () {
|
|
|
|
it('sets the tx status to signed', function () {
|
|
|
|
var tx = { id: 1, status: 'unapproved', metamaskNetworkId: currentNetworkId, txParams: {} }
|
|
|
|
txController.addTx(tx, noop)
|
|
|
|
txController.setTxStatusSigned(1)
|
|
|
|
var result = txController.getTxList()
|
|
|
|
assert.ok(Array.isArray(result))
|
|
|
|
assert.equal(result.length, 1)
|
|
|
|
assert.equal(result[0].status, 'signed')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('should emit a signed event to signal the exciton of callback', (done) => {
|
|
|
|
this.timeout(10000)
|
|
|
|
var tx = { id: 1, status: 'unapproved', metamaskNetworkId: currentNetworkId, txParams: {} }
|
|
|
|
const noop = function () {
|
|
|
|
assert(true, 'event listener has been triggered and noop executed')
|
|
|
|
done()
|
|
|
|
}
|
|
|
|
txController.addTx(tx)
|
|
|
|
txController.on('1:signed', noop)
|
|
|
|
txController.setTxStatusSigned(1)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('#setTxStatusRejected', function () {
|
|
|
|
it('sets the tx status to rejected', function () {
|
|
|
|
var tx = { id: 1, status: 'unapproved', metamaskNetworkId: currentNetworkId, txParams: {} }
|
|
|
|
txController.addTx(tx)
|
|
|
|
txController.setTxStatusRejected(1)
|
|
|
|
var result = txController.getTxList()
|
|
|
|
assert.ok(Array.isArray(result))
|
|
|
|
assert.equal(result.length, 1)
|
|
|
|
assert.equal(result[0].status, 'rejected')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('should emit a rejected event to signal the exciton of callback', (done) => {
|
|
|
|
this.timeout(10000)
|
|
|
|
var tx = { id: 1, status: 'unapproved', metamaskNetworkId: currentNetworkId, txParams: {} }
|
|
|
|
txController.addTx(tx)
|
|
|
|
const noop = function (err, txId) {
|
|
|
|
assert(true, 'event listener has been triggered and noop executed')
|
|
|
|
done()
|
|
|
|
}
|
|
|
|
txController.on('1:rejected', noop)
|
|
|
|
txController.setTxStatusRejected(1)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('#updateTx', function () {
|
|
|
|
it('replaces the tx with the same id', function () {
|
|
|
|
txController.addTx({ id: '1', status: 'unapproved', metamaskNetworkId: currentNetworkId, txParams: {} }, noop)
|
|
|
|
txController.addTx({ id: '2', status: 'confirmed', metamaskNetworkId: currentNetworkId, txParams: {} }, noop)
|
|
|
|
txController.updateTx({ id: '1', status: 'blah', hash: 'foo', metamaskNetworkId: currentNetworkId, txParams: {} })
|
|
|
|
var result = txController.getTx('1')
|
|
|
|
assert.equal(result.hash, 'foo')
|
|
|
|
})
|
|
|
|
|
|
|
|
it('updates gas price', function () {
|
|
|
|
const originalGasPrice = '0x01'
|
|
|
|
const desiredGasPrice = '0x02'
|
|
|
|
|
|
|
|
const txMeta = {
|
|
|
|
id: '1',
|
|
|
|
status: 'unapproved',
|
|
|
|
metamaskNetworkId: currentNetworkId,
|
|
|
|
txParams: {
|
|
|
|
gasPrice: originalGasPrice,
|
|
|
|
},
|
|
|
|
}
|
|
|
|
|
|
|
|
const updatedMeta = clone(txMeta)
|
|
|
|
|
|
|
|
txController.addTx(txMeta)
|
|
|
|
updatedMeta.txParams.gasPrice = desiredGasPrice
|
|
|
|
txController.updateTx(updatedMeta)
|
|
|
|
var result = txController.getTx('1')
|
|
|
|
assert.equal(result.txParams.gasPrice, desiredGasPrice, 'gas price updated')
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('#getUnapprovedTxList', function () {
|
|
|
|
it('returns unapproved txs in a hash', function () {
|
|
|
|
txController.addTx({ id: '1', status: 'unapproved', metamaskNetworkId: currentNetworkId, txParams: {} }, noop)
|
|
|
|
txController.addTx({ id: '2', status: 'confirmed', metamaskNetworkId: currentNetworkId, txParams: {} }, noop)
|
|
|
|
const result = txController.getUnapprovedTxList()
|
|
|
|
assert.equal(typeof result, 'object')
|
|
|
|
assert.equal(result['1'].status, 'unapproved')
|
|
|
|
assert.equal(result['2'], undefined)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('#getTx', function () {
|
|
|
|
it('returns a tx with the requested id', function () {
|
|
|
|
txController.addTx({ id: '1', status: 'unapproved', metamaskNetworkId: currentNetworkId, txParams: {} }, noop)
|
|
|
|
txController.addTx({ id: '2', status: 'confirmed', metamaskNetworkId: currentNetworkId, txParams: {} }, noop)
|
|
|
|
assert.equal(txController.getTx('1').status, 'unapproved')
|
|
|
|
assert.equal(txController.getTx('2').status, 'confirmed')
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('#getFilteredTxList', function () {
|
|
|
|
it('returns a tx with the requested data', function () {
|
|
|
|
const txMetas = [
|
|
|
|
{ id: 0, status: 'unapproved', txParams: { from: '0xaa', to: '0xbb' }, metamaskNetworkId: currentNetworkId },
|
|
|
|
{ id: 1, status: 'unapproved', txParams: { from: '0xaa', to: '0xbb' }, metamaskNetworkId: currentNetworkId },
|
|
|
|
{ id: 2, status: 'unapproved', txParams: { from: '0xaa', to: '0xbb' }, metamaskNetworkId: currentNetworkId },
|
|
|
|
{ id: 3, status: 'unapproved', txParams: { from: '0xbb', to: '0xaa' }, metamaskNetworkId: currentNetworkId },
|
|
|
|
{ id: 4, status: 'unapproved', txParams: { from: '0xbb', to: '0xaa' }, metamaskNetworkId: currentNetworkId },
|
|
|
|
{ id: 5, status: 'confirmed', txParams: { from: '0xaa', to: '0xbb' }, metamaskNetworkId: currentNetworkId },
|
|
|
|
{ id: 6, status: 'confirmed', txParams: { from: '0xaa', to: '0xbb' }, metamaskNetworkId: currentNetworkId },
|
|
|
|
{ id: 7, status: 'confirmed', txParams: { from: '0xbb', to: '0xaa' }, metamaskNetworkId: currentNetworkId },
|
|
|
|
{ id: 8, status: 'confirmed', txParams: { from: '0xbb', to: '0xaa' }, metamaskNetworkId: currentNetworkId },
|
|
|
|
{ id: 9, status: 'confirmed', txParams: { from: '0xbb', to: '0xaa' }, metamaskNetworkId: currentNetworkId },
|
|
|
|
]
|
|
|
|
txMetas.forEach((txMeta) => txController.addTx(txMeta, noop))
|
|
|
|
let filterParams
|
|
|
|
|
|
|
|
filterParams = { status: 'unapproved', from: '0xaa' }
|
|
|
|
assert.equal(txController.getFilteredTxList(filterParams).length, 3, `getFilteredTxList - ${JSON.stringify(filterParams)}`)
|
|
|
|
filterParams = { status: 'unapproved', to: '0xaa' }
|
|
|
|
assert.equal(txController.getFilteredTxList(filterParams).length, 2, `getFilteredTxList - ${JSON.stringify(filterParams)}`)
|
|
|
|
filterParams = { status: 'confirmed', from: '0xbb' }
|
|
|
|
assert.equal(txController.getFilteredTxList(filterParams).length, 3, `getFilteredTxList - ${JSON.stringify(filterParams)}`)
|
|
|
|
filterParams = { status: 'confirmed' }
|
|
|
|
assert.equal(txController.getFilteredTxList(filterParams).length, 5, `getFilteredTxList - ${JSON.stringify(filterParams)}`)
|
|
|
|
filterParams = { from: '0xaa' }
|
|
|
|
assert.equal(txController.getFilteredTxList(filterParams).length, 5, `getFilteredTxList - ${JSON.stringify(filterParams)}`)
|
|
|
|
filterParams = { to: '0xaa' }
|
|
|
|
assert.equal(txController.getFilteredTxList(filterParams).length, 5, `getFilteredTxList - ${JSON.stringify(filterParams)}`)
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('#approveTransaction', function () {
|
|
|
|
let txMeta, originalValue
|
|
|
|
|
|
|
|
beforeEach(function () {
|
|
|
|
originalValue = '0x01'
|
|
|
|
txMeta = {
|
|
|
|
id: '1',
|
|
|
|
status: 'unapproved',
|
|
|
|
metamaskNetworkId: currentNetworkId,
|
|
|
|
txParams: {
|
|
|
|
nonce: originalValue,
|
|
|
|
gas: originalValue,
|
|
|
|
gasPrice: originalValue,
|
|
|
|
},
|
|
|
|
}
|
|
|
|
})
|
|
|
|
|
|
|
|
|
|
|
|
it('does not overwrite set values', function (done) {
|
|
|
|
const wrongValue = '0x05'
|
|
|
|
|
|
|
|
txController.addTx(txMeta)
|
|
|
|
|
|
|
|
const estimateStub = sinon.stub(txController.txProviderUtils.query, 'estimateGas')
|
|
|
|
.callsArgWith(1, null, wrongValue)
|
|
|
|
|
|
|
|
const priceStub = sinon.stub(txController.txProviderUtils.query, 'gasPrice')
|
|
|
|
.callsArgWith(0, null, wrongValue)
|
|
|
|
|
|
|
|
const nonceStub = sinon.stub(txController.txProviderUtils.query, 'getTransactionCount')
|
|
|
|
.callsArgWith(2, null, wrongValue)
|
|
|
|
|
|
|
|
const signStub = sinon.stub(txController, 'signTransaction')
|
|
|
|
.callsArgWith(1, null, noop)
|
|
|
|
|
|
|
|
const pubStub = sinon.stub(txController.txProviderUtils, 'publishTransaction')
|
|
|
|
.callsArgWith(1, null, originalValue)
|
|
|
|
|
|
|
|
txController.approveTransaction(txMeta.id, (err) => {
|
|
|
|
assert.ifError(err, 'should not error')
|
|
|
|
|
|
|
|
const result = txController.getTx(txMeta.id)
|
|
|
|
const params = result.txParams
|
|
|
|
|
|
|
|
assert.equal(params.gas, originalValue, 'gas unmodified')
|
|
|
|
assert.equal(params.gasPrice, originalValue, 'gas price unmodified')
|
|
|
|
assert.equal(params.nonce, originalValue, 'nonce unmodified')
|
|
|
|
assert.equal(result.hash, originalValue, 'hash was set')
|
|
|
|
|
|
|
|
estimateStub.restore()
|
|
|
|
priceStub.restore()
|
|
|
|
signStub.restore()
|
|
|
|
nonceStub.restore()
|
|
|
|
pubStub.restore()
|
|
|
|
|
|
|
|
done()
|
|
|
|
})
|
|
|
|
})
|
|
|
|
})
|
|
|
|
|
|
|
|
describe('#sign replay-protected tx', function () {
|
|
|
|
it('prepares a tx with the chainId set', function (done) {
|
|
|
|
txController.addTx({ id: '1', status: 'unapproved', metamaskNetworkId: currentNetworkId, txParams: {} }, noop)
|
|
|
|
txController.signTransaction('1', (err, rawTx) => {
|
|
|
|
if (err) return done('it should not fail')
|
|
|
|
const ethTx = new EthTx(ethUtil.toBuffer(rawTx))
|
|
|
|
assert.equal(ethTx.getChainId(), currentNetworkId)
|
|
|
|
done()
|
|
|
|
})
|
|
|
|
})
|
|
|
|
})
|
|
|
|
})
|