diff options
author | Tor Arne Vestbø <tor.arne.vestbo@qt.io> | 2021-08-06 13:22:06 +0200 |
---|---|---|
committer | Tor Arne Vestbø <tor.arne.vestbo@qt.io> | 2021-08-06 20:05:15 +0200 |
commit | eb4fb9fa32797d48d4779464483230ad92f349a6 (patch) | |
tree | d5cd1ac15d13b83a808f76d6a6feeadd9936d0d2 /src | |
parent | 84a93d94ac8dfa3c962c0dda0ab19bbd80ac50ed (diff) |
testlib: Don't report blacklisted tests as failures in JUnit reporter
We don't produce <failure> elements for them, so we shouldn't include
them in the total count of failures.
In the future we might produce <skipped> elements instead, but for now
remove the incorrect increment.
Pick-to: 6.2
Change-Id: Ia89a5cdaf79f6aa2f92be53180112c40ad6da2c6
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Reviewed-by: Mårten Nordheim <marten.nordheim@qt.io>
Diffstat (limited to 'src')
-rw-r--r-- | src/testlib/qjunittestlogger.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/testlib/qjunittestlogger.cpp b/src/testlib/qjunittestlogger.cpp index 4d8ed2ae59..1e375bb33f 100644 --- a/src/testlib/qjunittestlogger.cpp +++ b/src/testlib/qjunittestlogger.cpp @@ -240,14 +240,12 @@ void QJUnitTestLogger::addIncident(IncidentTypes type, const char *description, typeBuf = "bpass"; break; case QAbstractTestLogger::BlacklistedFail: - ++failureCounter; typeBuf = "bfail"; break; case QAbstractTestLogger::BlacklistedXPass: typeBuf = "bxpass"; break; case QAbstractTestLogger::BlacklistedXFail: - ++failureCounter; typeBuf = "bxfail"; break; default: |