test(cli): the IIF warning count ignores P8's audit notice; the harness asks the platform where the root is
GitHub's check on 880278ea: `cpa_package_holds_the_year_and_its_evidence`
counts the IIF transliteration warnings, and P8's "No counted audit
stands behind this package" notice (a warning while the books hold no
audit run) made four into five; the count now excludes it.
The portable macOS job, now that the harness prints the server's log:
"there is no books' private folder yet: run `wanderbooks setup`
first". `Home::root()` spelled out `.local/share/wanderbooks`, the
Linux data folder; `setup --home` on macOS makes the root under
`Library/Application Support`. The harness asks `default_root` now,
the same answer on every OS.
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HS8WZziDnyxunC23MUxrLp
This commit is contained in:
1 parent
2774680d65
commit
2a6bac02d5
2 files changed
+8
-1
No files matched your search
@@ -78,8 +78,12 @@ impl Home {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// The root `setup --home` made: the platform's data folder under the
|
||||||
|
/// home (`.local/share` on Linux, `Library/Application Support` on
|
||||||
|
/// macOS), never a Linux path spelled out (the portable macOS job
|
||||||
|
/// found the server refusing a root that was never set up).
|
||||||
pub fn root(&self) -> PathBuf {
|
pub fn root(&self) -> PathBuf {
|
||||||
self.dir.path().join(".local/share/wanderbooks")
|
ledger_store::root::default_root(self.dir.path())
|
||||||
}
|
}
|
||||||
|
|
||||||
pub fn run(&self) -> PathBuf {
|
pub fn run(&self) -> PathBuf {
|
||||||
|
|||||||
@@ -3497,11 +3497,14 @@ fn cpa_package_holds_the_year_and_its_evidence() {
|
|||||||
// and the em dashes of the depreciation memos: written differently, so
|
// and the em dashes of the depreciation memos: written differently, so
|
||||||
// said, the memos grouped (four lines for one name and four memos), in
|
// said, the memos grouped (four lines for one name and four memos), in
|
||||||
// warnings and the README alike.
|
// warnings and the README alike.
|
||||||
|
// (The audit notice P8 adds while no counted audit stands behind the
|
||||||
|
// package is a warning too; it is not about the IIF.)
|
||||||
let warnings: Vec<String> = out["warnings"]
|
let warnings: Vec<String> = out["warnings"]
|
||||||
.as_array()
|
.as_array()
|
||||||
.unwrap()
|
.unwrap()
|
||||||
.iter()
|
.iter()
|
||||||
.map(|w| w.as_str().unwrap().to_string())
|
.map(|w| w.as_str().unwrap().to_string())
|
||||||
|
.filter(|w| !w.starts_with("No counted audit"))
|
||||||
.collect();
|
.collect();
|
||||||
assert_eq!(warnings.len(), 4, "{out}");
|
assert_eq!(warnings.len(), 4, "{out}");
|
||||||
let readme = std::str::from_utf8(&files["README.txt"]).unwrap();
|
let readme = std::str::from_utf8(&files["README.txt"]).unwrap();
|
||||||
|
|||||||
Reference in new issue
Block a user