Skip to content

Commit b2c4965

Browse files
committed
fix(test): locate duplicate/result via group in geometry_results test
After eba8385, plot_results creates a "Results" group in the signal panel, breaking panel-wide index lookups. Use group-local lookups so the test is robust against extra groups. (cherry picked from commit 5e08f9c)
1 parent 7b614b0 commit b2c4965

1 file changed

Lines changed: 12 additions & 3 deletions

File tree

datalab/tests/features/common/geometry_results_app_test.py

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -116,11 +116,20 @@ def test_geometry_results() -> None:
116116
with Conf.proc.keep_results.temp(keep_results):
117117
# Test merging result shapes (duplicate obj, then compute average):
118118
for panel in (win.signalpanel, win.imagepanel):
119-
panel.objview.select_objects((2,))
119+
src_obj = panel[2]
120+
group_id = panel.objmodel.get_object_group_id(src_obj)
121+
panel.objview.select_objects([src_obj])
120122
panel.duplicate_object()
121-
panel.objview.select_objects((2, len(panel)))
123+
# The duplicate is the last object of the source object's group
124+
duplicate = panel.objmodel[
125+
panel.objmodel.get_group_object_ids(group_id)[-1]
126+
]
127+
panel.objview.select_objects([src_obj, duplicate])
122128
panel.processor.run_feature("average")
123-
last_obj = panel[len(panel)]
129+
# The average result is the last object of the source group
130+
last_obj = panel.objmodel[
131+
panel.objmodel.get_group_object_ids(group_id)[-1]
132+
]
124133
if keep_results:
125134
__check_geometry_results_merge(panel[2], last_obj)
126135
if panel is win.imagepanel:

0 commit comments

Comments
 (0)