Fix forked tests

sprintertech · Aug 5, 2024 · 606da56 · 606da56
1 parent 48428e3
commit 606da56
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -41,7 +41,7 @@ jobs:
         run: |
           fuser -k 8545/tcp
           make start-forkedMainnet FORKED_TESTS_PROVIDER=${{ secrets.FORKED_TESTS_PROVIDER }}
-          npx truffle test testUnderForked/*
+          make test-forked
 
   coverage:
     needs: test

diff --git a/Makefile b/Makefile
@@ -21,6 +21,10 @@ start-forkedMainnet:
 	@echo " > \033[32mStarting forked environment... \033[0m "
 	ganache -f $(FORKED_TESTS_PROVIDER) & sleep 3
 
+test-forked:
+	@echo " > \033[32mTesting contracts... \033[0m "
+	truffle test --stacktrace testUnderForked/*
+
 start-geth:
 	@echo " > \033[32mStarting geth... \033[0m "
 	./scripts/geth/start_geth.sh

diff --git a/testUnderForked/admin.js b/testUnderForked/admin.js
@@ -31,7 +31,7 @@ contract("TwapFeeHandler - [admin]", async (accounts) => {
   const currentFeeHandlerAdmin = accounts[0];
 
   const assertOnlyAdmin = (method, ...params) => {
-    return TruffleAssert.reverts(
+    return Helpers.reverts(
       method(...params, {from: initialRelayers[1]}),
       "sender doesn't have admin role"
     );
@@ -198,7 +198,7 @@ contract("TwapFeeHandler - [admin]", async (accounts) => {
       )
     );
 
-    await TruffleAssert.passes(
+    await Helpers.passes(
       DynamicFeeHandlerInstance.renounceAdmin(
         expectedDynamicFeeHandlerAdmin
       )