resolve and un-skip issue: web3.js#6327

packages/codec/lib/evm/utils.ts

@@ -22,9 +22,7 @@ export const ZERO_ADDRESS = "0x" + "00".repeat(ADDRESS_SIZE);
 export function keccak256(...args: any[]): BN {
   // debug("args %o", args);
 
-  // TODO: Remove the next line once https://github.com/web3/web3.js/issues/6327 is fixed
-  // this hacky workaround does not work for large numbers (above 53 bit)
-  args = args.map(v => (BN.isBN(v) ? v.toNumber() : v));
+  args = args.map(v => (BN.isBN(v) ? BigInt(v.toString()) : v));
   const rawSha: string | null = Web3Utils.soliditySha3(...args);
   debug("rawSha %o", rawSha);
   let sha: string;

packages/debugger/test/data/helpers.js

@@ -30,9 +30,7 @@ function fileName(testName) {
 
 function generateTests(fixtures) {
   for (let { name, value: expected } of fixtures) {
-    // TODO: un-skip once the following issue have been resolved:
-    // https://github.com/web3/web3.js/issues/6327
-    it.skip(`correctly decodes ${name}`, async () => {
+    it(`correctly decodes ${name}`, async () => {
       const response = await this.decode(name);
       assert.deepEqual(response, expected);
     });

packages/debugger/test/data/more-decoding.js

@@ -527,9 +527,7 @@ describe("Further Decoding", function () {
     assert.deepInclude(variables, expectedResult);
   });
 
-  // TODO: un-skip once the following issue have been resolved:
-  // https://github.com/web3/web3.js/issues/6327
-  it.skip("Decodes inner mappings correctly and keeps path info", async function () {
+  it("Decodes inner mappings correctly and keeps path info", async function () {
     this.timeout(12000);
 
     let instance = await abstractions.ComplexMappingTest.deployed();

packages/debugger/test/stacktrace.js

@@ -158,9 +158,6 @@ const migrations = {
   "2_deploy_contracts.js": __MIGRATION
 };
 
-// TODO: un-skip once the following issue have been resolved:
-//  https://github.com/web3/web3.js/issues/6327 (soliditySha3 behave differently on 4.x when one of the values is BN)
-// To test run: cd packages/debugger && yarn test test/stacktrace.js
 describe("Stack tracing", function () {
   let provider;
   let abstractions;

packages/decoder/test/current/test/decoding-test.js

@@ -36,18 +36,7 @@ describe("State variable decoding", function () {
     abstractions = prepared.abstractions;
   });
 
-  //todo debug test
-  //should get the initial state properly:
-  //
-  //       AssertionError [ERR_ASSERTION]: '0' == '4'
-  //       + expected - actual
-  //
-  //       -0
-  //       +4
-  // TODO: un-skip once the following has been resolved
-  //  https://github.com/web3/web3.js/issues/6327 (soliditySha3 behave differently on 4.x when one of the values is BN)
-
-  it.skip("should get the initial state properly", async function () {
+  it("should get the initial state properly", async function () {
     let deployedContract = await abstractions.DecodingSample.deployed();
     let address = deployedContract.address;
     const decoder = await Decoder.forContractInstance(deployedContract);