From 607289c88a9ac89bab193aedbfed89f39e6dfe20 Mon Sep 17 00:00:00 2001
From: Lars Lubkoll <11710767+lubkoll@users.noreply.github.com>
Date: Thu, 22 Aug 2024 13:51:24 +0000
Subject: [PATCH] [CI] Add cachepot -- part2: use cache

---
 .github/workflows/all_checks_osmosis.yml | 13 +++++++++----
 .github/workflows/cargo_build_cache.yml  |  9 ++++++---
 .github/workflows/rust_basic.yml         | 19 +++++++++++++------
 .github/workflows/test_tube.yml          |  4 ++++
 4 files changed, 32 insertions(+), 13 deletions(-)

diff --git a/.github/workflows/all_checks_osmosis.yml b/.github/workflows/all_checks_osmosis.yml
index 378082b54..c37bf9c3a 100644
--- a/.github/workflows/all_checks_osmosis.yml
+++ b/.github/workflows/all_checks_osmosis.yml
@@ -3,6 +3,10 @@ name: Test Tube (osmosis)
 on:
   workflow_call:
     inputs:
+      unittest:
+        required: false
+        default: false
+        type: boolean
       proptest:
         required: false
         default: false
@@ -36,7 +40,7 @@ jobs:
             ~/target
           key: ${{ runner.os }}-cargo-osmosis-${{github.sha}}
           restore-keys: ${{ runner.os }}-cargo-osmosis
-      - name: Update PATH and RUSTC_WRAPPER
+      - name: Update PATH
         run: echo "$HOME/.cargo/bin" >> $GITHUB_PATH
       - name: Install cachepot
         run: test -e $HOME/.cargo/bin/cachepot && echo "cachepot found -- skipping install" || cargo install --git https://github.com/paritytech/cachepot
@@ -45,7 +49,8 @@ jobs:
       - name: Rust lint
         run:  cargo clippy --features test-tube --all-targets -- -D warnings
         working-directory: smart-contracts/osmosis
-      - name: Unit tests
+      - if: inputs.unittest
+        name: Unit tests
         run:  cargo unit-test
         working-directory: smart-contracts/osmosis
       - name: Build merkle-incentives
@@ -72,8 +77,8 @@ jobs:
         env:
           PROPTEST_CASES: 10
         working-directory: smart-contracts/osmosis/contracts/cl-vault
-      - name: Store dependencies
-        if: inputs.store_deps
+      - if: inputs.store_deps
+        name: Store dependencies
         uses: actions/cache/save@v4
         with:
           path: |
diff --git a/.github/workflows/cargo_build_cache.yml b/.github/workflows/cargo_build_cache.yml
index 1921ab46b..1d1b107cc 100644
--- a/.github/workflows/cargo_build_cache.yml
+++ b/.github/workflows/cargo_build_cache.yml
@@ -7,6 +7,9 @@ on:
         required: true
         type: string
 
+env:
+  CARGO_TARGET_DIR: ~/target
+
 jobs:
   store-build-cache:
     runs-on: ubuntu-latest
@@ -22,11 +25,11 @@ jobs:
             ~/.cache/cachepot
             ~/.cargo
             ~/go
-            smart-contracts/${{ inputs.workspace }}/**/target
+            ~/target
           key: ${{ runner.os }}-cargo-${{ inputs.workspace }}-${{github.sha}}
           restore-keys: ${{ runner.os }}-cargo-${{ inputs.workspace }}
       - name: Install cachepot
-        run: test -e $HOME/.cargo/bin/cachepot && echo "cachepot found -- skipping install" || cargo install --git https://github.com/paritytech/cachepot
+        run: test -e ~/.cargo/bin/cachepot && echo "cachepot found -- skipping install" || cargo install --git https://github.com/paritytech/cachepot
       - name: Rust check
         run:  cargo check --all-targets --features test-tube
         working-directory: smart-contracts/${{ inputs.workspace }}
@@ -39,5 +42,5 @@ jobs:
             ~/.cache/cachepot
             ~/.cargo
             ~/go
-            smart-contracts/${{ inputs.workspace }}/**/target
+            ~/target
           key: ${{ runner.os }}-cargo-${{ inputs.workspace }}-${{github.sha}}
diff --git a/.github/workflows/rust_basic.yml b/.github/workflows/rust_basic.yml
index 2e5549cdc..c01b45948 100644
--- a/.github/workflows/rust_basic.yml
+++ b/.github/workflows/rust_basic.yml
@@ -14,6 +14,10 @@ on:
         default: true
         type: boolean
 
+env:
+  RUSTC_WRAPPER: cachepot
+  CARGO_TARGET_DIR: ~/target
+
 jobs:
   checks:
     runs-on: ubuntu-latest
@@ -26,18 +30,21 @@ jobs:
         uses: actions/cache/restore@v4
         with:
           path: |
+            ~/.cache/cachepot
             ~/.cargo
             ~/go
-            smart-contracts/${{ inputs.workspace }}/**/target
-          key: ${{ runner.os }}-cargo-${{ inputs.workspace }}-$GITHUB_SHA
+            ~/target
+          key: ${{ runner.os }}-cargo-${{ inputs.workspace }}-${{github.sha}}
           restore-keys: ${{ runner.os }}-cargo-${{ inputs.workspace }}
+      - name: Update PATH
+        run: echo "$HOME/.cargo/bin" >> $GITHUB_PATH
       - name: Rust lint
-        run: cargo lint
-        working-directory: smart-contracts/${{ inputs.workspace }}/${{ inputs.target }}
+        run:  cargo clippy --features test-tube --all-targets -- -D warnings
+        working-directory: smart-contracts/${{ inputs.workspace }}
       - name: Rust format check
         run: cargo fmt --all -- --check
         working-directory: smart-contracts/${{ inputs.workspace }}/${{ inputs.target }}
-      - name: Run unit-tests
-        if: inputs.unit_tests
+      - if: inputs.unit_tests
+        name: Run unit-tests
         run: cargo unit-test
         working-directory: smart-contracts/${{ inputs.workspace }}/${{ inputs.target }}
diff --git a/.github/workflows/test_tube.yml b/.github/workflows/test_tube.yml
index 2cb68d52c..b94245b19 100644
--- a/.github/workflows/test_tube.yml
+++ b/.github/workflows/test_tube.yml
@@ -24,5 +24,9 @@ jobs:
     uses: ./.github/workflows/all_checks_osmosis.yml
     if: github.event_name == 'push'
     with:
+      unittest: true
       proptest: true
       store_deps: true
+  pre-commit:
+    uses: ./.github/workflows/all_checks_osmosis.yml
+    if: github.event_name == 'pull_request'