[Libreoffice-commits] online.git: cypress_test/integration_tests

[Tamás Zolnai (via logerrit)]

 cypress_test/integration_tests/mobile/writer/bottom_toolbar_spec.js |   12 ++++
 cypress_test/integration_tests/mobile/writer/writer_helper.js       |   30 +++++++++-
 2 files changed, 39 insertions(+), 3 deletions(-)

New commits:
commit fdbca4dcf11f38c6e4da65634f88cba727ec4e9d
Author:     Tamás Zolnai <tamas.zolnai at collabora.com>
AuthorDate: Wed Mar 18 11:34:06 2020 +0100
Commit:     Tamás Zolnai <tamas.zolnai at collabora.com>
CommitDate: Wed Mar 18 20:44:48 2020 +0100

    cypress: mobile: restore original selectAllMobile() in writer_helper.js.
    
    Using CTRL+A is simpler, but with using the Hamburger menu we can
    make sure that menu item is working. CTRL+A is not an actual
    feature on mobile. Also it's good that we use hamburger menu, so
    we can check interaction of hamburger menu and other wizards
    using the same bottom panel.
    
    Change-Id: I640e515070046ad18b00be69509ed895844962a0
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/90710
    Tested-by: Tamás Zolnai <tamas.zolnai at collabora.com>
    Reviewed-by: Tamás Zolnai <tamas.zolnai at collabora.com>

diff --git a/cypress_test/integration_tests/mobile/writer/bottom_toolbar_spec.js b/cypress_test/integration_tests/mobile/writer/bottom_toolbar_spec.js
index edc05ca95..fd23fd030 100644
--- a/cypress_test/integration_tests/mobile/writer/bottom_toolbar_spec.js
+++ b/cypress_test/integration_tests/mobile/writer/bottom_toolbar_spec.js
@@ -35,6 +35,10 @@ describe('Pushing bottom toolbar items.', function() {
 	});
 
 	it('Apply bold, check keyboard.', function() {
+		cy.get('#document-container')
+			.type('{downarrow}');
+		helper.selectAllText();
+
 		cy.get('#tb_editbar_item_bold div table')
 			.should('not.have.class', 'checked');
 
@@ -71,6 +75,10 @@ describe('Pushing bottom toolbar items.', function() {
 	});
 
 	it('Apply italic, check keyboard.', function() {
+		cy.get('#document-container')
+			.type('{downarrow}');
+		helper.selectAllText();
+
 		cy.get('#tb_editbar_item_italic div table')
 			.should('not.have.class', 'checked');
 
@@ -107,6 +115,10 @@ describe('Pushing bottom toolbar items.', function() {
 	});
 
 	it('Apply underline, check keyboard.', function() {
+		cy.get('#document-container')
+			.type('{downarrow}');
+		helper.selectAllText();
+
 		cy.get('#tb_editbar_item_underline div table')
 			.should('not.have.class', 'checked');
 
diff --git a/cypress_test/integration_tests/mobile/writer/writer_helper.js b/cypress_test/integration_tests/mobile/writer/writer_helper.js
index e9e9ef59b..0e2385a51 100644
--- a/cypress_test/integration_tests/mobile/writer/writer_helper.js
+++ b/cypress_test/integration_tests/mobile/writer/writer_helper.js
@@ -95,13 +95,37 @@ function clearMobileWizardState() {
 }
 
 function selectAllMobile() {
+	cy.log('Select all via hamburger menu - start.');
+
 	// Remove selection if exist
-	//FIXME: this also gives the focus to the Writer doc,
-	// which shouldn't be needed (i.e. should have focus already).
 	cy.get('#document-container')
 		.type('{downarrow}');
+	cy.get('.leaflet-marker-icon')
+		.should('not.exist');
+
+	// Open hamburger menu
+	cy.get('#toolbar-hamburger')
+		.click();
+	cy.get('#mobile-wizard')
+		.should('be.visible', {timeout : 10000});
+
+	// Open edit menu
+	cy.get('.ui-header.level-0 .menu-entry-with-icon')
+		.contains('Edit')
+		.click();
+
+	cy.get('.ui-header.level-1 .menu-entry-with-icon')
+		.should('be.visible')
+		.wait(100);
+
+	// Do the selection
+	cy.get('.ui-header.level-1 .menu-entry-with-icon')
+		.contains('Select All')
+		.click();
+	cy.get('.leaflet-marker-icon')
+		.should('exist');
 
-	helper.selectAllText();
+	cy.log('Select all via hamburger menu - end.');
 }
 
 module.exports.copyTextToClipboard = copyTextToClipboard;