-
Notifications
You must be signed in to change notification settings - Fork 49
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Revert "ci: cache testing contracts (#681)"
This reverts commit d1157b4.
- Loading branch information
Showing
2 changed files
with
19 additions
and
87 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -78,24 +78,19 @@ jobs: | |
|
|
||
| - name: "Generate and prepare the contract artifacts" | ||
| run: "./shell/prepare-artifacts.sh" | ||
|
|
||
| - name: "Build the test contracts" | ||
| run: "FOUNDRY_PROFILE=test-optimized forge build" | ||
|
|
||
| - name: "Cache the build so that it can be re-used by the other jobs" | ||
| uses: "actions/cache/save@v3" | ||
| with: | ||
| path: "out-optimized" | ||
| key: "foundry-build-${{ github.sha }}" | ||
| path: | | ||
| cache | ||
| out | ||
| out-optimized | ||
|
|
||
| - name: "Store the contract artifacts in CI" | ||
| uses: "actions/upload-artifact@v3" | ||
| with: | ||
| name: "contract-artifacts" | ||
| path: "artifacts" | ||
|
|
||
| - name: "Add build summary" | ||
| run: | | ||
| echo "## Build result" >> $GITHUB_STEP_SUMMARY | ||
|
|
@@ -120,13 +115,10 @@ jobs: | |
| with: | ||
| fail-on-cache-miss: true | ||
| key: "foundry-build-${{ github.sha }}" | ||
| path: | | ||
| cache | ||
| out | ||
| out-optimized | ||
| path: "out-optimized" | ||
|
|
||
| - name: "Run the unit tests against the optimized build" | ||
| run: "FOUNDRY_PROFILE=test-optimized forge test --match-path \"test/unit/**/*.sol\"" | ||
| run: 'FOUNDRY_PROFILE=test-optimized forge test --match-path "test/unit/**/*.sol"' | ||
|
|
||
| - name: "Add test summary" | ||
| run: | | ||
|
|
@@ -152,13 +144,10 @@ jobs: | |
| with: | ||
| fail-on-cache-miss: true | ||
| key: "foundry-build-${{ github.sha }}" | ||
| path: | | ||
| cache | ||
| out | ||
| out-optimized | ||
| path: "out-optimized" | ||
|
|
||
| - name: "Run the integration tests against the optimized build" | ||
| run: "FOUNDRY_PROFILE=test-optimized forge test --match-path \"test/integration/**/*.sol\"" | ||
| run: 'FOUNDRY_PROFILE=test-optimized forge test --match-path "test/integration/**/*.sol"' | ||
|
|
||
| - name: "Add test summary" | ||
| run: | | ||
|
|
@@ -182,13 +171,10 @@ jobs: | |
| with: | ||
| fail-on-cache-miss: true | ||
| key: "foundry-build-${{ github.sha }}" | ||
| path: | | ||
| cache | ||
| out | ||
| out-optimized | ||
| path: "out-optimized" | ||
|
|
||
| - name: "Run the utils tests against the optimized build" | ||
| run: "FOUNDRY_PROFILE=test-optimized forge test --match-path \"test/utils/**/*.sol\"" | ||
| run: 'FOUNDRY_PROFILE=test-optimized forge test --match-path "test/utils/**/*.sol"' | ||
|
|
||
| - name: "Add test summary" | ||
| run: | | ||
|
|
@@ -212,13 +198,10 @@ jobs: | |
| with: | ||
| fail-on-cache-miss: true | ||
| key: "foundry-build-${{ github.sha }}" | ||
| path: | | ||
| cache | ||
| out | ||
| out-optimized | ||
| path: "out-optimized" | ||
|
|
||
| - name: "Run the invariant tests against the optimized build" | ||
| run: "FOUNDRY_PROFILE=test-optimized forge test --match-path \"test/invariant/**/*.sol\"" | ||
| run: 'FOUNDRY_PROFILE=test-optimized forge test --match-path "test/invariant/**/*.sol"' | ||
|
|
||
| - name: "Add test summary" | ||
| run: | | ||
|
|
@@ -244,17 +227,14 @@ jobs: | |
| with: | ||
| fail-on-cache-miss: true | ||
| key: "foundry-build-${{ github.sha }}" | ||
| path: | | ||
| cache | ||
| out | ||
| out-optimized | ||
| path: "out-optimized" | ||
|
|
||
| - name: "Generate fuzz seed that changes weekly to avoid burning through RPC allowance" | ||
| run: | | ||
| echo "FOUNDRY_FUZZ_SEED=$(echo $(($EPOCHSECONDS / 604800)))" >> $GITHUB_ENV | ||
| - name: "Run the fork tests against the optimized build" | ||
| run: "FOUNDRY_PROFILE=test-optimized forge test --match-path \"test/fork/**/*.sol\"" | ||
| run: 'FOUNDRY_PROFILE=test-optimized forge test --match-path "test/fork/**/*.sol"' | ||
|
|
||
| - name: "Add test summary" | ||
| run: | | ||
|
|
@@ -274,7 +254,7 @@ jobs: | |
| uses: "foundry-rs/foundry-toolchain@v1" | ||
|
|
||
| - name: "Generate the coverage report using the unit and the integration tests" | ||
| run: "forge coverage --match-path \"test/{unit,integration}/**/*.sol\" --report lcov" | ||
| run: 'forge coverage --match-path "test/{unit,integration}/**/*.sol" --report lcov' | ||
|
|
||
| - name: "Upload coverage report to Codecov" | ||
| uses: "codecov/codecov-action@v3" | ||
|
|
@@ -285,33 +265,3 @@ jobs: | |
| run: | | ||
| echo "## Coverage result" >> $GITHUB_STEP_SUMMARY | ||
| echo "✅ Uploaded to Codecov" >> $GITHUB_STEP_SUMMARY | ||
| slither-analyze: | ||
| needs: ["lint", "build"] | ||
| runs-on: "ubuntu-latest" | ||
| permissions: | ||
| actions: "read" | ||
| contents: "read" | ||
| security-events: "write" | ||
| steps: | ||
| - name: "Check out the repo" | ||
| uses: "actions/checkout@v3" | ||
| with: | ||
| submodules: "recursive" | ||
|
|
||
| - name: "Run Slither analysis" | ||
| uses: "crytic/[email protected]" | ||
| id: "slither" | ||
| with: | ||
| fail-on: "none" | ||
| sarif: "results.sarif" | ||
|
|
||
| - name: "Upload SARIF file to GitHub code scanning" | ||
| uses: "github/codeql-action/upload-sarif@v2" | ||
| with: | ||
| sarif_file: ${{ steps.slither.outputs.sarif }} | ||
|
|
||
| - name: "Add Slither summary" | ||
| run: | | ||
| echo "## Slither result" >> $GITHUB_STEP_SUMMARY | ||
| echo "✅ Uploaded to GitHub code scanning" >> $GITHUB_STEP_SUMMARY | ||