Cli: tests

Author: TimeEngineer

.config/nextest.toml

@@ -1,6 +1,8 @@
 [profile.default]
 # https://nexte.st/book/slow-tests.html
 slow-timeout = { period = "10s", terminate-after = 3 } # Timeout a test after 30s (3 * 10s)
+# Allow the server to shutdown after the tests
+leak-timeout = "500ms"
 
 [profile.ci]
 slow-timeout = { period = "10s", terminate-after = 5 } # Timeout a test after 50s (5 * 10s)
@@ -12,3 +14,5 @@ status-level = "all"
 failure-output = "immediate-final"
 # Continue even if a test fail.
 fail-fast = false
+# Allow the server to shutdown after the tests
+leak-timeout = "500ms"

.github/workflows/ci-rust.yml

@@ -26,6 +26,7 @@ concurrency:
   cancel-in-progress: true
 
 env:
+  poetry-version: 1.5.1
   CARGO_CI_FLAGS: --locked --profile ci-rust
   CARGO_NEXTEST_CI_FLAGS: --profile=ci --locked --cargo-profile ci-rust
   WINFSP_VERSION: 2.0.23075
@@ -89,6 +90,25 @@ jobs:
         with:
           tool: [email protected], [email protected], [email protected]
 
+      - uses: ./.github/actions/setup-python-poetry
+        id: setup-python
+        with:
+          poetry-version: ${{ env.poetry-version }}
+          project-path: ./server
+        timeout-minutes: 10
+
+      - name: Install python dependencies
+        run: python make.py python-dev-install
+        timeout-minutes: 10
+
+      - name: Build CLI binary
+        run: cargo build --profile ci-rust --package parsec_cli --features testenv
+        timeout-minutes: 5
+
+      - name: Test CLI
+        run: cargo nextest run ${{ env.CARGO_NEXTEST_CI_FLAGS }} --package parsec_cli --features testenv
+        timeout-minutes: 10
+
       - name: Categorize workspace crates
         id: crates
         run: |
@@ -224,3 +244,22 @@ jobs:
         env:
           RUST_LOG: debug
           TESTBED_SERVER: http://localhost:6777
+
+      - uses: ./.github/actions/setup-python-poetry
+        id: setup-python
+        with:
+          poetry-version: ${{ env.poetry-version }}
+          project-path: ./server
+        timeout-minutes: 10
+
+      - name: Install python dependencies
+        run: python make.py python-dev-install
+        timeout-minutes: 10
+
+      - name: Build CLI binary
+        run: cargo build --profile ci-rust --package parsec_cli --features testenv
+        timeout-minutes: 5
+
+      - name: Test CLI
+        run: cargo nextest run ${{ env.CARGO_NEXTEST_CI_FLAGS }} --package parsec_cli --features testenv
+        timeout-minutes: 20

Cargo.lock

@@ -92,6 +92,7 @@ libparsec_tests_lite = { path = "libparsec/crates/tests_lite", default-features
 android_logger = { version = "0.13", default-features = false }
 anyhow = { version = "1.0.75", default-features = false }
 argon2 = { version = "0.5.2", default-features = false }
+assert_cmd = { version = "2.0.12", default-features = false }
 async-lock = { version = "2.8.0", default-features = false }
 async-broadcast = { version = "0.5.1", default-features = false }
 async-trait = { version = "0.1.74", default-features = false }
@@ -148,6 +149,7 @@ openssl = { version = "0.10.61", default-features = false }
 paste = { version = "1.0.14", default-features = false }
 pin-project-lite = { version = "0.2.13", default-features = false }
 percent-encoding = { version = "2.3.1", default-features = false }
+predicates = { version = "3.0.4", default-features = false }
 pretty_assertions = { version = "1.4.0", default-features = false }
 proc-macro2 = { version = "1.0.70", default-features = false }
 pyo3 = { version = "0.19.2", default-features = false }

Cargo.toml

@@ -25,3 +25,10 @@ serde_json = { workspace = true }
 terminal-spinners = { workspace = true }
 tokio = { workspace = true, features = ["rt-multi-thread", "macros"] }
 uuid = { workspace = true }
+
+[dev-dependencies]
+libparsec = { workspace = true, features = ["cli-tests"] }
+
+assert_cmd = { workspace = true }
+predicates = { workspace = true, features = ["regex"] }
+rstest = { workspace = true }

cli/Cargo.toml

@@ -58,6 +58,9 @@ pub async fn bootstrap_organization(
         HumanHandle::new(&bootstrap_organization.email, &bootstrap_organization.label)
             .map_err(|e| anyhow::anyhow!("Cannot create human handle: {e}"))?;
 
+    #[cfg(feature = "testenv")]
+    let password = "test".to_string().into();
+    #[cfg(not(feature = "testenv"))]
     let password = rpassword::prompt_password("password:")?.into();
 
     let handle = SpinnerBuilder::new()

cli/src/bootstrap_organization.rs

@@ -69,9 +69,7 @@ pub async fn create_organization(create_organization: CreateOrganization) -> any
     let organization_addr =
         BackendOrganizationBootstrapAddr::new(addr, organization_id, Some(bootstrap_token));
 
-    let organization_addr_display = organization_addr.to_url();
-
-    println!("Organization bootstrap url: {YELLOW}{organization_addr_display}{RESET}");
+    println!("Organization bootstrap url: {YELLOW}{organization_addr}{RESET}");
 
     Ok(())
 }

cli/src/create_organization.rs

@@ -16,13 +16,14 @@ pub struct ListDevices {
 
 pub async fn list_devices(list_devices: ListDevices) -> anyhow::Result<()> {
     let config_dir = list_devices.config_dir.unwrap_or(get_default_config_dir());
+    let config_dir_str = config_dir.to_string_lossy();
     let devices = list_available_devices(&config_dir).await;
 
     if devices.is_empty() {
-        println!("No devices found in {YELLOW}{config_dir:?}{RESET}");
+        println!("No devices found in {YELLOW}{config_dir_str}{RESET}");
     } else {
         let n = devices.len();
-        println!("Found {GREEN}{n}{RESET} device(s) in {YELLOW}{config_dir:?}{RESET}:");
+        println!("Found {GREEN}{n}{RESET} device(s) in {YELLOW}{config_dir_str}{RESET}:");
         format_devices(&devices);
     }
 

cli/src/list_devices.rs

@@ -5,6 +5,8 @@ mod create_organization;
 mod list_devices;
 #[cfg(feature = "testenv")]
 mod run_testenv;
+#[cfg(all(test, feature = "testenv"))]
+mod tests;
 mod utils;
 
 use clap::{Parser, Subcommand};