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

[Tamás Zolnai (via logerrit)]

 cypress_test/integration_tests/mobile/impress/apply_font_spec.js |   25 ++++++----
 1 file changed, 17 insertions(+), 8 deletions(-)

New commits:
commit 3b22b819659c2aaf38cbaecd48ab57335e21f3fe
Author:     Tamás Zolnai <tamas.zolnai at collabora.com>
AuthorDate: Fri May 29 11:43:49 2020 +0200
Commit:     Tamás Zolnai <tamas.zolnai at collabora.com>
CommitDate: Fri May 29 13:27:45 2020 +0200

    cypress: fix unstable clear formatting test.
    
    Change-Id: Id218dca4d7499e17b4b50c1b4422b12eaf96ac95
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/95102
    Tested-by: Jenkins
    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/impress/apply_font_spec.js b/cypress_test/integration_tests/mobile/impress/apply_font_spec.js
index 74d0f1611..18cfa750c 100644
--- a/cypress_test/integration_tests/mobile/impress/apply_font_spec.js
+++ b/cypress_test/integration_tests/mobile/impress/apply_font_spec.js
@@ -13,10 +13,7 @@ describe('Apply font on text shape.', function() {
 
 		selectTextShape();
 
-		mobileHelper.openMobileWizard();
-
-		cy.get('#TextPropertyPanel')
-			.click();
+		openTextPropertiesPanel();
 	});
 
 	afterEach(function() {
@@ -64,6 +61,16 @@ describe('Apply font on text shape.', function() {
 		selectTextShape();
 	}
 
+	function openTextPropertiesPanel() {
+		mobileHelper.openMobileWizard();
+
+		cy.get('#TextPropertyPanel')
+			.click();
+
+		cy.get('.ui-content.level-0.mobile-wizard')
+			.should('be.visible');
+	}
+
 	it('Apply bold.', function() {
 		cy.get('#Bold')
 			.click();
@@ -261,7 +268,7 @@ describe('Apply font on text shape.', function() {
 			.should('have.attr', 'font-size', '368px');
 	});
 
-	it.skip('Clear direct formatting.', function() {
+	it('Clear direct formatting.', function() {
 		// Change the font size first
 		cy.get('.leaflet-pane.leaflet-overlay-pane g.Page .TextParagraph')
 			.should('have.attr', 'font-size', '635px');
@@ -275,10 +282,12 @@ describe('Apply font on text shape.', function() {
 			.should('have.attr', 'font-size', '705px');
 
 		// Remove direct formatting
-		mobileHelper.openMobileWizard();
+		openTextPropertiesPanel();
 
-		cy.get('#TextPropertyPanel')
-			.click();
+		// Opening the mobile wizard for the second time
+		// triggeres two jsdialog message and so the mobile
+		// wizard is rendered twice.
+		cy.wait(500);
 
 		cy.get('#clearFormatting')
 			.click();