Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-94476

Add varargs variants of QFAIL(), QSKIP(), QVERIFY2() that take printf-type format strings and parameters for them

    XMLWordPrintable

Details

    • Suggestion
    • Resolution: Unresolved
    • P3: Somewhat important
    • None
    • None
    • Testing: qtestlib
    • None
    • All
    • 8
    • Foundation PM Prioritized

    Description

      At present, if one wants to embed any relevant data in the messages these macros report, it has to be done by qPrintable(QString("...").arg(...)), which is unwieldy, ugly and expensive.
      Now that macro-varargs is a thing, there is nothing to keep us from supporting printf()-style formatting directly in such strings, just the same as we can in qDebug("...", ...) messages.
      Let's not name these by suffixing with numbers; I suggest QARG_FAIL(), QARG_SKIP() and QARG_VERIFY().

      Attachments

        Issue Links

          relates to

          Suggestion - Public suggestions QTBUG-66320 Provide a way to conveniently use test helper functions with qtestlib

          • P3: Somewhat important - Should be fixed, but doesn't affect the release in any way.
          • Open

          Task - A task that needs to be done. QTBUG-96295 Replace blacklisting of tests with QKNOWN_FLAKY() macro

          • P3: Somewhat important - Should be fixed, but doesn't affect the release in any way.
          • Reported

          Task - A task that needs to be done. QTBUG-101725 Port QSKIP/QFAIL/QVERIFY2/etc to QAnyStringView descriptions

          • P3: Somewhat important - Should be fixed, but doesn't affect the release in any way.
          • Reported

          Activity

            People

              cnn Qt Core & Network
              Eddy Edward Welbourne
              Vladimir Minenko Vladimir Minenko
              Alex Blasche Alex Blasche
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated: