test: simplify boolean assertions in import_export_sync tests
Replace verbose assert_eq!(value, true) with idiomatic assert!(value) for improved readability and adherence to Rust best practices
This commit is contained in:
@@ -498,8 +498,8 @@ url = "https://example.com"
|
|||||||
.expect("unified servers should exist");
|
.expect("unified servers should exist");
|
||||||
|
|
||||||
let echo = servers.get("echo_server").expect("echo server");
|
let echo = servers.get("echo_server").expect("echo server");
|
||||||
assert_eq!(
|
assert!(
|
||||||
echo.apps.codex, true,
|
echo.apps.codex,
|
||||||
"Codex app should be enabled for echo_server"
|
"Codex app should be enabled for echo_server"
|
||||||
);
|
);
|
||||||
let server_spec = echo.server.as_object().expect("server spec");
|
let server_spec = echo.server.as_object().expect("server spec");
|
||||||
@@ -512,8 +512,8 @@ url = "https://example.com"
|
|||||||
);
|
);
|
||||||
|
|
||||||
let http = servers.get("http_server").expect("http server");
|
let http = servers.get("http_server").expect("http server");
|
||||||
assert_eq!(
|
assert!(
|
||||||
http.apps.codex, true,
|
http.apps.codex,
|
||||||
"Codex app should be enabled for http_server"
|
"Codex app should be enabled for http_server"
|
||||||
);
|
);
|
||||||
let http_spec = http.server.as_object().expect("http spec");
|
let http_spec = http.server.as_object().expect("http spec");
|
||||||
@@ -577,10 +577,7 @@ command = "echo"
|
|||||||
.expect("existing entry");
|
.expect("existing entry");
|
||||||
|
|
||||||
// 验证 Codex 应用已启用
|
// 验证 Codex 应用已启用
|
||||||
assert_eq!(
|
assert!(entry.apps.codex, "Codex app should be enabled after import");
|
||||||
entry.apps.codex, true,
|
|
||||||
"Codex app should be enabled after import"
|
|
||||||
);
|
|
||||||
|
|
||||||
// 验证现有配置被保留(server 不应被覆盖)
|
// 验证现有配置被保留(server 不应被覆盖)
|
||||||
let spec = entry.server.as_object().expect("server spec");
|
let spec = entry.server.as_object().expect("server spec");
|
||||||
@@ -702,8 +699,8 @@ fn import_from_claude_merges_into_config() {
|
|||||||
.expect("entry exists");
|
.expect("entry exists");
|
||||||
|
|
||||||
// 验证 Claude 应用已启用
|
// 验证 Claude 应用已启用
|
||||||
assert_eq!(
|
assert!(
|
||||||
entry.apps.claude, true,
|
entry.apps.claude,
|
||||||
"Claude app should be enabled after import"
|
"Claude app should be enabled after import"
|
||||||
);
|
);
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user