Details
-
Task
-
Resolution: Done
-
P2: Important
-
None
-
None
-
13
-
Qt6_Foundation_Sprint 27, Qt6_Foundation_Sprint 28
Description
- Currently most of the C++ unit tests use raw new/delete operations to create objects, which leads to memleaks if the tests fail. It makes debugging tests under ASAN or memcheck quite annoying. Makes sense to consistently use QScopedPointer or std::unique_ptr for such cases.
- Most of declarative unit tests use both QtPositioning and QtLocation. It might be possible to separate this tests, so that declarative part of QtPositioning could be tested without compiling QtLocation. If separation is not possible, it might make sense to create new unit tests for such cases.
- Some tests are added as qt_internal_add_executable instead of qt_internal_add_test. This prevents the test autodetection & execution with ctest.
Attachments
Issue Links
- is required for
-
QTBUG-88613
Migrate Qt Positioning to Qt 6
-
- Closed
-
| For Gerrit Dashboard: QTBUG-90836 | ||||||
|---|---|---|---|---|---|---|
| # | Subject | Branch | Project | Status | CR | V |
| 331785,12 | QtPositioning: refactor unit tests | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
| 332162,3 | QtPositioning: fix unit tests | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
| 334114,2 | QtPositioning: fix build for tests | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
| 334781,2 | QtPositioning: fix test dependencies | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
| 334824,2 | QtPositioning: use smart pointers in tests | dev | qt/qtlocation | Status: MERGED | +2 | 0 |
| 334832,3 | QtPositioning: unblacklist tests | dev | qt/qtlocation | Status: MERGED | +2 | 0 |