Use press() in rule_builder_column_values, check nested identifiers

Anchor on any rendered cell and press Home first so columns can be
read in any order. Assert outer/inner identifiers in the nested test.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

client/src/utils/navigation/navigation.yml

@@ -1749,6 +1749,8 @@ rule_builder:
     source: '.rule-source'
     table: '#rules-ag-grid .ag-root'
     table_row: '#rules-ag-grid .ag-row'
+    table_row_cell: '#rules-ag-grid .ag-row[row-index="${row_index}"] .ag-cell'
+    # row_index is 0-based, column_number is ag-grid's 1-based aria-colindex
     table_cell: '#rules-ag-grid .ag-row[row-index="${row_index}"] .ag-cell[aria-colindex="${column_number}"]'
     extension_select: '.rule-footer-extension-group .extension-select'
 

lib/galaxy/selenium/navigates_galaxy.py

@@ -1238,14 +1238,12 @@ class NavigatesGalaxy(HasDriverProxy[WaitType]):
     def rule_builder_column_values(self, column_index: int) -> list[str]:
         """Text of each rendered cell in a rule builder column, in row order.
 
-        The grid only renders cells in view, so this moves focus to the column to
-        scroll it in. Rows are not scrolled, so this suits small tables.
+        The grid only renders cells in view, so this moves keyboard focus to the
+        column to scroll it in. Rows are not scrolled, so this suits small tables.
         """
         rule_builder = self.components.rule_builder
-        # Focus without clicking - a wide first cell's center can lie outside the grid.
-        first_cell = rule_builder.table_cell(row_index=0, column_number=1).wait_for_visible()
-        self.execute_script("arguments[0].focus();", first_cell)
-        self.send_keys_to_page(Keys.ARROW_RIGHT * column_index)
+        rendered_cell = rule_builder.table_row_cell(row_index=0).wait_for_visible()
+        self.press(Key.HOME, *[Key.ARROW_RIGHT] * column_index, element=rendered_cell)
         row_count = len(rule_builder.table_row.all())
         return [
             rule_builder.table_cell(row_index=row_index, column_number=column_index + 1).wait_for_visible().text

lib/galaxy_test/selenium/test_workbook_mapping.py

@@ -38,11 +38,13 @@ class TestWorkbookMapping(SeleniumTestCase, UsesUploadActivity):
     def _file_names(self, column_index: int) -> list[str]:
         return [url.rsplit("/", 1)[-1] for url in self.rule_builder_column_values(column_index)]
 
+    def _assert_creating_datasets(self) -> None:
+        self.components.rule_builder.collection_name_input.assert_absent()
+
     def _assert_creating_collections(self) -> None:
         self.components.rule_builder.collection_name_input.wait_for_visible()
 
     def _assert_example_1_mapping(self) -> None:
-        self.components.rule_builder.collection_name_input.assert_absent()
         mapping = self._mapping(3)
         self._assert_mapping(mapping, 0, "name", [0])
         self._assert_mapping(mapping, 1, "url", [1])
@@ -52,6 +54,7 @@ class TestWorkbookMapping(SeleniumTestCase, UsesUploadActivity):
     def test_dataset_mapping_from_workbook(self):
         """Dataset name, URL and genome columns are recognized by their headers."""
         self.upload_context("rule").upload_workbook(self._workbook(1))
+        self._assert_creating_datasets()
         self._assert_example_1_mapping()
 
     @selenium_test
@@ -60,6 +63,7 @@ class TestWorkbookMapping(SeleniumTestCase, UsesUploadActivity):
         rule_import = self.upload_context("rule").creating("datasets").from_source("workbook")
         assert "type=datasets" in rule_import.workbook_download_url()
         rule_import.upload_workbook_from_card(self._workbook(1))
+        self._assert_creating_datasets()
         self._assert_example_1_mapping()
 
     @selenium_test
@@ -115,6 +119,15 @@ class TestWorkbookMapping(SeleniumTestCase, UsesUploadActivity):
         self._assert_mapping(mapping, 1, "list_identifiers", [1, 2])
         self._assert_mapping(mapping, 2, "paired_identifier", [3])
 
-        runs = ["SRR5680996", "SRR5680997", "SRR5681007", "SRR5681006", "SRR5680998", "SRR5681008"]
-        assert self._file_names(0) == [f"{r}_{i}.fastq.gz" for r in runs for i in "12"]
+        rows = [
+            ("SRR5680996", "H3K4me3", "rep1"),
+            ("SRR5680997", "H3K27me3", "rep2"),
+            ("SRR5681007", "H3K27me3", "rep1"),
+            ("SRR5681006", "H3K4me3", "rep2"),
+            ("SRR5680998", "CTCF", "rep1"),
+            ("SRR5681008", "CTCF", "rep2"),
+        ]
+        assert self._file_names(0) == [f"{run}_{i}.fastq.gz" for run, _, _ in rows for i in "12"]
+        assert self.rule_builder_column_values(1) == [outer for _, outer, _ in rows for _ in range(2)]
+        assert self.rule_builder_column_values(2) == [inner for _, _, inner in rows for _ in range(2)]
         assert self.rule_builder_column_values(3) == ["1", "2"] * 6