Skip to content

Commit

Permalink
Merge ab09897 into 73ef7a9
Browse files Browse the repository at this point in the history
  • Loading branch information
@nivida
nivida committed Nov 22, 2019
2 parents 73ef7a9 + ab09897 commit f3e07c2
Show file tree
Hide file tree
Showing 23 changed files with 106 additions and 100 deletions.
30 changes: 17 additions & 13 deletions packages/web3-core-helpers/src/formatters.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,8 @@ var inputStorageKeysFormatter = function (keys) {
*/
var outputProofFormatter = function (proof) {
proof.address = utils.toChecksumAddress(proof.address);
proof.nonce = utils.hexToNumberString(proof.nonce);
proof.balance = utils.hexToNumberString(proof.balance);
proof.nonce = outputBigNumberFormatter(proof.nonce);
proof.balance = outputBigNumberFormatter(proof.balance);

return proof;
};
Expand All @@ -68,6 +68,10 @@ var outputProofFormatter = function (proof) {
* @returns {BN} object
*/
var outputBigNumberFormatter = function (number) {
if (!number) {
return number;
}

return utils.toBN(number).toString(10);
};

Expand Down Expand Up @@ -225,13 +229,13 @@ var inputSignFormatter = function (data) {
*/
var outputTransactionFormatter = function (tx) {
if (tx.blockNumber !== null)
tx.blockNumber = utils.hexToNumber(tx.blockNumber);
tx.blockNumber = outputBigNumberFormatter(tx.blockNumber);
if (tx.transactionIndex !== null)
tx.transactionIndex = utils.hexToNumber(tx.transactionIndex);
tx.nonce = utils.hexToNumber(tx.nonce);
tx.gas = utils.hexToNumber(tx.gas);
tx.gasPrice = outputBigNumberFormatter(tx.gasPrice);
tx.nonce = outputBigNumberFormatter(tx.nonce);
tx.value = outputBigNumberFormatter(tx.value);
tx.gasPrice = outputBigNumberFormatter(tx.gasPrice);
tx.gas = utils.hexToNumber(tx.gas);

if (tx.to && utils.isAddress(tx.to)) { // tx.to could be `0x0` or `null` while contract creation
tx.to = utils.toChecksumAddress(tx.to);
Expand Down Expand Up @@ -259,7 +263,7 @@ var outputTransactionReceiptFormatter = function (receipt) {
}

if (receipt.blockNumber !== null)
receipt.blockNumber = utils.hexToNumber(receipt.blockNumber);
receipt.blockNumber = outputBigNumberFormatter(receipt.blockNumber);
if (receipt.transactionIndex !== null)
receipt.transactionIndex = utils.hexToNumber(receipt.transactionIndex);
receipt.cumulativeGasUsed = utils.hexToNumber(receipt.cumulativeGasUsed);
Expand Down Expand Up @@ -293,9 +297,9 @@ var outputBlockFormatter = function (block) {
block.gasLimit = utils.hexToNumber(block.gasLimit);
block.gasUsed = utils.hexToNumber(block.gasUsed);
block.size = utils.hexToNumber(block.size);
block.timestamp = utils.hexToNumber(block.timestamp);
block.timestamp = outputBigNumberFormatter(block.timestamp);
if (block.number !== null)
block.number = utils.hexToNumber(block.number);
block.number = outputBigNumberFormatter(block.number);

if (block.difficulty)
block.difficulty = outputBigNumberFormatter(block.difficulty);
Expand Down Expand Up @@ -380,7 +384,7 @@ var outputLogFormatter = function (log) {
}

if (log.blockNumber !== null)
log.blockNumber = utils.hexToNumber(log.blockNumber);
log.blockNumber = outputBigNumberFormatter(log.blockNumber);
if (log.transactionIndex !== null)
log.transactionIndex = utils.hexToNumber(log.transactionIndex);
if (log.logIndex !== null)
Expand Down Expand Up @@ -469,9 +473,9 @@ var inputAddressFormatter = function (address) {

var outputSyncingFormatter = function (result) {

result.startingBlock = utils.hexToNumber(result.startingBlock);
result.currentBlock = utils.hexToNumber(result.currentBlock);
result.highestBlock = utils.hexToNumber(result.highestBlock);
result.startingBlock = outputBigNumberFormatter(result.startingBlock);
result.currentBlock = outputBigNumberFormatter(result.currentBlock);
result.highestBlock = outputBigNumberFormatter(result.highestBlock);
if (result.knownStates) {
result.knownStates = utils.hexToNumber(result.knownStates);
result.pulledStates = utils.hexToNumber(result.pulledStates);
Expand Down
2 changes: 1 addition & 1 deletion packages/web3-core-method/src/index.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -299,7 +299,7 @@ Method.prototype._confirmTransaction = function(defer, result, payload) {
if (existingReceipt === undefined || confirmationCount !== 0) {
if (isPolling) { // Check if actually a new block is existing on polling
if (lastBlock) {
block = await _ethereumCall.getBlockByNumber(lastBlock.number + 1);
block = await _ethereumCall.getBlockByNumber(parseInt(lastBlock.number) + 1);
if (block) {
lastBlock = block;
defer.eventEmitter.emit('confirmation', confirmationCount, receipt);
Expand Down
2 changes: 1 addition & 1 deletion packages/web3-eth-accounts/src/index.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ Accounts.prototype.signTransaction = function signTransaction(tx, privateKey, ca
error = new Error('"gas" is missing');
}

if (tx.nonce < 0 ||
if (utils.toBN(tx.nonce).ltn(0) ||
tx.gas < 0 ||
tx.gasPrice < 0 ||
tx.chainId < 0) {
Expand Down
6 changes: 4 additions & 2 deletions packages/web3-eth-ens/src/ENS.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
var config = require('./config');
var Registry = require('./contracts/Registry');
var ResolverMethodHandler = require('./lib/ResolverMethodHandler');
var utils = require('web3-utils');

/**
* Constructs a new instance of ENS
Expand Down Expand Up @@ -171,8 +172,9 @@ ENS.prototype.setMultihash = function (name, hash, sendOptions, callback) {
ENS.prototype.checkNetwork = function () {
var self = this;
return self.eth.getBlock('latest').then(function (block) {
var headAge = new Date() / 1000 - block.timestamp;
if (headAge > 3600) {
// TODO: Check with Quorum timestamp
var headAge = utils.toBN((Math.floor(new Date() / 1000))).sub(utils.toBN(block.timestamp));
if (headAge.gtn(3600)) {
throw new Error("Network not synced; last block was " + headAge + " seconds ago");
}
return self.eth.net.getNetworkType();
Expand Down
4 changes: 2 additions & 2 deletions packages/web3-eth/src/index.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -341,7 +341,7 @@ var Eth = function Eth() {
name: 'getBlockNumber',
call: 'eth_blockNumber',
params: 0,
outputFormatter: utils.hexToNumber
outputFormatter: formatter.outputBigNumberFormatter
}),
new Method({
name: 'getBalance',
Expand Down Expand Up @@ -417,7 +417,7 @@ var Eth = function Eth() {
call: 'eth_getTransactionCount',
params: 2,
inputFormatter: [formatter.inputAddressFormatter, formatter.inputDefaultBlockNumberFormatter],
outputFormatter: utils.hexToNumber
outputFormatter: formatter.outputBigNumberFormatter
}),
new Method({
name: 'sendSignedTransaction',
Expand Down
34 changes: 17 additions & 17 deletions test/contract.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -461,7 +461,7 @@ var runTests = function(contractFactory) {
'0000000000000000000000000000000000000000000000000000000000000008'
});

assert.equal(result.blockNumber, 3);
assert.equal(result.blockNumber, '3');
assert.equal(result.blockHash, '0x1345');
assert.equal(result.logIndex, 4);
assert.equal(result.id, 'log_9ff24cb4');
Expand Down Expand Up @@ -567,7 +567,7 @@ var runTests = function(contractFactory) {
contractAddress: address,
cumulativeGasUsed: 10,
transactionIndex: 3,
blockNumber: 10,
blockNumber: '10',
blockHash: '0xbf1234',
gasUsed: 0
});
Expand Down Expand Up @@ -661,7 +661,7 @@ var runTests = function(contractFactory) {
contractAddress: address,
cumulativeGasUsed: 10,
transactionIndex: 3,
blockNumber: 10,
blockNumber: '10',
blockHash: '0xbf1234',
gasUsed: 0
});
Expand Down Expand Up @@ -1799,13 +1799,13 @@ var runTests = function(contractFactory) {
cumulativeGasUsed: 10,
transactionIndex: 3,
transactionHash: '0x1234',
blockNumber: 10,
blockNumber: '10',
blockHash: '0x1234',
gasUsed: 0,
events: {
Unchanged: {
address: address,
blockNumber: 10,
blockNumber: '10',
transactionHash: '0x1234',
blockHash: '0x1345',
logIndex: 4,
Expand All @@ -1830,7 +1830,7 @@ var runTests = function(contractFactory) {
},
Changed: {
address: '0x11f4d0A3c12e86B4b5F39B213F7E19D048276DAe',
blockNumber: 10,
blockNumber: '10',
transactionHash: '0x1234',
blockHash: '0x1345',
logIndex: 4,
Expand Down Expand Up @@ -1962,14 +1962,14 @@ var runTests = function(contractFactory) {
cumulativeGasUsed: 10,
transactionIndex: 3,
transactionHash: '0x1234',
blockNumber: 10,
blockNumber: '10',
blockHash: '0x1234',
gasUsed: 0,
events: {
Changed: [
{
address: '0x11f4d0A3c12e86B4b5F39B213F7E19D048276DAe',
blockNumber: 10,
blockNumber: '10',
transactionHash: '0x1234',
blockHash: '0x1345',
logIndex: 4,
Expand All @@ -1995,7 +1995,7 @@ var runTests = function(contractFactory) {
}
}, {
address: '0x11f4d0A3c12e86B4b5F39B213F7E19D048276DAe',
blockNumber: 10,
blockNumber: '10',
transactionHash: '0x1234',
blockHash: '0x1345',
logIndex: 5,
Expand Down Expand Up @@ -2114,13 +2114,13 @@ var runTests = function(contractFactory) {
cumulativeGasUsed: 10,
transactionIndex: 3,
transactionHash: '0x1234',
blockNumber: 10,
blockNumber: '10',
blockHash: '0x43ffdd',
gasUsed: 0,
events: {
Unchanged: {
address: address,
blockNumber: 10,
blockNumber: '10',
transactionHash: '0x1234',
blockHash: '0x1345',
logIndex: 4,
Expand All @@ -2145,7 +2145,7 @@ var runTests = function(contractFactory) {
},
Changed: {
address: '0x11f4d0A3c12e86B4b5F39B213F7E19D048276DAe',
blockNumber: 10,
blockNumber: '10',
transactionHash: '0x1234',
blockHash: '0x1345',
logIndex: 4,
Expand Down Expand Up @@ -2229,7 +2229,7 @@ var runTests = function(contractFactory) {
cumulativeGasUsed: 10,
transactionIndex: 3,
transactionHash: '0x1234',
blockNumber: 10,
blockNumber: '10',
blockHash: '0x1234',
gasUsed: 0,
events: {}
Expand All @@ -2243,7 +2243,7 @@ var runTests = function(contractFactory) {
cumulativeGasUsed: 10,
transactionIndex: 3,
transactionHash: '0x1234',
blockNumber: 10,
blockNumber: '10',
blockHash: '0x1234',
gasUsed: 0,
events: {}
Expand Down Expand Up @@ -2693,7 +2693,7 @@ var runTests = function(contractFactory) {
signature: "0xc00c1c37cc8b83163fb4fddc06c74d1d5c00d74648e7cb28c0ebada3e32fd62c",
id: "log_9ff24cb4",
address: address,
blockNumber: 3,
blockNumber: '3',
transactionHash: '0x1234',
blockHash: '0x1345',
logIndex: 4,
Expand All @@ -2719,7 +2719,7 @@ var runTests = function(contractFactory) {
signature: "0xc00c1c37cc8b83163fb4fddc06c74d1d5c00d74648e7cb28c0ebada3e32fd62c",
id: "log_29c93e15",
address: address,
blockNumber: 4,
blockNumber: '4',
transactionHash: '0x1235',
blockHash: '0x1346',
logIndex: 1,
Expand Down Expand Up @@ -3091,7 +3091,7 @@ describe('typical usage', function() {
gas: 54872,
gasPrice: 3000,
chainId: 1,
nonce: 1,
nonce: '1',
chain: 'mainnet',
hardfork: 'petersburg'
})
Expand Down
2 changes: 1 addition & 1 deletion test/e2e.contract.deploy.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ describe('contract.deploy [ @E2E ]', function() {
.on('confirmation', async (number, receipt) => {
if (number === 1) { // Confirmation numbers are zero indexed
var endBlock = await web3.eth.getBlockNumber();
assert(endBlock >= (startBlock + 2));
assert(parseInt(endBlock) >= (parseInt(startBlock) + 2));
resolve();
}
})
Expand Down
2 changes: 1 addition & 1 deletion test/e2e.method.send.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ describe('method.send [ @E2E ]', function() {
.on('confirmation', async (number, receipt) => {
if (number === 1) { // Confirmation numbers are zero indexed
var endBlock = await web3.eth.getBlockNumber();
assert(endBlock >= (startBlock + 2));
assert(parseInt(endBlock) >= (parseInt(startBlock) + 2));
resolve();
}
})
Expand Down
12 changes: 6 additions & 6 deletions test/eth.getBlock.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ var blockResult = {
"uncles": ["0x460cfb8472af2c5fd05b5a2", "0xd5460cfb8472af2c5fd05b5a2"]
};
var formattedBlockResult = {
"number": 436,
"number": '436',
"hash": "0xe670ec64341771606e55d6b4ca35a1a6b75ee3d5145a99d05921026d1527331",
"parentHash": "0x9646252be9520f6e71339a8df9c55e4d7619deeb018d2a3f2d21fc165dde5eb5",
"nonce": "0xe04d296d2460cfb8472af2c5fd05b5a214109c25688d3704aed5484f9a7792f2",
Expand All @@ -38,7 +38,7 @@ var formattedBlockResult = {
"extraData": "0x0000000000000000000000000000000000000000000000000000000000000000",
"gasLimit": 653145,
"gasUsed": 653145,
"timestamp": 1424182926,
"timestamp": '1424182926',
"transactions": ['0x460cfb8472af2c5fd05b5a2','0x460cfb8472af2c5fd05b5a2'],
"uncles": ["0x460cfb8472af2c5fd05b5a2", "0xd5460cfb8472af2c5fd05b5a2"]
};
Expand Down Expand Up @@ -76,7 +76,7 @@ var blockResultWithTx = {
"uncles": ["0x460cfb8472af2c5fd05b5a2", "0xd5460cfb8472af2c5fd05b5a2"]
};
var formattedBlockResultWithTx = {
"number": 436,
"number": '436',
"hash": "0xe670ec64341771606e55d6b4ca35a1a6b75ee3d5145a99d05921026d1527331",
"parentHash": "0x9646252be9520f6e71339a8df9c55e4d7619deeb018d2a3f2d21fc165dde5eb5",
"nonce": "0xe04d296d2460cfb8472af2c5fd05b5a214109c25688d3704aed5484f9a7792f2",
Expand All @@ -91,13 +91,13 @@ var formattedBlockResultWithTx = {
"extraData": "0x0000000000000000000000000000000000000000000000000000000000000000",
"gasLimit": 653145,
"gasUsed": 653145,
"timestamp": 1424182926,
"timestamp": '1424182926',
"transactions": [{
"status": "mined",
"hash":"0xc6ef2fc5426d6ad6fd9e2a26abeab0aa2411b7ab17f30a99d3cb96aed1d1055b",
"nonce": 2,
"nonce": '2',
"blockHash": "0x6fd9e2a26ab",
"blockNumber": 5599,
"blockNumber": '5599',
"transactionIndex": 1,
"from":"0x407D73d8a49eeb85D32Cf465507dd71d507100c1",
"to":"0x85F43D8a49eeB85d32Cf465507DD71d507100C1d",
Expand Down
2 changes: 1 addition & 1 deletion test/eth.getBlockNumber.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ var method = 'getBlockNumber';

var tests = [{
result: '0xb',
formattedResult: 11,
formattedResult: '11',
call: 'eth_blockNumber'
}];

Expand Down
6 changes: 3 additions & 3 deletions test/eth.getTransaction.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,10 @@ var txResult = {
"input":"0x603880600c6000396000f30060"
};
var formattedTxResult = {
"hash":"0xc6ef2fc5426d6ad6fd9e2a26abeab0aa2411b7ab17f30a99d3cb96aed1d1055b",
"nonce":5,
"hash": "0xc6ef2fc5426d6ad6fd9e2a26abeab0aa2411b7ab17f30a99d3cb96aed1d1055b",
"nonce": '5',
"blockHash": "0x6fd9e2a26ab",
"blockNumber": 5599,
"blockNumber": '5599',
"transactionIndex": 1,
"from":"0x407D73d8a49eeb85D32Cf465507dd71d507100c1", // checksum address
"to":"0x85F43D8a49eeB85d32Cf465507DD71d507100C1d", // checksum address
Expand Down
6 changes: 3 additions & 3 deletions test/eth.getTransactionFromBlock.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,10 @@ var txResult = {
};
var formattedTxResult = {
"status": "mined",
"hash":"0xc6ef2fc5426d6ad6fd9e2a26abeab0aa2411b7ab17f30a99d3cb96aed1d1055b",
"nonce":11,
"hash": "0xc6ef2fc5426d6ad6fd9e2a26abeab0aa2411b7ab17f30a99d3cb96aed1d1055b",
"nonce": '11',
"blockHash": "0x6fd9e2a26ab",
"blockNumber": 5599,
"blockNumber": '5599',
"transactionIndex": 1,
"from":"0x407D73d8a49eeb85D32Cf465507dd71d507100c1", // checksum address
"to":"0x85F43D8a49eeB85d32Cf465507DD71d507100C1d", // checksum address
Expand Down
Loading

0 comments on commit f3e07c2

Please sign in to comment.