test(cli): tighten last-export label assertions to exact match
Drop the dead `stdout.contains("last export:")` + `.to_lowercase()` fallback
in status_shows_last_backup_line and status_shows_recent_backup_after_export;
assert `stdout.contains("Last export:")` verbatim instead.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -115,8 +115,7 @@ fn status_shows_last_backup_line() {
|
|||||||
let out = v.run(&["status"]);
|
let out = v.run(&["status"]);
|
||||||
assert!(out.status.success());
|
assert!(out.status.success());
|
||||||
let stdout = String::from_utf8(out.stdout).unwrap();
|
let stdout = String::from_utf8(out.stdout).unwrap();
|
||||||
assert!(stdout.contains("last export:") || stdout.to_lowercase().contains("last export:"),
|
assert!(stdout.contains("Last export:"), "missing last export line: {stdout}");
|
||||||
"missing last export line: {stdout}");
|
|
||||||
assert!(stdout.contains("never"), "fresh vault should report 'never': {stdout}");
|
assert!(stdout.contains("never"), "fresh vault should report 'never': {stdout}");
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -130,8 +129,7 @@ fn status_shows_recent_backup_after_export() {
|
|||||||
);
|
);
|
||||||
let out = v.run(&["status"]);
|
let out = v.run(&["status"]);
|
||||||
let stdout = String::from_utf8(out.stdout).unwrap();
|
let stdout = String::from_utf8(out.stdout).unwrap();
|
||||||
assert!(stdout.contains("last export:") || stdout.to_lowercase().contains("last export:"),
|
assert!(stdout.contains("Last export:"), "{stdout}");
|
||||||
"{stdout}");
|
|
||||||
assert!(!stdout.contains("never"), "should NOT say 'never' after export: {stdout}");
|
assert!(!stdout.contains("never"), "should NOT say 'never' after export: {stdout}");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user