Details
-
Bug
-
Resolution: Unresolved
-
P3: Somewhat important
-
5.0.0
-
None
Description
In tests/auto/network/socket/qtcpsocket/tst_qtcpsocket.cpp, the connectToMultiIP() test function is effectively disabled with:
QSKIP("TODO: setup DNS in the new network");
If this skip was added due to a known bug, it would be better to use QEXPECT_FAIL() here, so that when the bug is fixed the test clearly reports that it needs to be updated. (If this remains a QSKIP, there is no warning that the test is out-of-date and the bug could reappear without anybody being informed.) The QEXPECT_FAIL should refer to an appropriate bug report documenting the known bug.
If the skip was added to avoid a piece of code that crashes, the text of the skip message must be updated to refer to an appropriate bug report.