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

New commits:
commit 7705ece999c0be69448a12cd21d23bb8442bfd2e
Author:     Tamás Zolnai <tamas.zol...@collabora.com>
AuthorDate: Thu Jun 4 10:56:01 2020 +0200
Commit:     Tamás Zolnai <tamas.zol...@collabora.com>
CommitDate: Thu Jun 4 11:29:25 2020 +0200

    cypress: finish test for changing highlight color in Impress (mobile)
    
    Unfortunately, the core generated SVG does not
    have the text highlighting, so we can only check
    the state of the mobile wizard.
    
    Change-Id: I49a916113a986568c552917a5fe953165282de30
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/95489
    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 d24b48e75..5540067ad 100644
--- a/cypress_test/integration_tests/mobile/impress/apply_font_spec.js
+++ b/cypress_test/integration_tests/mobile/impress/apply_font_spec.js
@@ -244,7 +244,7 @@ describe('Apply font on text and on text shape.', 
function() {
                        .should('have.attr', 'fill', 'rgb(106,168,79)');
        });
 
-       it.skip('Apply highlight on text shape.', function() {
+       it('Apply highlight on text shape.', function() {
                openTextPropertiesPanel();
 
                cy.get('.leaflet-pane.leaflet-overlay-pane g.Page 
.TextParagraph')
@@ -255,9 +255,17 @@ describe('Apply font on text and on text shape.', 
function() {
 
                mobileHelper.selectFromColorPalette(1, 2, 2);
 
+               cy.get('#CharBackColor .color-sample-selected')
+                       .should('have.attr', 'style', 'background-color: 
rgb(204, 0, 0);');
+
                triggerNewSVG();
 
-               // TODO: highlight color is not applied on the shape
+               // TODO: highlight color is not in the SVG
+               // At least check the mobile wizard's state
+               openTextPropertiesPanel();
+
+               cy.get('#CharBackColor .color-sample-selected')
+                       .should('have.attr', 'style', 'background-color: 
rgb(204, 0, 0);');
        });
 
        it('Apply superscript on text shape.', function() {
@@ -505,17 +513,24 @@ describe('Apply font on text and on text shape.', 
function() {
 
                mobileHelper.openMobileWizard();
 
-               cy.get('.leaflet-pane.leaflet-overlay-pane g.Page 
.TextParagraph')
-                       .should('not.have.attr', 'font-color');
-
                cy.get('#CharBackColor')
                        .click();
 
                mobileHelper.selectFromColorPalette(1, 2, 2);
 
+               cy.get('#CharBackColor .color-sample-selected')
+                       .should('have.attr', 'style', 'background-color: 
rgb(204, 0, 0);');
+
                triggerNewSVG();
 
                // TODO: highlight color is not in the SVG
+               // At least check the mobile wizard's state
+               selectTextOfShape();
+
+               mobileHelper.openMobileWizard();
+
+               cy.get('#CharBackColor .color-sample-selected')
+                       .should('have.attr', 'style', 'background-color: 
rgb(204, 0, 0);');
        });
 
        it('Apply superscript on selected text.', function() {
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to