cypress_test/Makefile.am |   13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

New commits:
commit f2738885986ce0dcc3b15bd55e742e3b14a03a7b
Author:     Tamás Zolnai <tamas.zol...@collabora.com>
AuthorDate: Wed Aug 5 04:37:07 2020 +0200
Commit:     Tamás Zolnai <tamas.zol...@collabora.com>
CommitDate: Wed Aug 5 13:48:16 2020 +0200

    cypress: update run-cov command to run multi-user tests too.
    
    Change-Id: If13dea05f5a506331f9f76177655ed65716d391e
    Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100165
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com>
    Reviewed-by: Tamás Zolnai <tamas.zol...@collabora.com>

diff --git a/cypress_test/Makefile.am b/cypress_test/Makefile.am
index ef91b4134..64dbf40a2 100644
--- a/cypress_test/Makefile.am
+++ b/cypress_test/Makefile.am
@@ -163,7 +163,10 @@ run-mobile: @JAILS_PATH@ $(NODE_BINS)
                        || true
        @$(KILL_COMMAND) || true
 
-run-cov: @JAILS_PATH@ $(NODE_BINS)
+run-cov: do-run-cov
+       $(if $(wildcard $(ERROR_LOG)),@cat $(ERROR_LOG))
+
+do-run-cov: @JAILS_PATH@ $(NODE_BINS)
        @echo
        @echo "Setup coverage tools..."
        @echo
@@ -176,9 +179,11 @@ run-cov: @JAILS_PATH@ $(NODE_BINS)
        @echo
        @echo "Run all tests..."
        @echo
+       @rm -f $(ERROR_LOG)
        $(call start_loolwsd)
        $(call run_desktop_tests,,,COVERAGE_RUN="1")
        $(call run_mobile_tests,,,COVERAGE_RUN="1")
+       $(call run_all_multiuser_tests,COVERAGE_RUN="1")
        @$(KILL_COMMAND) || true
 
 @JAILS_PATH@:
@@ -306,7 +311,7 @@ define run_mobile_tests
 endef
 
 define run_all_multiuser_tests
-       $(foreach test,$(MULTIUSER_TESTS),$(call run_multiuser_test,$(test)))
+       $(foreach test,$(MULTIUSER_TESTS),$(call 
run_multiuser_test,$(test),$(1)))
 endef
 
 define run_multiuser_test
@@ -319,14 +324,14 @@ define run_multiuser_test
        @$(PARALLEL_SCRIPT) \
                --browser $(CHROME) \
                --config $(MULTIUSER_CONFIG) \
-               --env $(MULTIUSER_ENV) \
+               --env $(MULTIUSER_ENV)$(if $(2),$(COMMA)$(2)) \
                --spec $(USER1_SPEC) \
                --type multi-user \
                --log-file $(USER1_LOG) & \
        $(PARALLEL_SCRIPT) \
                --browser $(CHROME) \
                --config $(MULTIUSER_CONFIG) \
-               --env $(MULTIUSER_ENV) \
+               --env $(MULTIUSER_ENV)$(if $(2),$(COMMA)$(2)) \
                --spec $(USER2_SPEC) \
                --type multi-user \
                --log-file $(USER2_LOG) && \
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to