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

[Tamás Zolnai (via logerrit)]

 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.zolnai at collabora.com>
AuthorDate: Thu Jun 4 10:49:45 2020 +0200
Commit:     Tamás Zolnai <tamas.zolnai at 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 <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 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() {