diff options
| author | Ryan Chu <ryan.chu@qt.io> | 2018-08-28 23:04:08 +0200 |
|---|---|---|
| committer | Tor Arne Vestbø <tor.arne.vestbo@qt.io> | 2018-08-30 04:06:51 +0000 |
| commit | 09f59271e677259e1e2ffd3b42fff107001f89a5 (patch) | |
| tree | 6ddd2e825468eb713977cf36349e6ea89e33e2fd /tests/auto | |
| parent | 384fc28b908c23dfbd1a955945f17178bba96ff4 (diff) | |
Postpone testing docker images and files until running 'make check'
To avoid bothering the user who just wants to build all the test cases
in qtbase. It is better to move the docker related testing to the check
target. Then, it won't break the build procedure and force users to
finish test server provisioning in advance.
Change-Id: I2c4212297342c3eec7bcbebff9f991b7346d9503
Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@qt.io>
(cherry picked from commit e798e8a03ba51fad795c3407dce2aec18027b5f1)
Diffstat (limited to 'tests/auto')
| -rw-r--r-- | tests/auto/testserver.pri | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/tests/auto/testserver.pri b/tests/auto/testserver.pri index 2adf85a044..b2f593235f 100644 --- a/tests/auto/testserver.pri +++ b/tests/auto/testserver.pri @@ -53,7 +53,6 @@ TESTSERVER_COMPOSE_FILE = $$dirname(_QMAKE_CONF_)/tests/testserver/docker-compose.yml TESTSERVER_VERSION = $$system(docker-compose --version) -TESTSERVER_IMAGES = $$system(docker images -aq "qt-test-server-*") equals(QMAKE_HOST.os, Windows)|isEmpty(TESTSERVER_VERSION) { # Make check with server "qt-test-server.qt-test-net" as a fallback @@ -66,11 +65,14 @@ equals(QMAKE_HOST.os, Windows)|isEmpty(TESTSERVER_VERSION) { # file which is mandatory for all docker-compose commands. You can get more # detail from the description of TESTSERVER_COMPOSE_FILE above. There is # also an example showing how to configure it manually. - isEmpty(TESTSERVER_COMPOSE_FILE): error("Project variable 'TESTSERVER_COMPOSE_FILE' is not set") + FILE_PRETEST_MSG = "Project variable 'TESTSERVER_COMPOSE_FILE' is not set" + testserver_pretest.commands = $(if $$TESTSERVER_COMPOSE_FILE,,$(error $$FILE_PRETEST_MSG)) # Before starting the test servers, it requires the user to run the setup # script (coin/provisioning/.../testserver/docker_testserver.sh) in advance. - isEmpty(TESTSERVER_IMAGES): error("Docker image qt-test-server-* not found") + IMAGE_PRETEST_CMD = docker images -aq "qt-test-server-*" + IMAGE_PRETEST_MSG = "Docker image qt-test-server-* not found" + testserver_pretest.commands += $(if $(shell $$IMAGE_PRETEST_CMD),,$(error $$IMAGE_PRETEST_MSG)) # The domain name is relevant to https keycert (qnetworkreply/crts/qt-test-net-cacert.pem). DNSDOMAIN = test-net.qt.local @@ -85,6 +87,9 @@ equals(QMAKE_HOST.os, Windows)|isEmpty(TESTSERVER_VERSION) { check.target = check_network testserver_test.target = check + # Pretesting test servers environment + testserver_test.depends = testserver_pretest + # Bring up test servers and make sure the services are ready. testserver_test.commands = $$TEST_ENV docker-compose -f $$TESTSERVER_COMPOSE_FILE up -d \ --force-recreate --timeout 1 $${QT_TEST_SERVER_LIST} && @@ -100,5 +105,5 @@ equals(QMAKE_HOST.os, Windows)|isEmpty(TESTSERVER_VERSION) { testserver_clean.commands = $$TEST_ENV docker-compose -f $$TESTSERVER_COMPOSE_FILE down \ --rmi all - QMAKE_EXTRA_TARGETS += testserver_test testserver_clean + QMAKE_EXTRA_TARGETS += testserver_pretest testserver_test testserver_clean } |