QAbstractItemModelTester: fix the autotests

Move the modeltest autotest in the right place, and fix the
other autotests that were using it to use the version now in QtTestLib.

Change-Id: Ic6838945f616d580f357c872ce0956c341be3b16
Reviewed-by: David Faure <david.faure@kdab.com>

7 files changed, 0 insertions, 972 deletions

diff --git a/tests/auto/other/modeltest/modeltest.cpp b/tests/auto/other/modeltest/modeltest.cpp
deleted file mode 100644
index 1f4a96f7b6..0000000000
--- a/tests/auto/other/modeltest/modeltest.cpp
+++ /dev/null
@@ -1,585 +0,0 @@
-/****************************************************************************
-**
-** Copyright (C) 2016 The Qt Company Ltd.
-** Contact: https://www.qt.io/licensing/
-**
-** This file is part of the test suite of the Qt Toolkit.
-**
-** $QT_BEGIN_LICENSE:GPL-EXCEPT$
-** Commercial License Usage
-** Licensees holding valid commercial Qt licenses may use this file in
-** accordance with the commercial license agreement provided with the
-** Software or, alternatively, in accordance with the terms contained in
-** a written agreement between you and The Qt Company. For licensing terms
-** and conditions see https://www.qt.io/terms-conditions. For further
-** information use the contact form at https://www.qt.io/contact-us.
-**
-** GNU General Public License Usage
-** Alternatively, this file may be used under the terms of the GNU
-** General Public License version 3 as published by the Free Software
-** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT
-** included in the packaging of this file. Please review the following
-** information to ensure the GNU General Public License requirements will
-** be met: https://www.gnu.org/licenses/gpl-3.0.html.
-**
-** $QT_END_LICENSE$
-**
-****************************************************************************/
-
-#include "modeltest.h"
-
-#include <QtTest/QtTest>
-
-Q_LOGGING_CATEGORY(lcModelTest, "qt.modeltest")
-
-/*!
-    Connect to all of the models signals.  Whenever anything happens recheck everything.
-*/
-ModelTest::ModelTest(QAbstractItemModel *model, QObject *parent)
-    : QObject(parent),
-      model(model),
-      fetchingMore(false)
-{
-    if (!model)
-        qFatal("%s: model must not be null", Q_FUNC_INFO);
-
-    connect(model, &QAbstractItemModel::columnsAboutToBeInserted,
-            this, &ModelTest::runAllTests);
-    connect(model, &QAbstractItemModel::columnsAboutToBeRemoved,
-            this, &ModelTest::runAllTests);
-    connect(model, &QAbstractItemModel::columnsInserted,
-            this, &ModelTest::runAllTests);
-    connect(model, &QAbstractItemModel::columnsRemoved,
-            this, &ModelTest::runAllTests);
-    connect(model, &QAbstractItemModel::dataChanged,
-            this, &ModelTest::runAllTests);
-    connect(model, &QAbstractItemModel::headerDataChanged,
-            this, &ModelTest::runAllTests);
-    connect(model, &QAbstractItemModel::layoutAboutToBeChanged,
-            this, &ModelTest::runAllTests);
-    connect(model, &QAbstractItemModel::layoutChanged,
-            this, &ModelTest::runAllTests);
-    connect(model, &QAbstractItemModel::modelReset,
-            this, &ModelTest::runAllTests);
-    connect(model, &QAbstractItemModel::rowsAboutToBeInserted,
-            this, &ModelTest::runAllTests);
-    connect(model, &QAbstractItemModel::rowsAboutToBeRemoved,
-            this, &ModelTest::runAllTests);
-    connect(model, &QAbstractItemModel::rowsInserted,
-            this, &ModelTest::runAllTests);
-    connect(model, &QAbstractItemModel::rowsRemoved,
-            this, &ModelTest::runAllTests);
-
-    // Special checks for changes
-    connect(model, &QAbstractItemModel::layoutAboutToBeChanged,
-            this, &ModelTest::layoutAboutToBeChanged);
-    connect(model, &QAbstractItemModel::layoutChanged,
-            this, &ModelTest::layoutChanged);
-
-    connect(model, &QAbstractItemModel::rowsAboutToBeInserted,
-            this, &ModelTest::rowsAboutToBeInserted);
-    connect(model, &QAbstractItemModel::rowsAboutToBeRemoved,
-            this, &ModelTest::rowsAboutToBeRemoved);
-    connect(model, &QAbstractItemModel::rowsInserted,
-            this, &ModelTest::rowsInserted);
-    connect(model, &QAbstractItemModel::rowsRemoved,
-            this, &ModelTest::rowsRemoved);
-    connect(model, &QAbstractItemModel::dataChanged,
-            this, &ModelTest::dataChanged);
-    connect(model, &QAbstractItemModel::headerDataChanged,
-            this, &ModelTest::headerDataChanged);
-
-    runAllTests();
-}
-
-void ModelTest::runAllTests()
-{
-    if (fetchingMore)
-        return;
-    nonDestructiveBasicTest();
-    rowCount();
-    columnCount();
-    hasIndex();
-    index();
-    parent();
-    data();
-}
-
-/*!
-    nonDestructiveBasicTest tries to call a number of the basic functions (not all)
-    to make sure the model doesn't outright segfault, testing the functions that makes sense.
-*/
-void ModelTest::nonDestructiveBasicTest()
-{
-    QVERIFY(!model->buddy(QModelIndex()).isValid());
-    model->canFetchMore(QModelIndex());
-    QVERIFY(model->columnCount(QModelIndex()) >= 0);
-    QCOMPARE(model->data(QModelIndex()), QVariant());
-    fetchingMore = true;
-    model->fetchMore(QModelIndex());
-    fetchingMore = false;
-    Qt::ItemFlags flags = model->flags(QModelIndex());
-    QVERIFY(flags == Qt::ItemIsDropEnabled || flags == 0);
-    model->hasChildren(QModelIndex());
-    model->hasIndex(0, 0);
-    model->headerData(0, Qt::Horizontal);
-    model->index(0, 0);
-    model->itemData(QModelIndex());
-    QVariant cache;
-    model->match(QModelIndex(), -1, cache);
-    model->mimeTypes();
-    QVERIFY(!model->parent(QModelIndex()).isValid());
-    QVERIFY(model->rowCount() >= 0);
-    QVariant variant;
-    model->setData(QModelIndex(), variant, -1);
-    model->setHeaderData(-1, Qt::Horizontal, QVariant());
-    model->setHeaderData(999999, Qt::Horizontal, QVariant());
-    QMap<int, QVariant> roles;
-    model->sibling(0, 0, QModelIndex());
-    model->span(QModelIndex());
-    model->supportedDropActions();
-}
-
-/*!
-    Tests model's implementation of QAbstractItemModel::rowCount() and hasChildren()
-
-    Models that are dynamically populated are not as fully tested here.
- */
-void ModelTest::rowCount()
-{
-    // check top row
-    QModelIndex topIndex = model->index(0, 0, QModelIndex());
-    int rows = model->rowCount(topIndex);
-    QVERIFY(rows >= 0);
-    if (rows > 0)
-        QVERIFY(model->hasChildren(topIndex));
-
-    QModelIndex secondLevelIndex = model->index(0, 0, topIndex);
-    if (secondLevelIndex.isValid()) {   // not the top level
-        // check a row count where parent is valid
-        rows = model->rowCount(secondLevelIndex);
-        QVERIFY(rows >= 0);
-        if (rows > 0)
-            QVERIFY(model->hasChildren(secondLevelIndex));
-    }
-
-    // The models rowCount() is tested more extensively in checkChildren(),
-    // but this catches the big mistakes
-}
-
-/*!
-    Tests model's implementation of QAbstractItemModel::columnCount() and hasChildren()
- */
-void ModelTest::columnCount()
-{
-    // check top row
-    QModelIndex topIndex = model->index(0, 0, QModelIndex());
-    QVERIFY(model->columnCount(topIndex) >= 0);
-
-    // check a column count where parent is valid
-    QModelIndex childIndex = model->index(0, 0, topIndex);
-    if (childIndex.isValid())
-        QVERIFY(model->columnCount(childIndex) >= 0);
-
-    // columnCount() is tested more extensively in checkChildren(),
-    // but this catches the big mistakes
-}
-
-/*!
-    Tests model's implementation of QAbstractItemModel::hasIndex()
- */
-void ModelTest::hasIndex()
-{
-    // Make sure that invalid values returns an invalid index
-    QVERIFY(!model->hasIndex(-2, -2));
-    QVERIFY(!model->hasIndex(-2, 0));
-    QVERIFY(!model->hasIndex(0, -2));
-
-    int rows = model->rowCount();
-    int columns = model->columnCount();
-
-    // check out of bounds
-    QVERIFY(!model->hasIndex(rows, columns));
-    QVERIFY(!model->hasIndex(rows + 1, columns + 1));
-
-    if (rows > 0)
-        QVERIFY(model->hasIndex(0, 0));
-
-    // hasIndex() is tested more extensively in checkChildren(),
-    // but this catches the big mistakes
-}
-
-/*!
-    Tests model's implementation of QAbstractItemModel::index()
- */
-void ModelTest::index()
-{
-    // Make sure that invalid values returns an invalid index
-    QVERIFY(!model->index(-2, -2).isValid());
-    QVERIFY(!model->index(-2, 0).isValid());
-    QVERIFY(!model->index(0, -2).isValid());
-
-    int rows = model->rowCount();
-    int columns = model->columnCount();
-
-    if (rows == 0)
-        return;
-
-    // Catch off by one errors
-    QVERIFY(!model->index(rows, columns).isValid());
-    QVERIFY(model->index(0, 0).isValid());
-
-    // Make sure that the same index is *always* returned
-    QModelIndex a = model->index(0, 0);
-    QModelIndex b = model->index(0, 0);
-    QCOMPARE(a, b);
-
-    // index() is tested more extensively in checkChildren(),
-    // but this catches the big mistakes
-}
-
-/*!
-    Tests model's implementation of QAbstractItemModel::parent()
- */
-void ModelTest::parent()
-{
-    // Make sure the model won't crash and will return an invalid QModelIndex
-    // when asked for the parent of an invalid index.
-    QVERIFY(!model->parent(QModelIndex()).isValid());
-
-    if (model->rowCount() == 0)
-        return;
-
-    // Column 0                | Column 1    |
-    // QModelIndex()           |             |
-    //    \- topIndex          | topIndex1   |
-    //         \- childIndex   | childIndex1 |
-
-    // Common error test #1, make sure that a top level index has a parent
-    // that is a invalid QModelIndex.
-    QModelIndex topIndex = model->index(0, 0, QModelIndex());
-    QVERIFY(!model->parent(topIndex).isValid());
-
-    // Common error test #2, make sure that a second level index has a parent
-    // that is the first level index.
-    if (model->rowCount(topIndex) > 0) {
-        QModelIndex childIndex = model->index(0, 0, topIndex);
-        QCOMPARE(model->parent(childIndex), topIndex);
-    }
-
-    // Common error test #3, the second column should NOT have the same children
-    // as the first column in a row.
-    // Usually the second column shouldn't have children.
-    QModelIndex topIndex1 = model->index(0, 1, QModelIndex());
-    if (model->rowCount(topIndex1) > 0) {
-        QModelIndex childIndex = model->index(0, 0, topIndex);
-        QModelIndex childIndex1 = model->index(0, 0, topIndex1);
-        QVERIFY(childIndex != childIndex1);
-    }
-
-    // Full test, walk n levels deep through the model making sure that all
-    // parent's children correctly specify their parent.
-    checkChildren(QModelIndex());
-}
-
-/*!
-    Called from the parent() test.
-
-    A model that returns an index of parent X should also return X when asking
-    for the parent of the index.
-
-    This recursive function does pretty extensive testing on the whole model in an
-    effort to catch edge cases.
-
-    This function assumes that rowCount(), columnCount() and index() already work.
-    If they have a bug it will point it out, but the above tests should have already
-    found the basic bugs because it is easier to figure out the problem in
-    those tests then this one.
- */
-void ModelTest::checkChildren(const QModelIndex &parent, int currentDepth)
-{
-    // First just try walking back up the tree.
-    QModelIndex p = parent;
-    while (p.isValid())
-        p = p.parent();
-
-    // For models that are dynamically populated
-    if (model->canFetchMore(parent)) {
-        fetchingMore = true;
-        model->fetchMore(parent);
-        fetchingMore = false;
-    }
-
-    const int rows = model->rowCount(parent);
-    const int columns = model->columnCount(parent);
-
-    if (rows > 0)
-        QVERIFY(model->hasChildren(parent));
-
-    // Some further testing against rows(), columns(), and hasChildren()
-    QVERIFY(rows >= 0);
-    QVERIFY(columns >= 0);
-    if (rows > 0)
-        QVERIFY(model->hasChildren(parent));
-
-    const QModelIndex topLeftChild = model->index(0, 0, parent);
-
-    QVERIFY(!model->hasIndex(rows + 1, 0, parent));
-    for (int r = 0; r < rows; ++r) {
-        if (model->canFetchMore(parent)) {
-            fetchingMore = true;
-            model->fetchMore(parent);
-            fetchingMore = false;
-        }
-        QVERIFY(!model->hasIndex(r, columns + 1, parent));
-        for (int c = 0; c < columns; ++c) {
-            QVERIFY(model->hasIndex(r, c, parent));
-            QModelIndex index = model->index(r, c, parent);
-            // rowCount() and columnCount() said that it existed...
-            if (!index.isValid())
-                qCWarning(lcModelTest) << "Got invalid index at row=" << r << "col=" << c << "parent=" << parent;
-            QVERIFY(index.isValid());
-
-            // index() should always return the same index when called twice in a row
-            QModelIndex modifiedIndex = model->index(r, c, parent);
-            QCOMPARE(index, modifiedIndex);
-
-            // Make sure we get the same index if we request it twice in a row
-            QModelIndex a = model->index(r, c, parent);
-            QModelIndex b = model->index(r, c, parent);
-            QCOMPARE(a, b);
-
-            {
-                const QModelIndex sibling = model->sibling(r, c, topLeftChild);
-                QCOMPARE(index, sibling);
-            }
-            {
-                const QModelIndex sibling = topLeftChild.sibling(r, c);
-                QCOMPARE(index, sibling);
-            }
-
-            // Some basic checking on the index that is returned
-            QCOMPARE(index.model(), model);
-            QCOMPARE(index.row(), r);
-            QCOMPARE(index.column(), c);
-
-            // If the next test fails here is some somewhat useful debug you play with.
-            if (model->parent(index) != parent) {
-                qCWarning(lcModelTest) << "Inconsistent parent() implementation detected:";
-                qCWarning(lcModelTest) << "    index=" << index << "exp. parent=" << parent << "act. parent=" << model->parent(index);
-                qCWarning(lcModelTest) << "    row=" << r << "col=" << c << "depth=" << currentDepth;
-                qCWarning(lcModelTest) << "    data for child" << model->data(index).toString();
-                qCWarning(lcModelTest) << "    data for parent" << model->data(parent).toString();
-            }
-
-            // Check that we can get back our real parent.
-            QCOMPARE(model->parent(index), parent);
-
-            // recursively go down the children
-            if (model->hasChildren(index) && currentDepth < 10)
-                checkChildren(index, ++currentDepth);
-
-            // make sure that after testing the children that the index doesn't change.
-            QModelIndex newerIndex = model->index(r, c, parent);
-            QCOMPARE(index, newerIndex);
-        }
-    }
-}
-
-/*!
-    Tests model's implementation of QAbstractItemModel::data()
- */
-void ModelTest::data()
-{
-    // Invalid index should return an invalid qvariant
-    QVERIFY(!model->data(QModelIndex()).isValid());
-
-    if (model->rowCount() == 0)
-        return;
-
-    // A valid index should have a valid QVariant data
-    QVERIFY(model->index(0, 0).isValid());
-
-    // shouldn't be able to set data on an invalid index
-    QVERIFY(!model->setData(QModelIndex(), QLatin1String("foo"), Qt::DisplayRole));
-
-    // General Purpose roles that should return a QString
-    QVariant variant = model->data(model->index(0, 0), Qt::ToolTipRole);
-    if (variant.isValid())
-        QVERIFY(variant.canConvert<QString>());
-    variant = model->data(model->index(0, 0), Qt::StatusTipRole);
-    if (variant.isValid())
-        QVERIFY(variant.canConvert<QString>());
-    variant = model->data(model->index(0, 0), Qt::WhatsThisRole);
-    if (variant.isValid())
-        QVERIFY(variant.canConvert<QString>());
-
-    // General Purpose roles that should return a QSize
-    variant = model->data(model->index(0, 0), Qt::SizeHintRole);
-    if (variant.isValid())
-        QVERIFY(variant.canConvert<QSize>());
-
-    // General Purpose roles that should return a QFont
-    QVariant fontVariant = model->data(model->index(0, 0), Qt::FontRole);
-    if (fontVariant.isValid())
-        QVERIFY(fontVariant.canConvert<QFont>());
-
-    // Check that the alignment is one we know about
-    QVariant textAlignmentVariant = model->data(model->index(0, 0), Qt::TextAlignmentRole);
-    if (textAlignmentVariant.isValid()) {
-        Qt::Alignment alignment = textAlignmentVariant.value<Qt::Alignment>();
-        QCOMPARE(alignment, (alignment & (Qt::AlignHorizontal_Mask | Qt::AlignVertical_Mask)));
-    }
-
-    // General Purpose roles that should return a QColor
-    QVariant colorVariant = model->data(model->index(0, 0), Qt::BackgroundColorRole);
-    if (colorVariant.isValid())
-        QVERIFY(colorVariant.canConvert<QColor>());
-
-    colorVariant = model->data(model->index(0, 0), Qt::TextColorRole);
-    if (colorVariant.isValid())
-        QVERIFY(colorVariant.canConvert<QColor>());
-
-    // Check that the "check state" is one we know about.
-    QVariant checkStateVariant = model->data(model->index(0, 0), Qt::CheckStateRole);
-    if (checkStateVariant.isValid()) {
-        int state = checkStateVariant.toInt();
-        QVERIFY(state == Qt::Unchecked
-                || state == Qt::PartiallyChecked
-                || state == Qt::Checked);
-    }
-}
-
-/*!
-    Store what is about to be inserted to make sure it actually happens
-
-    \sa rowsInserted()
- */
-void ModelTest::rowsAboutToBeInserted(const QModelIndex &parent, int start, int end)
-{
-    qCDebug(lcModelTest) << "rowsAboutToBeInserted"
-                         << "start=" << start << "end=" << end << "parent=" << parent
-                         << "parent data=" << model->data(parent).toString()
-                         << "current count of parent=" << model->rowCount(parent)
-                         << "last before insertion=" << model->index(start - 1, 0, parent) << model->data(model->index(start - 1, 0, parent));
-
-    Changing c;
-    c.parent = parent;
-    c.oldSize = model->rowCount(parent);
-    c.last = model->data(model->index(start - 1, 0, parent));
-    c.next = model->data(model->index(start, 0, parent));
-    insert.push(c);
-}
-
-/*!
-    Confirm that what was said was going to happen actually did
-
-    \sa rowsAboutToBeInserted()
- */
-void ModelTest::rowsInserted(const QModelIndex &parent, int start, int end)
-{
-    qCDebug(lcModelTest) << "rowsInserted"
-                         << "start=" << start << "end=" << end << "parent=" << parent
-                         << "parent data=" << model->data(parent).toString()
-                         << "current count of parent=" << model->rowCount(parent);
-
-    for (int i = start; i <= end; ++i) {
-        qCDebug(lcModelTest) << "    itemWasInserted:" << i
-                             << model->index(i, 0, parent).data();
-    }
-
-
-    Changing c = insert.pop();
-    QCOMPARE(parent, c.parent);
-
-    QCOMPARE(model->rowCount(parent), c.oldSize + (end - start + 1));
-    QCOMPARE(model->data(model->index(start - 1, 0, c.parent)), c.last);
-
-    if (c.next != model->data(model->index(end + 1, 0, c.parent))) {
-        qDebug() << start << end;
-        for (int i = 0; i < model->rowCount(); ++i)
-            qDebug() << model->index(i, 0).data().toString();
-        qDebug() << c.next << model->data(model->index(end + 1, 0, c.parent));
-    }
-
-    QCOMPARE(model->data(model->index(end + 1, 0, c.parent)), c.next);
-}
-
-void ModelTest::layoutAboutToBeChanged()
-{
-    for (int i = 0; i < qBound(0, model->rowCount(), 100); ++i)
-        changing.append(QPersistentModelIndex(model->index(i, 0)));
-}
-
-void ModelTest::layoutChanged()
-{
-    for (int i = 0; i < changing.count(); ++i) {
-        QPersistentModelIndex p = changing[i];
-        QCOMPARE(model->index(p.row(), p.column(), p.parent()), QModelIndex(p));
-    }
-    changing.clear();
-}
-
-/*!
-    Store what is about to be inserted to make sure it actually happens
-
-    \sa rowsRemoved()
- */
-void ModelTest::rowsAboutToBeRemoved(const QModelIndex &parent, int start, int end)
-{
-    qCDebug(lcModelTest) << "rowsAboutToBeRemoved"
-                         << "start=" << start << "end=" << end << "parent=" << parent
-                         << "parent data=" << model->data(parent).toString()
-                         << "current count of parent=" << model->rowCount(parent)
-                         << "last before removal=" << model->index(start - 1, 0, parent) << model->data(model->index(start - 1, 0, parent));
-
-    Changing c;
-    c.parent = parent;
-    c.oldSize = model->rowCount(parent);
-    c.last = model->data(model->index(start - 1, 0, parent));
-    c.next = model->data(model->index(end + 1, 0, parent));
-    remove.push(c);
-}
-
-/*!
-    Confirm that what was said was going to happen actually did
-
-    \sa rowsAboutToBeRemoved()
- */
-void ModelTest::rowsRemoved(const QModelIndex &parent, int start, int end)
-{
-    qCDebug(lcModelTest) << "rowsRemoved"
-                         << "start=" << start << "end=" << end << "parent=" << parent
-                         << "parent data=" << model->data(parent).toString()
-                         << "current count of parent=" << model->rowCount(parent);
-
-    Changing c = remove.pop();
-    QCOMPARE(parent, c.parent);
-    QCOMPARE(model->rowCount(parent), c.oldSize - (end - start + 1));
-    QCOMPARE(model->data(model->index(start - 1, 0, c.parent)), c.last);
-    QCOMPARE(model->data(model->index(start, 0, c.parent)), c.next);
-}
-
-void ModelTest::dataChanged(const QModelIndex &topLeft, const QModelIndex &bottomRight)
-{
-    QVERIFY(topLeft.isValid());
-    QVERIFY(bottomRight.isValid());
-    QModelIndex commonParent = bottomRight.parent();
-    QCOMPARE(topLeft.parent(), commonParent);
-    QVERIFY(topLeft.row() <= bottomRight.row());
-    QVERIFY(topLeft.column() <= bottomRight.column());
-    int rowCount = model->rowCount(commonParent);
-    int columnCount = model->columnCount(commonParent);
-    QVERIFY(bottomRight.row() < rowCount);
-    QVERIFY(bottomRight.column() < columnCount);
-}
-
-void ModelTest::headerDataChanged(Qt::Orientation orientation, int start, int end)
-{
-    QVERIFY(start >= 0);
-    QVERIFY(end >= 0);
-    QVERIFY(start <= end);
-    int itemCount = orientation == Qt::Vertical ? model->rowCount() : model->columnCount();
-    QVERIFY(start < itemCount);
-    QVERIFY(end < itemCount);
-}
diff --git a/tests/auto/other/modeltest/modeltest.h b/tests/auto/other/modeltest/modeltest.h
deleted file mode 100644
index 9d09c6da5b..0000000000
--- a/tests/auto/other/modeltest/modeltest.h
+++ /dev/null
@@ -1,80 +0,0 @@
-/****************************************************************************
-**
-** Copyright (C) 2016 The Qt Company Ltd.
-** Contact: https://www.qt.io/licensing/
-**
-** This file is part of the test suite of the Qt Toolkit.
-**
-** $QT_BEGIN_LICENSE:GPL-EXCEPT$
-** Commercial License Usage
-** Licensees holding valid commercial Qt licenses may use this file in
-** accordance with the commercial license agreement provided with the
-** Software or, alternatively, in accordance with the terms contained in
-** a written agreement between you and The Qt Company. For licensing terms
-** and conditions see https://www.qt.io/terms-conditions. For further
-** information use the contact form at https://www.qt.io/contact-us.
-**
-** GNU General Public License Usage
-** Alternatively, this file may be used under the terms of the GNU
-** General Public License version 3 as published by the Free Software
-** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT
-** included in the packaging of this file. Please review the following
-** information to ensure the GNU General Public License requirements will
-** be met: https://www.gnu.org/licenses/gpl-3.0.html.
-**
-** $QT_END_LICENSE$
-**
-****************************************************************************/
-
-#ifndef MODELTEST_H
-#define MODELTEST_H
-
-#include <QtCore/QObject>
-#include <QtCore/QAbstractItemModel>
-#include <QtCore/QStack>
-
-class ModelTest : public QObject
-{
-    Q_OBJECT
-
-public:
-    ModelTest(QAbstractItemModel *model, QObject *parent = nullptr);
-
-private:
-    void nonDestructiveBasicTest();
-    void rowCount();
-    void columnCount();
-    void hasIndex();
-    void index();
-    void parent();
-    void data();
-
-    void runAllTests();
-    void layoutAboutToBeChanged();
-    void layoutChanged();
-    void rowsAboutToBeInserted(const QModelIndex &parent, int start, int end);
-    void rowsInserted(const QModelIndex &parent, int start, int end);
-    void rowsAboutToBeRemoved(const QModelIndex &parent, int start, int end);
-    void rowsRemoved(const QModelIndex &parent, int start, int end);
-    void dataChanged(const QModelIndex &topLeft, const QModelIndex &bottomRight);
-    void headerDataChanged(Qt::Orientation orientation, int start, int end);
-
-    void checkChildren(const QModelIndex &parent, int currentDepth = 0);
-
-    QAbstractItemModel *model;
-
-    struct Changing {
-        QModelIndex parent;
-        int oldSize;
-        QVariant last;
-        QVariant next;
-    };
-    QStack<Changing> insert;
-    QStack<Changing> remove;
-
-    bool fetchingMore;
-
-    QList<QPersistentModelIndex> changing;
-};
-
-#endif
diff --git a/tests/auto/other/modeltest/modeltest.pro b/tests/auto/other/modeltest/modeltest.pro
deleted file mode 100644
index 8e96058449..0000000000
--- a/tests/auto/other/modeltest/modeltest.pro
+++ /dev/null
@@ -1,5 +0,0 @@
-CONFIG += testcase
-TARGET = tst_modeltest
-QT += widgets testlib
-SOURCES         += tst_modeltest.cpp modeltest.cpp dynamictreemodel.cpp
-HEADERS         += modeltest.h dynamictreemodel.h
diff --git a/tests/auto/other/modeltest/tst_modeltest.cpp b/tests/auto/other/modeltest/tst_modeltest.cpp
deleted file mode 100644
index e2d002844b..0000000000
--- a/tests/auto/other/modeltest/tst_modeltest.cpp
+++ /dev/null
@@ -1,300 +0,0 @@
-/****************************************************************************
-**
-** Copyright (C) 2016 The Qt Company Ltd.
-** Contact: https://www.qt.io/licensing/
-**
-** This file is part of the test suite of the Qt Toolkit.
-**
-** $QT_BEGIN_LICENSE:GPL-EXCEPT$
-** Commercial License Usage
-** Licensees holding valid commercial Qt licenses may use this file in
-** accordance with the commercial license agreement provided with the
-** Software or, alternatively, in accordance with the terms contained in
-** a written agreement between you and The Qt Company. For licensing terms
-** and conditions see https://www.qt.io/terms-conditions. For further
-** information use the contact form at https://www.qt.io/contact-us.
-**
-** GNU General Public License Usage
-** Alternatively, this file may be used under the terms of the GNU
-** General Public License version 3 as published by the Free Software
-** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT
-** included in the packaging of this file. Please review the following
-** information to ensure the GNU General Public License requirements will
-** be met: https://www.gnu.org/licenses/gpl-3.0.html.
-**
-** $QT_END_LICENSE$
-**
-****************************************************************************/
-
-#include <QtTest/QtTest>
-#include <QtGui/QtGui>
-#include <QtWidgets/QtWidgets>
-
-#include "modeltest.h"
-#include "dynamictreemodel.h"
-
-class tst_ModelTest : public QObject
-{
-    Q_OBJECT
-
-private slots:
-    void stringListModel();
-    void treeWidgetModel();
-    void standardItemModel();
-    void testInsertThroughProxy();
-    void moveSourceItems();
-    void testResetThroughProxy();
-};
-
-/*
-  tests
-*/
-
-void tst_ModelTest::stringListModel()
-{
-    QStringListModel model;
-    QSortFilterProxyModel proxy;
-
-    ModelTest t1(&model);
-    ModelTest t2(&proxy);
-
-    proxy.setSourceModel(&model);
-
-    model.setStringList(QStringList() << "2" << "3" << "1");
-    model.setStringList(QStringList() << "a" << "e" << "plop" << "b" << "c");
-
-    proxy.setDynamicSortFilter(true);
-    proxy.setFilterRegExp(QRegExp("[^b]"));
-}
-
-void tst_ModelTest::treeWidgetModel()
-{
-    QTreeWidget widget;
-
-    ModelTest t1(widget.model());
-
-    QTreeWidgetItem *root = new QTreeWidgetItem(&widget, QStringList("root"));
-    for (int i = 0; i < 20; ++i)
-        new QTreeWidgetItem(root, QStringList(QString::number(i)));
-    QTreeWidgetItem *remove = root->child(2);
-    root->removeChild(remove);
-    QTreeWidgetItem *parent = new QTreeWidgetItem(&widget, QStringList("parent"));
-    new QTreeWidgetItem(parent, QStringList("child"));
-    widget.setItemHidden(parent, true);
-
-    widget.sortByColumn(0);
-}
-
-void tst_ModelTest::standardItemModel()
-{
-    QStandardItemModel model(10, 10);
-    QSortFilterProxyModel proxy;
-
-    ModelTest t1(&model);
-    ModelTest t2(&proxy);
-
-    proxy.setSourceModel(&model);
-
-    model.insertRows(2, 5);
-    model.removeRows(4, 5);
-
-    model.insertColumns(2, 5);
-    model.removeColumns(4, 5);
-
-    model.insertRows(0, 5, model.index(1, 1));
-    model.insertColumns(0, 5, model.index(1, 3));
-}
-
-void tst_ModelTest::testInsertThroughProxy()
-{
-    DynamicTreeModel *model = new DynamicTreeModel(this);
-
-    QSortFilterProxyModel *proxy = new QSortFilterProxyModel(this);
-    proxy->setSourceModel(model);
-
-    new ModelTest(proxy, this);
-
-    ModelInsertCommand *insertCommand = new ModelInsertCommand(model, this);
-    insertCommand->setNumCols(4);
-    insertCommand->setStartRow(0);
-    insertCommand->setEndRow(9);
-    // Parent is QModelIndex()
-    insertCommand->doCommand();
-
-    insertCommand = new ModelInsertCommand(model, this);
-    insertCommand->setNumCols(4);
-    insertCommand->setAncestorRowNumbers(QList<int>() << 5);
-    insertCommand->setStartRow(0);
-    insertCommand->setEndRow(9);
-    insertCommand->doCommand();
-
-    ModelMoveCommand *moveCommand = new ModelMoveCommand(model, this);
-    moveCommand->setNumCols(4);
-    moveCommand->setStartRow(0);
-    moveCommand->setEndRow(0);
-    moveCommand->setDestRow(9);
-    moveCommand->setDestAncestors(QList<int>() << 5);
-    moveCommand->doCommand();
-}
-
-/**
-  Makes the persistent index list publicly accessible
-*/
-class AccessibleProxyModel : public QSortFilterProxyModel
-{
-    Q_OBJECT
-public:
-    AccessibleProxyModel(QObject *parent = 0) : QSortFilterProxyModel(parent)
-    {
-    }
-
-    QModelIndexList persistent()
-    {
-        return persistentIndexList();
-    }
-};
-
-class ObservingObject : public QObject
-{
-    Q_OBJECT
-public:
-    ObservingObject(AccessibleProxyModel *proxy, QObject *parent = 0) :
-        QObject(parent),
-        m_proxy(proxy),
-        storePersistentFailureCount(0),
-        checkPersistentFailureCount(0)
-    {
-        connect(m_proxy, SIGNAL(rowsAboutToBeMoved(QModelIndex,int,int,QModelIndex,int)),
-                SLOT(storePersistent()));
-        connect(m_proxy, SIGNAL(rowsMoved(QModelIndex,int,int,QModelIndex,int)),
-                SLOT(checkPersistent()));
-    }
-
-public slots:
-
-    void storePersistent(const QModelIndex &parent)
-    {
-        for (int row = 0; row < m_proxy->rowCount(parent); ++row) {
-            QModelIndex proxyIndex = m_proxy->index(row, 0, parent);
-            QModelIndex sourceIndex = m_proxy->mapToSource(proxyIndex);
-            if (!proxyIndex.isValid()) {
-                qWarning("%s: Invalid proxy index", Q_FUNC_INFO);
-                ++storePersistentFailureCount;
-            }
-            if (!sourceIndex.isValid()) {
-                qWarning("%s: invalid source index", Q_FUNC_INFO);
-                ++storePersistentFailureCount;
-            }
-            m_persistentSourceIndexes.append(sourceIndex);
-            m_persistentProxyIndexes.append(proxyIndex);
-            if (m_proxy->hasChildren(proxyIndex))
-                storePersistent(proxyIndex);
-        }
-    }
-
-    void storePersistent()
-    {
-        // This method is called from rowsAboutToBeMoved. Persistent indexes should be valid
-        foreach (const QModelIndex &idx, m_persistentProxyIndexes)
-            if (!idx.isValid()) {
-                qWarning("%s: persistentProxyIndexes contains invalid index", Q_FUNC_INFO);
-                ++storePersistentFailureCount;
-            }
-
-        if (!m_proxy->persistent().isEmpty()) {
-            qWarning("%s: proxy should have no persistent indexes when storePersistent called",
-                     Q_FUNC_INFO);
-            ++storePersistentFailureCount;
-        }
-        storePersistent(QModelIndex());
-        if (m_proxy->persistent().isEmpty()) {
-            qWarning("%s: proxy should have persistent index after storePersistent called",
-                     Q_FUNC_INFO);
-            ++storePersistentFailureCount;
-        }
-    }
-
-    void checkPersistent()
-    {
-        for (int row = 0; row < m_persistentProxyIndexes.size(); ++row) {
-            m_persistentProxyIndexes.at(row);
-            m_persistentSourceIndexes.at(row);
-        }
-        for (int row = 0; row < m_persistentProxyIndexes.size(); ++row) {
-            QModelIndex updatedProxy = m_persistentProxyIndexes.at(row);
-            QModelIndex updatedSource = m_persistentSourceIndexes.at(row);
-            if (m_proxy->mapToSource(updatedProxy) != updatedSource) {
-                qWarning("%s: check failed at row %d", Q_FUNC_INFO, row);
-                ++checkPersistentFailureCount;
-            }
-        }
-        m_persistentSourceIndexes.clear();
-        m_persistentProxyIndexes.clear();
-    }
-
-private:
-    AccessibleProxyModel *m_proxy;
-    QList<QPersistentModelIndex> m_persistentSourceIndexes;
-    QList<QPersistentModelIndex> m_persistentProxyIndexes;
-public:
-    int storePersistentFailureCount;
-    int checkPersistentFailureCount;
-};
-
-void tst_ModelTest::moveSourceItems()
-{
-    DynamicTreeModel *model = new DynamicTreeModel(this);
-    AccessibleProxyModel *proxy = new AccessibleProxyModel(this);
-    proxy->setSourceModel(model);
-
-    ModelInsertCommand *insertCommand = new ModelInsertCommand(model, this);
-    insertCommand->setStartRow(0);
-    insertCommand->setEndRow(2);
-    insertCommand->doCommand();
-
-    insertCommand = new ModelInsertCommand(model, this);
-    insertCommand->setAncestorRowNumbers(QList<int>() << 1);
-    insertCommand->setStartRow(0);
-    insertCommand->setEndRow(2);
-    insertCommand->doCommand();
-
-    ObservingObject observer(proxy);
-
-    ModelMoveCommand *moveCommand = new ModelMoveCommand(model, this);
-    moveCommand->setStartRow(0);
-    moveCommand->setEndRow(0);
-    moveCommand->setDestAncestors(QList<int>() << 1);
-    moveCommand->setDestRow(0);
-    moveCommand->doCommand();
-
-    QCOMPARE(observer.storePersistentFailureCount, 0);
-    QCOMPARE(observer.checkPersistentFailureCount, 0);
-}
-
-void tst_ModelTest::testResetThroughProxy()
-{
-    DynamicTreeModel *model = new DynamicTreeModel(this);
-
-    ModelInsertCommand *insertCommand = new ModelInsertCommand(model, this);
-    insertCommand->setStartRow(0);
-    insertCommand->setEndRow(2);
-    insertCommand->doCommand();
-
-    QPersistentModelIndex persistent = model->index(0, 0);
-
-    AccessibleProxyModel *proxy = new AccessibleProxyModel(this);
-    proxy->setSourceModel(model);
-
-    ObservingObject observer(proxy);
-    observer.storePersistent();
-
-    ModelResetCommand *resetCommand = new ModelResetCommand(model, this);
-    resetCommand->setNumCols(0);
-    resetCommand->doCommand();
-
-    QCOMPARE(observer.storePersistentFailureCount, 0);
-    QCOMPARE(observer.checkPersistentFailureCount, 0);
-}
-
-QTEST_MAIN(tst_ModelTest)
-#include "tst_modeltest.moc"
diff --git a/tests/auto/other/other.pro b/tests/auto/other/other.pro
index 0e84b187fa..4119a40279 100644
--- a/tests/auto/other/other.pro
+++ b/tests/auto/other/other.pro
@@ -9,7 +9,6 @@ SUBDIRS=\
    macgui \
    macnativeevents \
    macplist \
-   modeltest \
    networkselftest \
    qaccessibility \
    # qaccessibilitylinux \ # QTBUG-44434
@@ -32,7 +31,6 @@ SUBDIRS=\
    gestures \
    lancelot \
    languagechange \
-   modeltest \
    qaccessibility \
    qfocusevent \
    qnetworkaccessmanager_and_qprogressdialog \
diff --git a/tests/auto/other/modeltest/dynamictreemodel.cpp b/tests/auto/other/qabstractitemmodelutils/dynamictreemodel.cpp
index fc979bce2d..fc979bce2d 100644
--- a/tests/auto/other/modeltest/dynamictreemodel.cpp
+++ b/tests/auto/other/qabstractitemmodelutils/dynamictreemodel.cpp
diff --git a/tests/auto/other/modeltest/dynamictreemodel.h b/tests/auto/other/qabstractitemmodelutils/dynamictreemodel.h
index 709751dd27..709751dd27 100644
--- a/tests/auto/other/modeltest/dynamictreemodel.h
+++ b/tests/auto/other/qabstractitemmodelutils/dynamictreemodel.h