fix: update forge CI to include contract size checks (#402)

* fix: add contract size check * fix: ignore harness build size
Layr-Labs · Feb 16, 2024 · 8b4c713 · 8b4c713
1 parent b71ceb4
commit 8b4c713
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/.github/workflows/testinparallel.yml b/.github/workflows/testinparallel.yml
@@ -37,8 +37,11 @@ jobs:
       with:
         version: nightly
 
-    - name: Install forge dependencies
-      run: forge install
+    - name: Run Forge build
+      run: |
+        forge --version
+        forge build --sizes
+      id: build
 
     - name: Run forge test for the file
       run: forge test --match-path src/test/${{ matrix.file }} --no-match-contract FFI

diff --git a/src/test/harnesses/EigenPodHarness.sol b/src/test/harnesses/EigenPodHarness.sol
@@ -2,8 +2,9 @@
 pragma solidity =0.8.12;
 
 import "../../contracts/pods/EigenPod.sol";
+import "forge-std/Test.sol";
 
-contract EPInternalFunctions is EigenPod {
+contract EPInternalFunctions is EigenPod, Test {
 
     constructor(
         IETHPOSDeposit _ethPOS,