Merge pull request #96 from kdarkhan/fuzz-caching-fix

Fix caching of `target` dir for `fuzz` project
google · Oct 5, 2023 · 941a188 · 941a188
2 parents 92d3514 + 963ba2f
commit 941a188
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 9 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -19,6 +19,8 @@ jobs:
         uses: actions/checkout@v3
 
       - uses: Swatinem/rust-cache@v2
+        with:
+          save-if: ${{ github.ref == 'refs/heads/main' }}
 
       - name: Test
         run: cargo test
@@ -35,19 +37,18 @@ jobs:
 
       - uses: Swatinem/rust-cache@v2
         with:
-          workspaces: fuzz -> target
+          # Default is "v0-rust"
+          # Use a separate key to prevent collision with main cache
+          prefix-key: "fuzz"
+          # Cache only on main build
+          save-if: ${{ github.ref == 'refs/heads/main' }}
+          cache-directories: |
+            i18n-helpers/fuzz/target
+            i18n-helpers/fuzz/corpus
 
       - name: Install cargo-fuzz
         run: cargo install cargo-fuzz
 
-      - name: Cache fuzz corpus
-        uses: actions/cache@v3
-        with:
-          path: fuzz/corpus
-          key: fuzz-corpus-${{ github.run_id }}
-          restore-keys: |
-            fuzz-corpus
-
       - name: Run group_events fuzzer and minimize corpus
         run: |
           cd i18n-helpers

diff --git a/i18n-helpers/fuzz/Cargo.lock b/i18n-helpers/fuzz/Cargo.lock