Add test for pressing enter in a treeview in a dialog

I don't care if the dialog closes or stays open, but if it stays
open then the focus should stay in the dialog.

Pressing enter poses a problem with chrome during cypress tests
with CDP where the form submission default isn't blocked as it
is normally. So added a bodge around for that so the issue can
be property tested.

Signed-off-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Change-Id: I5e241ea91f7467643b4a9c63eb1ea045df8527fa

Author: caolanm

cypress_test/integration_tests/common/helper.js

@@ -1338,6 +1338,36 @@ function waitForMapState(command, expectedValue) {
 	cy.log('<< waitForMapState - end');
 }
 
+// cy.realPress('Enter') via CDP bypasses preventDefault on keydown,
+// which causes implicit form submission and spurious button clicks
+// that don't happen with real keyboard input. This helper blocks
+// those side effects for a single keypress.
+function realPressInDialog(key) {
+	cy.cGet('.jsdialog-window form').then($form => {
+		// Block implicit form submission for this keypress
+		function blockSubmit(e) {
+			e.preventDefault();
+			e.stopImmediatePropagation();
+		}
+		$form[0].addEventListener('submit', blockSubmit, true);
+
+		// Block the close button from being clicked
+		var closeBtn = $form[0].querySelector('.ui-dialog-titlebar-close');
+		var origOnclick = closeBtn ? closeBtn.onclick : null;
+		if (closeBtn) {
+			closeBtn.onclick = function(e) {
+				e.preventDefault();
+				e.stopImmediatePropagation();
+			};
+		}
+
+		cy.realPress(key).then(() => {
+			$form[0].removeEventListener('submit', blockSubmit, true);
+			if (closeBtn) closeBtn.onclick = origOnclick;
+		});
+	});
+}
+
 module.exports.setupDocument = setupDocument;
 module.exports.loadDocument = loadDocument;
 module.exports.setupAndLoadDocument = setupAndLoadDocument;
@@ -1395,6 +1425,7 @@ module.exports.waitForTimers = waitForTimers;
 module.exports.waitForMapState = waitForMapState;
 module.exports.maxScreenshotableViewportHeight = maxScreenshotableViewportHeight;
 module.exports.getMatchedCSSRules = getMatchedCSSRules;
+module.exports.realPressInDialog = realPressInDialog;
 
 // Find all author CSS rules that match an element, with the source
 // stylesheet filename and the full selector for each matching rule.

cypress_test/integration_tests/desktop/writer/a11y_dialog_spec.js

@@ -187,6 +187,54 @@ describe(['tagdesktop'], 'Accessibility Writer Dialog Tests', { testIsolation: f
         });
     });
 
+    it('Treeview Enter key keeps focus in dialog', function () {
+        cy.then(function () {
+            win.app.map.sendUnoCommand('.uno:ChapterNumberingDialog');
+        });
+
+        a11yHelper.getActiveDialog(1)
+            .then(() => helper.processToIdle(win))
+            .then(() => {
+                cy.cGet('#level .ui-treeview-entry:nth-child(1)').click();
+                return helper.processToIdle(win);
+            })
+            .then(() => {
+                cy.cGet('#level .ui-treeview-entry:nth-child(1)').should('have.class', 'selected');
+                cy.cGet('#level .ui-treeview-entry:nth-child(1)').focus();
+
+                // Move to second entry
+                cy.realPress('ArrowDown');
+                return helper.processToIdle(win);
+            })
+            .then(() => {
+                cy.cGet('#level .ui-treeview-entry:nth-child(2)').should('have.class', 'selected');
+                cy.cGet('#level .ui-treeview-entry:nth-child(2)').should('have.focus');
+
+                // Press Enter on the focused entry
+                helper.realPressInDialog('Enter');
+                return helper.processToIdle(win);
+            })
+            .then(() => {
+                // Either dialog should still exist with focus inside it,
+                // or dialog should have closed (OK activated)
+                cy.cGet('body').then($body => {
+                    const dialogExists = $body.find('.ui-dialog[role="dialog"]').length > 0;
+                    if (dialogExists) {
+                        // Dialog still open - focus must be inside it
+                        cy.cGet('.ui-dialog[role="dialog"]').then($dlg => {
+                            const activeEl = win.document.activeElement;
+                            const focusInDialog = $dlg[0].contains(activeEl);
+                            const focusDesc = activeEl.tagName + '#' + activeEl.id + '.' + activeEl.className;
+                            expect(focusInDialog, 'focus should stay in dialog, but is on: ' + focusDesc).to.be.true;
+                        });
+                        cy.cGet('#cancel-button').click();
+                    } else {
+                        cy.log('Dialog closed after Enter');
+                    }
+                });
+            });
+    });
+
     it('Treeview arrow key moves focus and selection together', function () {
         cy.then(function () {
             win.app.map.sendUnoCommand('.uno:ChapterNumberingDialog');