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

Tamás Zolnai (via logerrit) logerrit at kemper.freedesktop.org
Thu Jul 30 13:26:25 UTC 2020


 cypress_test/Makefile.am                                                   |    9 +++++++--
 cypress_test/integration_tests/multiuser/simultaneous_typing_user1_spec.js |    2 +-
 cypress_test/integration_tests/multiuser/simultaneous_typing_user2_spec.js |    2 +-
 3 files changed, 9 insertions(+), 4 deletions(-)

New commits:
commit 5b106d9edd5608bd3c7a3a95c44d4fc03b0c71b6
Author:     Tamás Zolnai <tamas.zolnai at collabora.com>
AuthorDate: Thu Jul 30 11:46:38 2020 +0200
Commit:     Tamás Zolnai <tamas.zolnai at collabora.com>
CommitDate: Thu Jul 30 15:26:19 2020 +0200

    cypress: fix-up check-multi command.
    
    We need to introduce separate rules to dump
    the log and to dump out the error message.
    Otherwise GNU make evaluation breaks the workflow.
    It evaluates $(wildcard $(ERROR_LOG)) before the
    test were actually executed. So first run the tests,
    then dump the error output and then go into error state.
    
    Change-Id: I0f8e5de6800f0691cca14019819972437a741382
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/99781
    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/Makefile.am b/cypress_test/Makefile.am
index a1b3acfcf..1908215b0 100644
--- a/cypress_test/Makefile.am
+++ b/cypress_test/Makefile.am
@@ -125,7 +125,13 @@ check-mobile: @JAILS_PATH@ $(NODE_BINS)
 	$(call run_mobile_tests,$(spec))
 	@$(KILL_COMMAND) || true
 
-check-multi: @JAILS_PATH@ $(NODE_BINS)
+check-multi: do-multi-check-log
+	$(if $(wildcard $(ERROR_LOG)),$(error CypressError: some tests failed!))
+
+do-multi-check-log: do-multi-check
+	$(if $(wildcard $(ERROR_LOG)), at cat $(ERROR_LOG))
+
+do-multi-check: @JAILS_PATH@ $(NODE_BINS)
 	@rm -f $(ERROR_LOG)
 	$(call run_JS_error_check)
 	$(call start_loolwsd)
@@ -133,7 +139,6 @@ check-multi: @JAILS_PATH@ $(NODE_BINS)
 		$(call run_multiuser_test,$(spec)), \
 		$(call run_all_multiuser_tests))
 	@$(KILL_COMMAND) || true
-	$(if $(wildcard $(ERROR_LOG)), at cat $(ERROR_LOG))
 
 run-desktop: @JAILS_PATH@ $(NODE_BINS)
 	$(call run_JS_error_check)
commit d43c97a9efe0c761ae06026027ca8452b1e2ff85
Author:     Tamás Zolnai <tamas.zolnai at collabora.com>
AuthorDate: Thu Jul 30 10:59:56 2020 +0200
Commit:     Tamás Zolnai <tamas.zolnai at collabora.com>
CommitDate: Thu Jul 30 15:26:05 2020 +0200

    cypress: make simultaneous typing test more stable.
    
    Sometimes this text contains an additional end line character.
    
    Change-Id: Id58eab9ea3b082781cdd8ec341fa2b058ba3b17e
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/99770
    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/multiuser/simultaneous_typing_user1_spec.js b/cypress_test/integration_tests/multiuser/simultaneous_typing_user1_spec.js
index e618512dc..293617bf5 100644
--- a/cypress_test/integration_tests/multiuser/simultaneous_typing_user1_spec.js
+++ b/cypress_test/integration_tests/multiuser/simultaneous_typing_user1_spec.js
@@ -32,7 +32,7 @@ describe('Simultaneous typing: user-1.', function() {
 		helper.selectAllText();
 
 		cy.get('#copy-paste-container p')
-			.should('have.text', text);
+			.should('contain.text', text);
 
 		// Change paragraph alignment to trigger user-2 actions
 		cy.get('textarea.clipboard')
diff --git a/cypress_test/integration_tests/multiuser/simultaneous_typing_user2_spec.js b/cypress_test/integration_tests/multiuser/simultaneous_typing_user2_spec.js
index 5e4eea2ee..dc4873d56 100644
--- a/cypress_test/integration_tests/multiuser/simultaneous_typing_user2_spec.js
+++ b/cypress_test/integration_tests/multiuser/simultaneous_typing_user2_spec.js
@@ -32,7 +32,7 @@ describe('Simultaneous typing: user-2.', function() {
 		helper.selectAllText();
 
 		cy.get('#copy-paste-container p')
-			.should('have.text', text);
+			.should('contain.text', text);
 
 		// user-1 changes the paragraph alignment after finished
 		cy.get('#tb_editbar_item_centerpara .w2ui-button')


More information about the Libreoffice-commits mailing list