web3js 4.x RC upgrade #6013
web3js 4.x RC upgrade #6013
Conversation
|
Thanks @nikoulai . |
| assert(error.message.includes("intrinsic gas too low")); | ||
| // assert(error.innerError.message.includes("intrinsic gas too low")); |
| it("overrides 50 blocks err / return a usable instance", async function () { | ||
| //todo web3js-migration fix test | ||
| it.skip("overrides 50 blocks err / return a usable instance", async function () { |
| //todo web3js-migration fix test | ||
| it.skip( | ||
| "overrides gateway tx propagation delay err / return a usable instance", |
| it("validates method arguments for .sends", async function () { | ||
| //todo web3js-migration remove skip | ||
| it.skip("validates method arguments for .sends", async function () { |
| if (!handlers.ignoreTimeoutError(context, error)) { | ||
| context.promiEvent.eventEmitter.emit("error", error); | ||
| this.removeListener("error", handlers.error); | ||
| this.off("error", handlers.error); |
There was a problem hiding this comment.
Shouldn't removeListener work here? What was the issue?
| return web3Utils.toBN(val); | ||
| return new BN(val); |
There was a problem hiding this comment.
FYI to Truffle team, we no longer use bignumber.js or bn.js, just BigInts
| return web3Utils.isBN(val) || web3Utils.isBigNumber(val); | ||
| //todo web3js-migration check if this is sufficient | ||
| return BN.isBN(val); | ||
| // || web3Utils.isBigNumber(val); |
| // and include anything mentioned in receipt logs | ||
| for (const { address } of receipt.logs) { | ||
| addresses.push(address); | ||
| addresses.push(address as string); |
There was a problem hiding this comment.
What was the type of address? It should be a string by default
| const ZERO_NODE = | ||
| "0x0000000000000000000000000000000000000000000000000000000000000000"; |
| eventArgs.error = err.error || err; | ||
| const message = await self.emit("deployment:failed", eventArgs); | ||
| self.close(); | ||
| console.warn(message); |
| console.log(libReceipt.blockNumber, startBlock); | ||
| console.log( | ||
| exampleReceipt.blockNumber, | ||
| libReceipt.blockNumber, | ||
| startBlock | ||
| ); | ||
| // The first confirmation is the block that accepts the tx. Then we wait two more. | ||
| // Then Example is deployed in the consequent block. | ||
| assert(libReceipt.blockNumber === startBlock + 1); | ||
| assert(exampleReceipt.blockNumber === libReceipt.blockNumber + 3); | ||
| assert(libReceipt.blockNumber === startBlock + BigInt(1)); | ||
| //todo web3js-migration this fails | ||
| // assert(exampleReceipt.blockNumber === libReceipt.blockNumber + BigInt(3)); |
There was a problem hiding this comment.
@nikoulai Did you add all this? Where is it from, what's it for?
Same with the files:
packages/encoder/test/initialization.test.jspackages/encoder/test/integer.test.jspackages/encoder/test/udvt.test.js
And these lib/ files shouldn't be included?
packages/fetch-and-compile/lib/debug.jspackages/fetch-and-compile/lib/fetch.jspackages/fetch-and-compile/lib/index.jspackages/fetch-and-compile/lib/multiple.jspackages/fetch-and-compile/lib/recognizer.jspackages/fetch-and-compile/lib/types.jspackages/fetch-and-compile/lib/utils.js
|
Note: @nazarhussain was (is?) working this issue and there might be some learnings in his PR #5393. |
|
@cds-amal @nikoulai Was holding it off to get fixed the identified issues in web3-4x first, publish it into a release, and then continue with this upgrade process. So that you know, I identified the following list of issues during this upgrade process. This is the reason my PR taking long time to shape up. web3/web3.js#5287 |
5 participants
web3js 4.x