removed all uses of ganache

Author: d10r

.github/workflows/call.deploy-dry-run.yml

@@ -6,9 +6,6 @@ on:
       network:
         required: true
         type: string
-      network-id:
-        required: true
-        type: string
       provider-url:
         required: true
         type: string
@@ -28,8 +25,11 @@ jobs:
           yarn install --frozen-lockfile
           yarn build-for-contracts-dev
 
-      - name: Start ganache
-        run: npx ganache --port 47545 --mnemonic --fork.url ${{ github.event.inputs.provider-url }} --network-id ${{ github.event.inputs.network-id }} --chain.chainId ${{ github.event.inputs.network-id }}
+      - name: Start hardhat node
+        run: |
+          cd ${{ env.ethereum-contracts-working-directory }}
+          npx hardhat node --port 47545 --fork ${{ github.event.inputs.provider-url }} &
+          sleep 5
 
       - name: Deploy framework
         run: |

packages/ethereum-contracts/README.md

@@ -355,7 +355,7 @@ Run the test suite for core contracts:
 yarn run-hardhat test testsuites/superfluid-core.js
 ```
 
-The `pretest` script starts a ganache instance with deterministic accounts in the background, the `posttest` script stops it.
+The `pretest` script starts a local dev chain with deterministic accounts in the background, the `posttest` script stops it.
 When running tests with `yarn test`, those get executed automatically (see [npm docs](https://docs.npmjs.com/cli/v7/using-npm/scripts#pre--post-scripts)).
 > NOTE: You don't need to run the `pretest` and `posttest` scripts when running hardhat tests, but you do when running tests with truffle.
 

packages/ethereum-contracts/ops-scripts/libs/getConfig.js

@@ -11,11 +11,6 @@ module.exports = function getConfig(chainId) {
             // this is a fake forwarder address, it is to test the deployment script
             trustedForwarders: ["0x3075b4dc7085C48A14A5A39BBa68F58B19545971"],
         },
-        5777: {
-            // for local testing (external ganache)
-            // this is a fake forwarder address, it is to test the deployment script
-            trustedForwarders: ["0x3075b4dc7085C48A14A5A39BBa68F58B19545971"],
-        },
         6777: {
             // for coverage testing
             // this is a fake forwarder address, it is to test the deployment script

packages/ethereum-contracts/package.json

@@ -22,7 +22,6 @@
         "async": "^3.2.6",
         "csv-writer": "^1.6.0",
         "ethers": "^5.7.2",
-        "ganache-time-traveler": "^1.0.16",
         "mochawesome": "^7.1.3",
         "readline": "^1.3.0",
         "solidity-coverage": "^0.8.17",

packages/ethereum-contracts/test/TestEnvironment.ts

@@ -32,7 +32,6 @@ import {
 
 const {web3tx, wad4human} = require("@decentral.ee/web3-helpers");
 const SuperfluidSDK = require("@superfluid-finance/js-sdk");
-const traveler = require("ganache-time-traveler");
 
 const deployFramework = require("../ops-scripts/deploy-framework");
 const deploySuperToken = require("../ops-scripts/deploy-super-token");
@@ -161,6 +160,22 @@ export default class TestEnvironment {
         console.debug("popEvmSnapshot", JSON.stringify(this._evmSnapshots));
     }
 
+    /**
+     * Advance time by specified seconds and mine a block (replaces ganache-time-traveler.advanceTimeAndBlock)
+     */
+    async _advanceTimeAndBlock(seconds: number) {
+        await network.provider.send("evm_increaseTime", [seconds]);
+        await network.provider.send("evm_mine", []);
+    }
+
+    /**
+     * Set the next block's timestamp and mine a block (replaces ganache-time-traveler.advanceBlockAndSetTime)
+     * Uses evm_mine with timestamp parameter, matching ganache-time-traveler behavior
+     */
+    async _advanceBlockAndSetTime(timestamp: number) {
+        await network.provider.send("evm_mine", [timestamp]);
+    }
+
     async useLastEvmSnapshot() {
         let oldEvmSnapshotId = "";
         const popped = this._evmSnapshots.pop();
@@ -171,10 +186,6 @@ export default class TestEnvironment {
             } = popped);
         }
         await this._revertToEvmSnapShot(oldEvmSnapshotId);
-        // move the time to now
-        await traveler.advanceBlockAndSetTime(
-            parseInt((Date.now() / 1000).toString())
-        );
         const newEvmSnapshotId = await this._takeEvmSnapshot();
         this._evmSnapshots.push({
             id: newEvmSnapshotId,
@@ -192,7 +203,7 @@ export default class TestEnvironment {
         const block1 = await ethers.provider.getBlock("latest");
         console.log("current block time", block1.timestamp);
         console.log(`time traveler going to the future +${jsNumTime}...`);
-        await traveler.advanceTimeAndBlock(jsNumTime);
+        await this._advanceTimeAndBlock(jsNumTime);
         const block2 = await ethers.provider.getBlock("latest");
         console.log("new block time", block2.timestamp);
     }

packages/ethereum-contracts/truffle-config.js

@@ -171,8 +171,8 @@ const E = (module.exports = {
     networks: {
         // Useful for testing. The `development` name is special - truffle uses it by default
         // if it's defined here and no other network is specified at the command line.
-        // You should run a client (like ganache-cli, geth or parity) in a separate terminal
-        // tab if you use this network and you must also set the `host`, `port` and `network_id`
+        // You should run a client in a separate terminal tab
+        // if you use this network and you must also set the `host`, `port` and `network_id`
         // options below to some value.
 
         //