refactor: build.yaml would pass on the hash of madara bin to use (WIP)

.github/workflows/build.yml

@@ -7,6 +7,8 @@ jobs:
     permissions:
       pull-requests: write
     runs-on: ubuntu-latest
+    outputs:
+      binary_hash: ${{ steps.binary_hash.outputs.hash }}
 
     steps:
       - uses: actions/checkout@v3

.github/workflows/pull-request.yml

@@ -48,8 +48,3 @@ jobs:
     name: Build Madara
     uses: ./.github/workflows/build.yml
     needs: changelog
-
-  js_test:
-    name: Run JS Tests
-    uses: ./.github/workflows/starknet-js-test.yml
-    needs: build

.github/workflows/starknet-js-test.yml

@@ -2,21 +2,28 @@ name: Starknet-js Compatibility Tests
 
 on:
   workflow_dispatch:
+  workflow_run:
+    workflows: ["build"]
+    types:
+      - completed
   workflow_call:
+    inputs:
+      binary_hash:
+        required: true
+        type: string
 
 jobs:
   js-test:
     runs-on: ubuntu-latest
+    if: ${{ github.event.workflow_run.conclusion == 'success' }}
     steps:
       - uses: actions/checkout@v3
       - name: Restore Madara binary
         uses: actions/cache@v3
         id: cache-madara
         with:
           path: target/release/madara
-          key: ${{ runner.os }}-madara-bin-
-          restore-keys: |
-            ${{ runner.os }}-madara-bin-
+          key: ${{ runner.os }}-madara-bin-${{ inputs.binary_hash }}
           fail-on-cache-miss: true
       - name: Restore Cairo build artifacts
         uses: actions/cache@v3

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## Next release
 
+- refactor: starknet js workflow updated to take the right binary hash
 - fix(rocksdb): update max open files opt
 - code: refactor to use otel tracing instead of prometheus (removed mc-metrics, added mc-analytics)
 - fix(version constants): 0.13.2 was mapped to wrong constants