From 17f1f89fef4a6bf520f4b09a2116548b967d030e Mon Sep 17 00:00:00 2001 From: Codex-ytr83r Date: Tue, 22 Sep 2026 23:32:36 -0700 Subject: [PATCH] Align traceability placement assertions with captured-view reveal --- crates/oa-desktop/src/app/traceability_navigation_tests.rs | 5 ++++- harness/traceability-view-20260923.md | 5 +++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/crates/oa-desktop/src/app/traceability_navigation_tests.rs b/crates/oa-desktop/src/app/traceability_navigation_tests.rs index 80b34948..e9558874 100644 --- a/crates/oa-desktop/src/app/traceability_navigation_tests.rs +++ b/crates/oa-desktop/src/app/traceability_navigation_tests.rs @@ -175,9 +175,11 @@ fn mixed_view_show_keeps_its_context_for_every_diagram_app_and_target_layer() { let before = app.project().clone(); let action = show_action(&mut app, target); send(&mut app, action); + // The existing RevealInDiagram action synchronizes the layer + // to the captured view, even if the prior perspective differed. assert_eq!( (app.active_diagram, app.layer, app.selected), - (Some(mixed), app_layer, target) + (Some(mixed), diagram_layer, target) ); let added = app.project().diagrams[&mixed] .nodes @@ -185,6 +187,7 @@ fn mixed_view_show_keeps_its_context_for_every_diagram_app_and_target_layer() { .find(|n| n.element == target) .unwrap() .clone(); + assert_eq!(app.selected_node, Some(added.id)); let mut expected = before; expected .diagrams diff --git a/harness/traceability-view-20260923.md b/harness/traceability-view-20260923.md index 2724e363..13f76426 100644 --- a/harness/traceability-view-20260923.md +++ b/harness/traceability-view-20260923.md @@ -64,8 +64,9 @@ ordinary OA/SA edges, multiple traces, explicit hides plus disabled Trace kind, port owners and relation correspondence. Do not infer visible lines from unfiltered edge_availability or confuse unrelated model traces with counterparts. -Native acceptance must start with an OA-only view and existing named SA -counterpart, find/place it through the visible controls, see/select the trace, +Native acceptance starts with an OA-owned mixed view, one already visible +correspondence pair and another existing named SA counterpart not yet placed. +Find/place it through the visible controls, see/select the additional trace, inspect an absent-line visibility reason, Undo/Redo and Save/reopen. Inspect toolbar/canvas space and chooser clarity at a realistic window size. Different help text or count tests alone do not resolve the user's comprehension complaint.