cypress_test/integration_tests/mobile/impress/apply_font_spec.js |   14 
+++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

New commits:
commit eba17c5f9c198748a8dc1dc07ead301a66b0b43a
Author:     Tamás Zolnai <tamas.zol...@collabora.com>
AuthorDate: Thu Jun 4 10:49:45 2020 +0200
Commit:     Tamás Zolnai <tamas.zol...@collabora.com>
CommitDate: Thu Jun 4 11:29:10 2020 +0200

    cypress: finish test for changing text color in Impress (mobile)
    
    Now this feature is fixed.
    
    Change-Id: I274e63f328e626a0e7c9ed262ad7baf651860b90
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/95488
    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 b95eb0ed1..d24b48e75 100644
--- a/cypress_test/integration_tests/mobile/impress/apply_font_spec.js
+++ b/cypress_test/integration_tests/mobile/impress/apply_font_spec.js
@@ -224,7 +224,10 @@ describe('Apply font on text and on text shape.', 
function() {
                        .should('have.attr', 'font-size', '564px');
        });
 
-       it.skip('Apply text color on text shape.', function() {
+       it('Apply text color on text shape.', function() {
+               cy.get('.leaflet-pane.leaflet-overlay-pane g.Page .TextPosition 
tspan')
+                       .should('have.attr', 'fill', 'rgb(0,0,0)');
+
                openTextPropertiesPanel();
 
                cy.get('.leaflet-pane.leaflet-overlay-pane g.Page 
.TextParagraph')
@@ -237,7 +240,8 @@ describe('Apply font on text and on text shape.', 
function() {
 
                triggerNewSVG();
 
-               // TODO: text color is not applied on the shape
+               cy.get('.leaflet-pane.leaflet-overlay-pane g.Page .TextPosition 
tspan')
+                       .should('have.attr', 'fill', 'rgb(106,168,79)');
        });
 
        it.skip('Apply highlight on text shape.', function() {
@@ -475,6 +479,9 @@ describe('Apply font on text and on text shape.', 
function() {
        });
 
        it('Apply text color on selected text.', function() {
+               cy.get('.leaflet-pane.leaflet-overlay-pane g.Page .TextPosition 
tspan')
+                       .should('have.attr', 'fill', 'rgb(0,0,0)');
+
                selectTextOfShape();
 
                mobileHelper.openMobileWizard();
@@ -489,7 +496,8 @@ describe('Apply font on text and on text shape.', 
function() {
 
                triggerNewSVG();
 
-               // TODO: text color is not in the SVG
+               cy.get('.leaflet-pane.leaflet-overlay-pane g.Page .TextPosition 
tspan')
+                       .should('have.attr', 'fill', 'rgb(106,168,79)');
        });
 
        it('Apply highlight 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