Writing good tests

From Qt Wiki
Revision as of 16:39, 4 December 2020 by Friedemannkleint (talk | contribs) (fix cleanup)
Jump to navigation Jump to search
The printable version is no longer supported and may have rendering errors. Please update your browser bookmarks and please use the default browser print function instead.

See also:

Aspects to consider

  • The tests are run on CI machines potentially under heavy load.
  • Tests might be run under various desktop systems.
  • The graphics setup is usually the GL enabling layer of some virtual machine.

Recommendations

General

  • Do not use hard-coded timeouts (e.g. qWait) to wait for some conditions to become true. Consider using QTRY_VERIFY and QTRY_COMPARE.
  • QVERIFY2 with an error message is preferable over a plain QVERIFY in order to obtain messages when something fails:
QVERIFY2(a < 2, (QByteArray::number(a) + QByteArrayLiteral(" is not less than 2"))
  • Do not re-use instances of the class under test in several tests. Test instances (for example widgets) should not be member variables of the tests, but preferably be instantiated on the stack to ensure proper cleanup even if a test fails, so that tests do not interfere with each other
  • Tests should ensure their resources are cleaned up even if a test fails (consider that a failed QCOMPARE/QVERIFY executes a return statement). Classes should be instantiated on the stack or use a QScopedPointer or be parented on a QObject whose deletion is guaranteed. It is recommended to check this in the slot cleanup():
void tst_QGraphicsProxyWidget::cleanup() // This will be called after every test function.
{
    QVERIFY(QApplication::topLevelWidgets().isEmpty());
}

Files, I/O resources

  • Tests should not create files in their build/source directories nor in common folders like home folders. Use QTemporaryDir and QTemporaryFile and ensure those folders and files are deleted after test execution (that is, all file handles are closed so that automatic deletion works). Always use QVERIFY(temporaryDir.isValid()).
  • Tests should not create file watchers on commonly used folders like home or temporary folders (remember that tests run in parallel). Use QTemporaryDir for this as well.

Widgets and Windows

  • If not required for testing purposes, use at most one top level window on the screen to prevent focus fights. If several windows are required, position them explicitly beside each other
  • Preferably center top level windows within QGuiApplication::primaryScreen()->availableGeometry(). Most importantly, do not position windows at 0, 0 since the CI uses Ubuntu's Unity as well, which has a taskbar on the left.
  • Top level windows with decoration should be at least 160×40 (else, a warning will appear on Windows 8).
  • Beware of interference from the current cursor position. If necessary, move the cursor away from the window under test using QCursor::setPos(). The bottom right corner of the available screen is considered a safe position. The top left corner should be avoided since it has been observed to trigger the "Present all windows" function of KDE's KWin (Ctrl-F9 or push mouse to top left, KDE 5.X).
  • If you must track mouse move events, consider replacing the use of QTest::mouseMove where the first parameter is a QWidget with the overload of QTest::mouseMove that takes a QWindow as parameter instead. For compatibility reasons the overload that takes a QWidget uses QCursor::setPos behind the scenes, which will result in an asynchronous turnaround to the window system server and also allows the window manager to influence the timing and placement. The overload that takes a QWindow however delivers a mouse move event instantly and synchronously without any window system interaction. A change towards the overload of QTest::mouseMove that takes a QWindow therefore often makes the test more reliable and allows replacing QTRY_VERIFY calls with QVERIFY or QCOMPARE. Note that for any QWidget you can get a reference to the underlying QWindow.
  • Note that more sophisticated use of QCursor-API needs to be conditioned on #ifndef QT_NO_CURSOR
  • If a widget/window needs to be visible on the screen or active for a test to succeed, use:
    QVERIFY(QTest::qWaitForWindowExposed(&view));
    QVERIFY(QTest::qWaitForWindowActive(&view));
  • Ensure widgets/windows are not leaked (also on failures) by instantiating them on the stack or using a QScopedPointer. A cleanup() slot can be used to verify this:
 void cleanup() { QVERIFY(QApplication::topLevelWidgets().isEmpty()); }

Practical hints

  • On Windows, a crashing test will not display a dialog prompting you to attach a debugger (to prevent the CI from getting stuck). This can be activated by passing the command line option -nocrashhandler.
  • It is possible to run a single test by passing its name on the command line (see Documentation). If you type a substring, the test will display the matches.
  • Qt Creator contains a Perl script scripts/test2tasks.pl that converts testlib text output into a Task file that is shown in the build issues pane and can be used to quickly navigate to test failures.

Hints for analyzing test flakyness (GUI tests)

  • Does the test create unrelated windows that overlap and interfere? For example, most existing widget tests still use a member variable test widget shared between the tests. If a single test then instantiates another widget, this can lead to focus issues. As stated above the member variable should be replaced by per-test widget instances. In some cases, windows created by skipped/failed tests leak.
  • Does the test create windows at random positions (notably on X11)? Such windows might end up in the taskbar area or interfere with notification windows of the OS. As stated above, windows should be centered.
  • Is the test being influenced by the mouse cursor position? (This is particularly an issue on Mac.) If so, the cursor should be moved to a well-defined position.