mirror of
https://github.com/fabro-sh/fabro.git
synced 2026-09-07 08:27:12 +00:00
fa73407b35
Add a server-targeted `fabro version` command for checking client and server build identity without reading local storage directly. This also removes version data from `/health`, moves doctor parity checks to diagnostics, and updates the API spec, docs, generated client, and coverage for the new contract.
112 lines
3.6 KiB
Rust
112 lines
3.6 KiB
Rust
use fabro_test::{fabro_snapshot, test_context};
|
|
use serde_json::Value;
|
|
|
|
#[test]
|
|
fn help() {
|
|
let context = test_context!();
|
|
let mut cmd = context.command();
|
|
cmd.args(["version", "--help"]);
|
|
fabro_snapshot!(context.filters(), cmd, @"
|
|
success: true
|
|
exit_code: 0
|
|
----- stdout -----
|
|
Show client and server version information
|
|
|
|
Usage: fabro version [OPTIONS]
|
|
|
|
Options:
|
|
--json Output as JSON [env: FABRO_JSON=]
|
|
--server <SERVER> Fabro server target: http(s) URL or absolute Unix socket path [env: FABRO_SERVER=]
|
|
--debug Enable DEBUG-level logging (default is INFO) [env: FABRO_DEBUG=]
|
|
--no-upgrade-check Disable automatic upgrade check [env: FABRO_NO_UPGRADE_CHECK=true]
|
|
--quiet Suppress non-essential output [env: FABRO_QUIET=]
|
|
--verbose Enable verbose output [env: FABRO_VERBOSE=]
|
|
-h, --help Print help
|
|
----- stderr -----
|
|
");
|
|
}
|
|
|
|
#[test]
|
|
fn client_info_always_prints() {
|
|
let context = test_context!();
|
|
let output = context
|
|
.command()
|
|
.args(["version"])
|
|
.output()
|
|
.expect("command should run");
|
|
|
|
assert!(output.status.success(), "version failed");
|
|
let stdout = String::from_utf8(output.stdout).expect("stdout should be utf-8");
|
|
assert!(
|
|
stdout.contains("Client:"),
|
|
"missing client section: {stdout}"
|
|
);
|
|
assert!(
|
|
stdout.contains("Server:"),
|
|
"missing server section: {stdout}"
|
|
);
|
|
}
|
|
|
|
#[test]
|
|
fn json_output() {
|
|
let context = test_context!();
|
|
let output = context
|
|
.command()
|
|
.args(["--json", "version"])
|
|
.output()
|
|
.expect("command should run");
|
|
|
|
assert!(output.status.success(), "json version failed");
|
|
let value: Value = serde_json::from_slice(&output.stdout).expect("json should parse");
|
|
assert!(value["client"]["version"].is_string());
|
|
assert!(value["server"]["version"].is_string());
|
|
}
|
|
|
|
#[test]
|
|
fn http_unreachable_shows_error() {
|
|
let context = test_context!();
|
|
let output = context
|
|
.command()
|
|
.args(["version", "--server", "http://127.0.0.1:1"])
|
|
.output()
|
|
.expect("command should run");
|
|
|
|
assert!(output.status.success(), "version should still succeed");
|
|
let stdout = String::from_utf8(output.stdout).expect("stdout should be utf-8");
|
|
assert!(
|
|
stdout.contains("Client:"),
|
|
"missing client section: {stdout}"
|
|
);
|
|
assert!(stdout.contains("Error:"), "missing error section: {stdout}");
|
|
}
|
|
|
|
#[test]
|
|
fn http_unreachable_json() {
|
|
let context = test_context!();
|
|
let output = context
|
|
.command()
|
|
.args(["--json", "version", "--server", "http://127.0.0.1:1"])
|
|
.output()
|
|
.expect("command should run");
|
|
|
|
assert!(output.status.success(), "version should still succeed");
|
|
let value: Value = serde_json::from_slice(&output.stdout).expect("json should parse");
|
|
assert!(value["client"]["version"].is_string());
|
|
assert!(value["server"]["address"].is_string());
|
|
assert!(value["server"]["error"].is_string());
|
|
assert!(value["server"].get("version").is_none());
|
|
}
|
|
|
|
#[test]
|
|
fn invalid_server_target_fails() {
|
|
let context = test_context!();
|
|
let output = context
|
|
.command()
|
|
.args(["version", "--server", "not-a-url"])
|
|
.output()
|
|
.expect("command should run");
|
|
|
|
assert!(!output.status.success(), "version should fail");
|
|
let stderr = String::from_utf8(output.stderr).expect("stderr should be utf-8");
|
|
assert!(!stderr.is_empty(), "expected error output");
|
|
}
|