diff options
author | Jason McDonald <jason.mcdonald@nokia.com> | 2012-01-17 15:06:23 +1000 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2012-01-18 02:10:09 +0100 |
commit | 8fbad679e921b2b394c46fbf2c602216cdb2f209 (patch) | |
tree | 5b0ebb39260a34c42b32e234477cb14c3d6ed63c /src/testlib/qtestcase.h | |
parent | 81e55fede783b3747998c98f2471d773805b9246 (diff) |
Change QSKIP to fail for outdated tests.
Be more insistent that tests using the old two-argument version of QSKIP
should be updated. After a grace period the warning will be removed and
incorrect usage of QSKIP will revert to a compilation failure.
Change-Id: Ifa19b856d9f45738bd9d790bb65a8741f965d0f4
Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>
Diffstat (limited to 'src/testlib/qtestcase.h')
-rw-r--r-- | src/testlib/qtestcase.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/testlib/qtestcase.h b/src/testlib/qtestcase.h index 4537e76920..2c299953c4 100644 --- a/src/testlib/qtestcase.h +++ b/src/testlib/qtestcase.h @@ -128,8 +128,8 @@ do {\ #define QSKIP(statement, ...) \ do {\ if (strcmp(#__VA_ARGS__, "") != 0)\ - QTest::qWarn("The two argument version of QSKIP is deprecated and will be removed soon. "\ - "Please update this test by removing the second parameter.", __FILE__, __LINE__);\ + QTest::qFail("The two argument version of QSKIP is no longer available. "\ + "Please update this test by removing the second argument in each QSKIP.", __FILE__, __LINE__);\ QTest::qSkip(statement, __FILE__, __LINE__);\ return;\ } while (0) |