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.zol...@collabora.com>
AuthorDate: Fri May 29 11:43:49 2020 +0200
Commit:     Tamás Zolnai <tamas.zol...@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 <jenkinscollaboraoff...@gmail.com>
    Reviewed-by: Tamás Zolnai <tamas.zol...@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();
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to