diff --git a/tests/units/presentation/querytreemodeltest.cpp b/tests/units/presentation/querytreemodeltest.cpp --- a/tests/units/presentation/querytreemodeltest.cpp +++ b/tests/units/presentation/querytreemodeltest.cpp @@ -24,7 +24,10 @@ #include +#include + #include +#include #include "utils/mockobject.h" @@ -827,10 +830,10 @@ new ModelTest(&model, this); // WHEN - auto data = model.mimeData(QList() << model.index(1, 0) << model.index(2, 0)); + auto data = std::unique_ptr(model.mimeData(QList() << model.index(1, 0) << model.index(2, 0))); // THEN - QVERIFY(data); + QVERIFY(data.get()); QVERIFY(model.mimeTypes().contains(QStringLiteral("application/x-zanshin-object"))); QList colors; colors << Qt::green << Qt::blue; @@ -896,14 +899,14 @@ new ModelTest(&model, this); // WHEN - auto data = new QMimeData; + auto data = std::make_unique(); const QModelIndex parent = parentRow >= 0 ? model.index(parentRow, 0) : QModelIndex(); - model.dropMimeData(data, Qt::MoveAction, row, column, parent); + model.dropMimeData(data.get(), Qt::MoveAction, row, column, parent); // THEN QCOMPARE(dropCalled, callExpected); if (callExpected) { - QCOMPARE(droppedData, data); + QCOMPARE(droppedData, data.get()); QCOMPARE(colorSeen, parent.data(Presentation::QueryTreeModelBase::ObjectRole).value()); } } @@ -968,9 +971,9 @@ << model.index(1, 0, model.index(1, 0)); // WHEN - auto data = model.mimeData(indexes); + auto data = std::unique_ptr(model.mimeData(indexes)); const auto parent = model.index(1, 0, model.index(0, 0, model.index(1, 0))); - model.dropMimeData(data, Qt::MoveAction, -1, -1, parent); + model.dropMimeData(data.get(), Qt::MoveAction, -1, -1, parent); // THEN QVERIFY(!dropCalled);