diff options
| author | Assam Boudjelthia <assam.boudjelthia@qt.io> | 2021-06-01 18:59:15 +0300 |
|---|---|---|
| committer | Assam Boudjelthia <assam.boudjelthia@qt.io> | 2021-06-02 21:02:41 +0300 |
| commit | 01e35a2145ba8afff69e53ea967a15a103a1631b (patch) | |
| tree | 9420c125289b75cc8fc061e1c102304da5a22802 /coin | |
| parent | e6a2892772c9ea18a941625b2b8dec38e6774f82 (diff) | |
Increase the timeout for test targets
The Android test VMs take almost the same amount of time of that set to
the test timeout, this is causing it to be killed by COIN in some cases.
This is for example a stat from a successful test instance:
Total Test time (real) = 7110.20 sec
It's very close to 7200 which is the timeout value. The VM is taking
that long because the tests are only half built, the projects' shared
libs are built during the Build step of the integration, and the actual
apk (Gradle build) is done later when doing a ninja tst_name_check.
This latter behavior should be changed so that the apks are built before
going to testing.
So it makes sense for now to increase the timeout and avoid having to
restage multiple times.
Task-number: QTBUG-94127
Change-Id: I31783bcc1255c7821134e174ab6b0fb199000272
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Reviewed-by: Heikki Halmet <heikki.halmet@qt.io>
Diffstat (limited to 'coin')
| -rw-r--r-- | coin/instructions/cmake_run_ctest_ignore_exit_code.yaml | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/coin/instructions/cmake_run_ctest_ignore_exit_code.yaml b/coin/instructions/cmake_run_ctest_ignore_exit_code.yaml index d4f0c13e3f..b94717730b 100644 --- a/coin/instructions/cmake_run_ctest_ignore_exit_code.yaml +++ b/coin/instructions/cmake_run_ctest_ignore_exit_code.yaml @@ -14,7 +14,7 @@ instructions: command: "{{.Env.TESTS_ENV_PREFIX}} ctest {{.Env.CTEST_ARGS}}" executeCommandArgumentSplitingBehavior: SplitAfterVariableSubstitution ignoreExitCode: true - maxTimeInSeconds: 7200 + maxTimeInSeconds: 10800 maxTimeBetweenOutput: 900 userMessageOnFailure: > Failed to run tests. |