Enable metadata feature flag

Analog-Labs · Oct 30, 2023 · fe47005 · fe47005
1 parent 3726bf7
commit fe47005
Show file tree

Hide file tree

Showing 2 changed files with 76 additions and 6 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -3,6 +3,7 @@ on:
     branches:
     - master
     - staging
+    - release-v*
   pull_request:
 
 name: rosetta-wallet
@@ -12,7 +13,7 @@ jobs:
     runs-on: self-hosted
     steps:
     - name: Checkout sources
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
 
     - name: Install rust toolchain
       uses: actions-rs/toolchain@v1
@@ -42,7 +43,7 @@ jobs:
     runs-on: self-hosted
     steps:
     - name: Checkout sources
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
 
     - name: Install rust toolchain
       uses: actions-rs/toolchain@v1
@@ -57,16 +58,62 @@ jobs:
     runs-on: self-hosted
     steps:
       - name: Checkout sources
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
 
       - name: cargo-deny check
         uses: EmbarkStudios/cargo-deny-action@v1
 
+  # Test each client individually, once testing the whole workspace can hide some compilation errors
+  # ex: testing using --workspace enable the same feature flags for all crates
+  test-client:
+    needs: [rustfmt, clippy]
+    strategy:
+      matrix:
+        crate: [rosetta-server-astar, rosetta-server-bitcoin, rosetta-server-ethereum, rosetta-server-polkadot, rosetta-client]
+    name: test ${{ matrix.crate }}
+    runs-on: self-hosted
+    steps:
+    - name: Checkout sources
+      uses: actions/checkout@v4
+
+    - run: sudo add-apt-repository ppa:ethereum/ethereum
+    - run: sudo apt-get update -y
+    - name: Install deps
+      run: sudo apt-get install -y musl-dev musl-tools solc
+
+    - name: Install rust toolchain
+      uses: actions-rs/toolchain@v1
+      with:
+        toolchain: stable
+        components: clippy
+        target: x86_64-unknown-linux-musl
+
+    - name: cargo clippy (${{ matrix.crate }})
+      run: |
+        cargo clippy --locked -p ${{ matrix.crate }} --examples --tests -- \
+          -Dwarnings \
+          -Dclippy::unwrap_used \
+          -Dclippy::expect_used \
+          -Dclippy::nursery \
+          -Dclippy::pedantic \
+          -Aclippy::module_name_repetitions
+  
+    - name: Cleanup Docker
+      run: ./scripts/reset_docker.sh
+
+    - name: Pull nodes
+      run: ./scripts/pull_nodes.sh
+
+    - name: test (${{ matrix.crate }})
+      run: cargo test --locked -p ${{ matrix.crate }}
+
+  # Test all crates, excluding the clients
   test:
+    needs: [rustfmt, clippy]
     runs-on: self-hosted
     steps:
     - name: Checkout sources
-      uses: actions/checkout@v3
+      uses: actions/checkout@v4
 
     - run: sudo add-apt-repository ppa:ethereum/ethereum
     - run: sudo apt-get update -y
@@ -77,7 +124,24 @@ jobs:
       uses: actions-rs/toolchain@v1
       with:
         toolchain: stable
+        components: clippy
         target: x86_64-unknown-linux-musl
+
+    - name: clippy
+      run: |
+        cargo clippy --locked --workspace --examples --tests --all-features \
+          --exclude rosetta-server-astar \
+          --exclude rosetta-server-bitcoin \
+          --exclude rosetta-server-ethereum \
+          --exclude rosetta-server-polkadot \
+          --exclude rosetta-client \
+          -- \
+          -Dwarnings \
+          -Dclippy::unwrap_used \
+          -Dclippy::expect_used \
+          -Dclippy::nursery \
+          -Dclippy::pedantic \
+          -Aclippy::module_name_repetitions
 
     - name: Cleanup Docker
       run: ./scripts/reset_docker.sh
@@ -86,4 +150,10 @@ jobs:
       run: ./scripts/pull_nodes.sh
 
     - name: cargo test
-      run: cargo test --locked --all-features --workspace
+      run: |
+        cargo test --locked --workspace --all-features \
+          --exclude rosetta-server-astar \
+          --exclude rosetta-server-bitcoin \
+          --exclude rosetta-server-ethereum \
+          --exclude rosetta-server-polkadot \
+          --exclude rosetta-client
diff --git a/chains/astar/server/Cargo.toml b/chains/astar/server/Cargo.toml
@@ -14,7 +14,7 @@ futures = { version = "0.3", default-features = false, features = ["std"] }
 hex = "0.4"
 log = "0.4"
 parity-scale-codec = { workspace = true, features = ["derive"] }
-rosetta-config-astar.workspace = true
+rosetta-config-astar = { workspace = true, features = ["astar-metadata"] }
 rosetta-config-ethereum.workspace = true
 rosetta-core.workspace = true
 rosetta-server = { workspace = true, features = ["ws", "webpki-tls"] }