cypress_test/integration_tests/mobile/writer/table_properties_spec.js | 41 ++++------
1 file changed, 20 insertions(+), 21 deletions(-)
New commits:
commit 7de5068426628df13d5b7d033a6d014664c0fa5a
Author: Tamás Zolnai <tamas.zolnai at collabora.com>
AuthorDate: Tue Feb 18 16:25:22 2020 +0100
Commit: Tamás Zolnai <tamas.zolnai at collabora.com>
CommitDate: Tue Feb 18 18:27:32 2020 +0100
cypress: mobile: Update table reated tests.
There is no focus any more after selecting the
table, so we need to use a different method for
moving the cursor in the first cell.
Change-Id: I9d03337d7cb710224a892472f0989896d2e03024
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/88971
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai at collabora.com>
diff --git a/cypress_test/integration_tests/mobile/writer/table_properties_spec.js b/cypress_test/integration_tests/mobile/writer/table_properties_spec.js
index 249dc413d..fed1ff70c 100644
--- a/cypress_test/integration_tests/mobile/writer/table_properties_spec.js
+++ b/cypress_test/integration_tests/mobile/writer/table_properties_spec.js
@@ -22,6 +22,19 @@ describe('Change table properties / layout via mobile wizard.', function() {
.should('not.have.class', 'disabled');
}
+ function moveCursorToFirstCell() {
+ helper.selectAllMobile();
+
+ cy.get('.blinking-cursor')
+ .then(function(cursor) {
+ expect(cursor).to.have.lengthOf(1) ;
+ var posX = cursor[0].getBoundingClientRect().right + 10;
+ var posY = cursor[0].getBoundingClientRect().top + 10;
+ cy.get('body')
+ .click(posX, posY);
+ });
+ }
+
it('Insert row before.', function() {
// TODO: Select all does not work with core/master
// Table panel layout is also broken
@@ -295,9 +308,7 @@ describe('Change table properties / layout via mobile wizard.', function() {
before('table.odt');
- // Select All + left arrow -> we have the cursor in the table
- helper.selectAllMobile();
- cy.get('body').type('{leftarrow}');
+ moveCursorToFirstCell();
cy.get('body').type('{shift}{downarrow}{rightarrow}');
@@ -408,9 +419,7 @@ describe('Change table properties / layout via mobile wizard.', function() {
before('table_with_text.odt');
- // Select All + left arrow -> we have the cursor in the table
- helper.selectAllMobile();
- cy.get('body').type('{leftarrow}');
+ moveCursorToFirstCell();
cy.get('body').type('{leftarrow}{shift}{downarrow}{downarrow}{downarrow}{rightarrow}');
@@ -444,9 +453,7 @@ describe('Change table properties / layout via mobile wizard.', function() {
before('table_with_text.odt');
- // Select All + left arrow -> we have the cursor in the table
- helper.selectAllMobile();
- cy.get('body').type('{leftarrow}');
+ moveCursorToFirstCell();
cy.get('body').type('{leftarrow}{shift}{downarrow}{downarrow}{downarrow}{rightarrow}');
@@ -490,9 +497,7 @@ describe('Change table properties / layout via mobile wizard.', function() {
before('table_with_text.odt');
- // Select All + left arrow -> we have the cursor in the table
- helper.selectAllMobile();
- cy.get('body').type('{leftarrow}');
+ moveCursorToFirstCell();
cy.get('body').type('{leftarrow}{shift}{downarrow}{downarrow}{downarrow}{rightarrow}');
@@ -536,9 +541,7 @@ describe('Change table properties / layout via mobile wizard.', function() {
before('table_with_text.odt');
- // Select All + left arrow -> we have the cursor in the table
- helper.selectAllMobile();
- cy.get('body').type('{leftarrow}');
+ moveCursorToFirstCell();
cy.get('body').type('{leftarrow}{shift}{downarrow}{downarrow}{downarrow}{rightarrow}');
@@ -577,9 +580,7 @@ describe('Change table properties / layout via mobile wizard.', function() {
before('table_with_text.odt');
- // Select All + left arrow -> we have the cursor in the table
- helper.selectAllMobile();
- cy.get('body').type('{leftarrow}');
+ moveCursorToFirstCell();
cy.get('body').type('{leftarrow}{shift}{downarrow}{downarrow}{downarrow}{rightarrow}');
@@ -621,9 +622,7 @@ describe('Change table properties / layout via mobile wizard.', function() {
before('table_with_text.odt');
- // Select All + left arrow -> we have the cursor in the table
- helper.selectAllMobile();
- cy.get('body').type('{leftarrow}');
+ moveCursorToFirstCell();
cy.get('body').type('{leftarrow}{shift}{downarrow}{downarrow}{downarrow}{rightarrow}');