test: add an installation CI test.

For some reason, cargo install and cargo build do not do exactly the same thing. My commit at Arnaud-de-Grandmaison-ARM@ed6ce14 broke cargo install without us noting it or CI catching it. This commit adds a test to cover this. It should be noted that cargo install does not yet (or still does not !) play nice with workspace, so we need to point it to each of the individual binaries that we want to install, for reference see: - rust-lang/cargo#7599 - rust-lang/cargo#4101 - rust-lang/cargo#7124 We install each of the binaries into a temporary directory provided by the runner and that will be cleaned up automatically when the job finishes. Signed-off-by: Arnaud de Grandmaison <[email protected]>
veraison · Oct 9, 2024 · f182a94 · f182a94
1 parent 4519538
commit f182a94
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml
@@ -27,3 +27,7 @@ jobs:
       run: cargo build --manifest-path=rust-keybroker/Cargo.toml --verbose
     - name: Run tests
       run: cargo test --manifest-path=rust-keybroker/Cargo.toml --verbose
+    - name: Install keybroker-app
+      run: cargo install --path=rust-keybroker/keybroker-app --root $RUNNER_TEMP/keybroker-demo
+    - name: Install keybroker-server
+      run: cargo install --path=rust-keybroker/keybroker-server --root $RUNNER_TEMP/keybroker-demo