[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, 20 insertions(+), 5 deletions(-)

New commits:
commit 7705ece999c0be69448a12cd21d23bb8442bfd2e
Author:     Tamás Zolnai <tamas.zolnai at collabora.com>
AuthorDate: Thu Jun 4 10:56:01 2020 +0200
Commit:     Tamás Zolnai <tamas.zolnai at 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 <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 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() {