mirror of
https://github.com/storytold/photocraft.git
synced 2026-10-27 19:36:56 -06:00
optimalbyteeandClaude Opus 5.5
cc0441f9ae
A successful Save As (UI, automation or headless) renames the document to the written file, so its tab and title stop saying Untitled. The name is file identity, not a history step: undo and redo keep it. Failed writes, Save a Copy and exports leave the name, path and dirty state alone, and a deferred save no longer steals the selected tab. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
58 lines
3.0 KiB
Rust
58 lines
3.0 KiB
Rust
//! Regression for #1350: saving an inactive .pcraft file must not select it.
|
|
|
|
use photocraft_automation::Headless;
|
|
use serde_json::json;
|
|
|
|
#[test]
|
|
fn indexed_native_save_preserves_active_document_and_future_command_target() {
|
|
let dir = std::env::temp_dir().join(format!(
|
|
"pc-inactive-save-{}-{}",
|
|
std::process::id(),
|
|
std::time::SystemTime::now().duration_since(std::time::UNIX_EPOCH).unwrap().as_nanos()
|
|
));
|
|
std::fs::create_dir_all(&dir).unwrap();
|
|
let path = dir.join("first.pcraft");
|
|
let mut headless = Headless::trusted_local();
|
|
|
|
headless.handle("doc.new", json!({"width": 2, "height": 2, "background": "white", "name": "first"})).unwrap();
|
|
headless.handle("doc.new", json!({"width": 2, "height": 2, "background": "white", "name": "second"})).unwrap();
|
|
assert_eq!(headless.session.active_index(), Some(1));
|
|
let first_revision = headless.session.documents()[0].revision;
|
|
let second_revision = headless.session.documents()[1].revision;
|
|
|
|
headless.handle("doc.save", json!({"index": 0, "path": path.to_str().unwrap()})).unwrap();
|
|
assert!(path.is_file(), "inactive document was saved");
|
|
assert_eq!(headless.session.active_index(), Some(1), "save must not select document 0");
|
|
let first = &headless.session.documents()[0];
|
|
assert_eq!(first.path.as_deref(), path.to_str());
|
|
assert_eq!(first.doc.name, "first.pcraft");
|
|
assert_eq!(first.revision, first.saved_revision, "saved document is clean");
|
|
|
|
headless.handle("engine.execute", json!({"command": "image.adjustments.invert", "params": {}})).unwrap();
|
|
assert_eq!(headless.session.active_index(), Some(1));
|
|
assert_eq!(headless.session.documents()[0].revision, first_revision);
|
|
assert!(headless.session.documents()[1].revision > second_revision);
|
|
for name in ["保存 first.psd", "renamed.psb"] {
|
|
let path = dir.join(name);
|
|
headless.handle("doc.save", json!({"index": 0, "path": path.to_str().unwrap()})).unwrap();
|
|
assert_eq!(headless.session.active_index(), Some(1));
|
|
assert_eq!(headless.session.documents()[0].doc.name, name);
|
|
assert_eq!(headless.session.documents()[1].doc.name, "second");
|
|
assert!(!headless.session.documents()[0].is_dirty());
|
|
}
|
|
let before = headless.session.documents()[0].clone();
|
|
let blocked = dir.join("not-a-directory");
|
|
std::fs::write(&blocked, b"blocks the parent directory").unwrap();
|
|
let failed = blocked.join("fail.psd");
|
|
assert!(headless.handle("doc.save", json!({"index": 0, "path": failed.to_str().unwrap()})).is_err());
|
|
assert_eq!(headless.session.active_index(), Some(1));
|
|
assert_eq!(headless.session.documents()[0].doc.name, before.doc.name);
|
|
assert_eq!(headless.session.documents()[0].path, before.path);
|
|
let copy = dir.join("copy.png");
|
|
headless.handle("doc.save", json!({"index": 0, "path": copy.to_str().unwrap()})).unwrap();
|
|
assert_eq!(headless.session.documents()[0].doc.name, before.doc.name);
|
|
assert_eq!(headless.session.documents()[0].path, before.path);
|
|
drop(headless);
|
|
std::fs::remove_dir_all(dir).unwrap();
|
|
}
|