fix: coverage

Author: ypatil12

.github/workflows/foundry.yml

@@ -193,7 +193,7 @@ jobs:
           FOUNDRY_PROFILE=coverage \
           FOUNDRY_CACHE=true \
           FOUNDRY_CACHE_PATH=cache \
-          forge coverage --report lcov --report summary --no-match-coverage "script|test" -j $(nproc) -vvv
+          forge coverage --report lcov --report summary --no-match-coverage "script|test" -j $(nproc)  --no-match-test test_noCoverage
           genhtml -q -o report ./lcov.info
 
       # Upload coverage report as artifact before potential failure

src/test/unit/AllocationManagerUnit.t.sol

@@ -4416,7 +4416,8 @@ contract AllocationManagerUnitTests_migrateSlashers is AllocationManagerUnitTest
         assertEq(reads.length, 3, "should have 3 reads");
     }
 
-    function test_noop_slasherAlreadySet() public {
+    /// @dev This test is ignored by coverage due to an extra vm.access call
+    function test_noCoverage_noop_slasherAlreadySet() public {
         // Register the operatorSet
         OperatorSet memory operatorSet = OperatorSet(defaultAVS, 1);
         CreateSetParams[] memory createSetParams = new CreateSetParams[](1);
@@ -4506,7 +4507,8 @@ contract AllocationManagerUnitTests_migrateSlashers is AllocationManagerUnitTest
         _assertNothingPending(defaultOperatorSet);
     }
 
-    function test_cannotMigrateMultipleTimes() public {
+    /// @dev This test is ignored by coverage due to an extra vm.access call
+    function test_noCoverage_cannotMigrateMultipleTimes() public {
         // Migrate the slasher
         allocationManager.migrateSlashers(defaultOperatorSet.toArray());
         assertEq(allocationManager.getSlasher(defaultOperatorSet), defaultAVS, "slasher should be the defaultAVS");