From 84e1fa3800b658b0a02033ddfeba642a569a2968 Mon Sep 17 00:00:00 2001 From: Marc Mutz Date: Wed, 14 Jul 2021 22:52:08 +0200 Subject: QCryptographicHash test: remove the moreThan4GiBOfData(SHA512) check from the CI MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit It timed out the other day on the CI. Even the SHA-1 check took 137s (with a 300s timeout). Skip the SHA-512 test on the CI, but keep it for manual runs. Change-Id: I49792ac9bcab6512e1803f66cd986b2830e634c0 Reviewed-by: Mårten Nordheim (cherry picked from commit c6e092a5f84b7fc72556897b50c7697152967925) Reviewed-by: Edward Welbourne --- .../corelib/tools/qcryptographichash/tst_qcryptographichash.cpp | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'tests') diff --git a/tests/auto/corelib/tools/qcryptographichash/tst_qcryptographichash.cpp b/tests/auto/corelib/tools/qcryptographichash/tst_qcryptographichash.cpp index 92a2c3516c..b7982bc397 100644 --- a/tests/auto/corelib/tools/qcryptographichash/tst_qcryptographichash.cpp +++ b/tests/auto/corelib/tools/qcryptographichash/tst_qcryptographichash.cpp @@ -453,8 +453,11 @@ void tst_QCryptographicHash::moreThan4GiBOfData_data() row(QCryptographicHash::Md4); row(QCryptographicHash::Md5); row(QCryptographicHash::Sha1); - // this is already significantly slower, but important (O(min) - row(QCryptographicHash::Sha512); + if (!qgetenv("QTEST_ENVIRONMENT").split(' ').contains("ci")) { + // This is important but so slow (O(minute)) that, on CI, it tends to time out. + // Retain it for manual runs, all the same, as most dev machines will be fast enough. + row(QCryptographicHash::Sha512); + } // the rest is just too slow #else QSKIP("This test is 64-bit only."); -- cgit v1.2.3