officejs_ui_test: use clickAndWait in xhtml_style

The previous pattern of using click and then pause is fragile and seems to be the cause for test failures.

officejs_ui_test: use clickAndWait in xhtml_style
The previous pattern of using click and then pause is fragile and seems to be the cause for test failures.
ebf18760 · Jérome Perrin · 3b91fe33 · ebf18760 · ebf18760 · ebf18760
Commit ebf18760 authored Dec 26, 2023 by Jérome Perrin
15 changed files
--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_codemirror_zuite/testOfficeJSCodeMirrorEditorClone.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_codemirror_zuite/testOfficeJSCodeMirrorEditorClone.zpt
@@ -454,20 +454,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>
@@ -494,20 +484,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_codemirror_zuite/testOfficeJSCodeMirrorErp5Synchronization.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_codemirror_zuite/testOfficeJSCodeMirrorErp5Synchronization.zpt
@@ -334,20 +334,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_imageeditor_zuite/testOfficeJSImageEditorClone.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_imageeditor_zuite/testOfficeJSImageEditorClone.zpt
@@ -454,20 +454,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>pause</td>
  <td>2000</td>
@@ -499,20 +489,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>pause</td>
  <td>2000</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_notebook_zuite/testOfficeJSNotebookClone.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_notebook_zuite/testOfficeJSNotebookClone.zpt
@@ -454,20 +454,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>
@@ -494,20 +484,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_notebook_zuite/testOfficeJSNotebookErp5Synchronization.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_notebook_zuite/testOfficeJSNotebookErp5Synchronization.zpt
@@ -367,20 +367,10 @@ For that I'm using this undocumented triggerOnKeyDown() method -->
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_ooo_presentation_zuite/testOnlyOfficePresentationClone.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_ooo_presentation_zuite/testOnlyOfficePresentationClone.zpt
@@ -457,20 +457,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>
@@ -497,20 +487,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_ooo_spreadsheet_zuite/testOnlyOfficeSpreadsheetClone.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_ooo_spreadsheet_zuite/testOnlyOfficeSpreadsheetClone.zpt
@@ -447,20 +447,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>
@@ -487,20 +477,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_ooo_text_zuite/testOnlyfficeTextClone.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_ooo_text_zuite/testOnlyfficeTextClone.zpt
@@ -457,20 +457,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>
@@ -497,20 +487,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_ooo_text_zuite/testOnlyfficeTextErp5Synchronization.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_ooo_text_zuite/testOnlyfficeTextErp5Synchronization.zpt
@@ -272,20 +272,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_pdfviewer_zuite/testOfficeJSPdfViewerClone.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_pdfviewer_zuite/testOfficeJSPdfViewerClone.zpt
@@ -454,20 +454,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>
@@ -494,20 +484,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_slideshow_editor_zuite/testOfficeJSUISlideshowEditorClone.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_slideshow_editor_zuite/testOfficeJSUISlideshowEditorClone.zpt
@@ -444,20 +444,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>
@@ -484,20 +474,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_smartassistant_zuite/testOfficeJSSmartAssistantErp5Synchronization.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_smartassistant_zuite/testOfficeJSSmartAssistantErp5Synchronization.zpt
@@ -159,20 +159,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>waitForElementPresent</td>
  <td>//div[@class="listbox-container"]</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_svgeditor_zuite/testOfficeJSsvgEditorClone.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_svgeditor_zuite/testOfficeJSsvgEditorClone.zpt
@@ -454,20 +454,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>
@@ -494,20 +484,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_texteditor_zuite/testOfficeJSTextEditorClone.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_texteditor_zuite/testOfficeJSTextEditorClone.zpt
@@ -455,20 +455,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>
@@ -495,20 +485,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>

--- a/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_texteditor_zuite/testOfficeJSTextEditorErp5Synchronization.zpt
+++ b/bt5/erp5_officejs_ui_test/PathTemplateItem/portal_tests/officejs_ui_texteditor_zuite/testOfficeJSTextEditorErp5Synchronization.zpt
@@ -380,20 +380,10 @@
  <td></td>
 </tr>
 <tr>
-  <td>click</td>
+  <td>clickAndWait</td>
  <td>//button[@title="Show All"]</td>
  <td></td>
 </tr>
-<tr>
-  <td>pause</td>
-  <td>1000</td>
-  <td></td>
-</tr>
-<tr>
-  <td>waitForElementPresent</td>
-  <td>//input[@name="listbox_title"]</td>
-  <td></td>
-</tr>
 <tr>
  <td>type</td>
  <td>name=listbox_title</td>