test: add tests for registering custom editor values while tabbing (#7753) (#7784)

vaadin-bot · ugur-vaadin · DiegoCardoso · web-flow · commit 64debe67bd6f · 2025-08-01T13:31:40.000+02:00
* fix: use new custom editor if editors are updated

* chore: fix formatting

---------

Co-authored-by: Ugur Saglam &lt;106508695+ugur-vaadin@users.noreply.github.com&gt;
Co-authored-by: Diego Cardoso &lt;diego@vaadin.com&gt;
diff --git a/vaadin-spreadsheet-flow-parent/vaadin-spreadsheet-flow-integration-tests/src/main/java/com/vaadin/flow/component/spreadsheet/tests/fixtures/AdjacentCustomEditorsFixture.java b/vaadin-spreadsheet-flow-parent/vaadin-spreadsheet-flow-integration-tests/src/main/java/com/vaadin/flow/component/spreadsheet/tests/fixtures/AdjacentCustomEditorsFixture.java
@@ -0,0 +1,62 @@
+/**
+ * Copyright 2000-2025 Vaadin Ltd.
+ *
+ * This program is available under Vaadin Commercial License and Service Terms.
+ *
+ * See {@literal <https://vaadin.com/commercial-license-and-service-terms>} for the full
+ * license.
+ */
+package com.vaadin.flow.component.spreadsheet.tests.fixtures;
+
+import org.apache.poi.ss.usermodel.Cell;
+import org.apache.poi.ss.usermodel.Sheet;
+
+import com.vaadin.flow.component.Component;
+import com.vaadin.flow.component.HasValue;
+import com.vaadin.flow.component.spreadsheet.Spreadsheet;
+import com.vaadin.flow.component.spreadsheet.SpreadsheetComponentFactory;
+import com.vaadin.flow.component.textfield.TextField;
+
+public class AdjacentCustomEditorsFixture implements SpreadsheetFixture {
+
+    @Override
+    public void loadFixture(final Spreadsheet spreadsheet) {
+        spreadsheet.setShowCustomEditorOnFocus(true);
+        spreadsheet.setSpreadsheetComponentFactory(new CustomEditorFactory());
+    }
+
+    private static class CustomEditorFactory
+            implements SpreadsheetComponentFactory {
+
+        @Override
+        public Component getCustomComponentForCell(Cell cell, int rowIndex,
+                int columnIndex, Spreadsheet spreadsheet, Sheet sheet) {
+            return null;
+        }
+
+        @Override
+        public Component getCustomEditorForCell(Cell cell, final int rowIndex,
+                final int columnIndex, final Spreadsheet spreadsheet,
+                Sheet sheet) {
+            if (rowIndex != 1 || columnIndex < 1 || columnIndex > 5) {
+                return null;
+            }
+            var textField = new TextField();
+            textField.addValueChangeListener(
+                    e -> spreadsheet.refreshCells(spreadsheet
+                            .createCell(rowIndex, columnIndex, e.getValue())));
+            return textField;
+        }
+
+        @Override
+        public void onCustomEditorDisplayed(Cell cell, int rowIndex,
+                int columnIndex, Spreadsheet spreadsheet, Sheet sheet,
+                Component customEditor) {
+            if (cell == null) {
+                return;
+            }
+            String cellValue = cell.getStringCellValue();
+            ((HasValue) customEditor).setValue(cellValue);
+        }
+    }
+}
diff --git a/vaadin-spreadsheet-flow-parent/vaadin-spreadsheet-flow-integration-tests/src/main/java/com/vaadin/flow/component/spreadsheet/tests/fixtures/TestFixtures.java b/vaadin-spreadsheet-flow-parent/vaadin-spreadsheet-flow-integration-tests/src/main/java/com/vaadin/flow/component/spreadsheet/tests/fixtures/TestFixtures.java
@@ -36,6 +36,7 @@ public enum TestFixtures {
     Rename(RenameFixture.class),
     CreateSheet(SheetsFixture.class),
     CustomEditor(SimpleCustomEditorFixture.class),
+    AdjacentCustomEditors(AdjacentCustomEditorsFixture.class),
     CustomEditorShared(CustomEditorSharedFixture.class),
     CustomEditorRow(CustomEditorRowFixture.class),
     Styles(StylesFixture.class),
diff --git a/vaadin-spreadsheet-flow-parent/vaadin-spreadsheet-flow-integration-tests/src/test/java/com/vaadin/flow/component/spreadsheet/test/CustomEditorIT.java b/vaadin-spreadsheet-flow-parent/vaadin-spreadsheet-flow-integration-tests/src/test/java/com/vaadin/flow/component/spreadsheet/test/CustomEditorIT.java
@@ -10,6 +10,7 @@
 
 import java.time.LocalDate;
 import java.time.format.DateTimeFormatter;
+import java.util.List;
 import java.util.Optional;
 
 import org.junit.Assert;
@@ -355,6 +356,20 @@ public void customEditorIsVisible_sheetIsChanged_editorsRemoved() {
     }
 
     @Test
+    public void adjacentCustomEditors_showOnFocus_navigateWithTabAndModify_valuesUpdated() {
+        getSpreadsheet().addSheet();
+        loadTestFixture(TestFixtures.AdjacentCustomEditors);
+        var cellsToModify = List.of("B2", "C2", "D2", "E2", "F2");
+        var valueToSet = "a";
+        selectCell("A2");
+        cellsToModify.forEach(
+                address -> moveToNextCellAndAssertEditorInCellIsFocusedWithKeyPress(
+                        address, valueToSet));
+        getActiveElement().sendKeys(Keys.TAB);
+        cellsToModify.forEach(address -> Assert.assertEquals(valueToSet,
+                getCellValue(address)));
+    }
+
     public void customEditorShared_persistsValuesCorrectly() {
         createNewSpreadsheet();
         loadTestFixture(TestFixtures.CustomEditorShared);
