qmlformat: Remove import sorting

Remove import sorting due to the fact that sorting imports can break code.

[ChangeLog][QML Tooling][qmlformat] Remove import sorting and the (now obsolete) -n parameter to disable it

Fixes: QTBUG-89295
Pick-to: 6.0
Change-Id: I5ff13d0ae3c715db7645b412152aadb31811ce5a
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>

7 files changed, 24 insertions, 285 deletions

diff --git a/tests/auto/qml/qmlformat/data/Annotations.formatted.nosort.qml b/tests/auto/qml/qmlformat/data/Annotations.formatted.nosort.qml
deleted file mode 100644
index a05c2125dc..0000000000
--- a/tests/auto/qml/qmlformat/data/Annotations.formatted.nosort.qml
+++ /dev/null
@@ -1,106 +0,0 @@
-/****************************************************************************
-**
-** Copyright (C) 2016 The Qt Company Ltd.
-** Contact: https://www.qt.io/licensing/
-**
-** This file is part of the Qt Charts module of the Qt Toolkit.
-**
-** $QT_BEGIN_LICENSE:GPL$
-** 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 or (at your option) any later version
-** approved by the KDE Free Qt Foundation. The licenses are as published by
-** the Free Software Foundation and appearing in the file LICENSE.GPL3
-** 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$
-**
-****************************************************************************/
-
-//![2]
-import QtQuick 2.0
-//![2]
-import QtCharts 2.0
-
-@Pippo {
-    atg1: 3
-}
-@Annotation2 {
-}
-Item {
-    //![1]
-
-    @AnnotateMore {
-        property int x: 5
-    }
-    @AnnotateALot {
-    }
-
-    property variant othersSlice: 0
-    @Annotate {
-    }
-
-    anchors.fill: parent
-    @SuperComplete {
-        binding: late
-    }
-    Component.onCompleted: {
-        // You can also manipulate slices dynamically, like append a slice or set a slice exploded
-        othersSlice = pieSeries.append("Others", 52);
-        pieSeries.find("Volkswagen").exploded = true;
-    }
-    //![1]
-    ChartView {
-        id: chart
-
-        title: "Top-5 car brand shares in Finland"
-        anchors.fill: parent
-        legend.alignment: Qt.AlignBottom
-        antialiasing: true
-
-        @ExtraAnnotation {
-            signal pippo()
-        }
-        PieSeries {
-            id: pieSeries
-
-            PieSlice {
-                label: "Volkswagen"
-                value: 13.5
-            }
-
-            PieSlice {
-                label: "Toyota"
-                value: 10.9
-            }
-
-            PieSlice {
-                label: "Ford"
-                value: 8.6
-            }
-
-            PieSlice {
-                label: "Skoda"
-                value: 8.2
-            }
-
-            PieSlice {
-                label: "Volvo"
-                value: 6.8
-            }
-
-        }
-
-    }
-
-}
diff --git a/tests/auto/qml/qmlformat/data/Annotations.formatted.qml b/tests/auto/qml/qmlformat/data/Annotations.formatted.qml
index a142d4cb74..a05c2125dc 100644
--- a/tests/auto/qml/qmlformat/data/Annotations.formatted.qml
+++ b/tests/auto/qml/qmlformat/data/Annotations.formatted.qml
@@ -28,9 +28,9 @@
 ****************************************************************************/
 
 //![2]
-import QtCharts 2.0
-//![2]
 import QtQuick 2.0
+//![2]
+import QtCharts 2.0
 
 @Pippo {
     atg1: 3
diff --git a/tests/auto/qml/qmlformat/data/Example1.formatted.2spaces.qml b/tests/auto/qml/qmlformat/data/Example1.formatted.2spaces.qml
index 20d2a03421..5e27aeec22 100644
--- a/tests/auto/qml/qmlformat/data/Example1.formatted.2spaces.qml
+++ b/tests/auto/qml/qmlformat/data/Example1.formatted.2spaces.qml
@@ -3,19 +3,19 @@
 	2. Goodbye
 */
 
-import A.B.B.A
-import A.LLOHA
 // Importing this is very important
 import QtQuick 5.15
 // Muddling the waters!
 import QtQuick.Models 3.14 as muddle
+// Importing that is important too
+import Z
 import That
 import This // THIS IS VERY IMPORTANT!
-import X.Y
-import X.Z
 import Y
-// Importing that is important too
-import Z
+import X.Z
+import X.Y
+import A.LLOHA
+import A.B.B.A
 
 // This comment is related to Item
 Item {
diff --git a/tests/auto/qml/qmlformat/data/Example1.formatted.nosort.qml b/tests/auto/qml/qmlformat/data/Example1.formatted.nosort.qml
deleted file mode 100644
index fdc0b1f68c..0000000000
--- a/tests/auto/qml/qmlformat/data/Example1.formatted.nosort.qml
+++ /dev/null
@@ -1,151 +0,0 @@
-/* This file is licensed under the not a license license
-	1. You may not comply
-	2. Goodbye
-*/
-
-// Importing this is very important
-import QtQuick 5.15
-// Muddling the waters!
-import QtQuick.Models 3.14 as muddle
-// Importing that is important too
-import Z
-import That
-import This // THIS IS VERY IMPORTANT!
-import Y
-import X.Z
-import X.Y
-import A.LLOHA
-import A.B.B.A
-
-// This comment is related to Item
-Item {
-    // Orphan comment
-    // Another orphan
-    // More orphans
-
-    // This to id
-    // Also id. (line 2)
-    // This is the third id
-    // fourth id comment
-    id: foo
-
-    // This to enum
-    enum Foo {
-        A = 3, // This is A
-        B, // This is B
-        C = 4, // This is C
-        D // This is D
-    }
-
-    property bool some_bool: false
-    property variant some_array_literal: [30, 20, Math["PI"], [4, 3, 2], "foo", 0.3]
-    property bool something_computed: function(x) {
-        // This is an orphan inside something_computed
-        // Are these getting duplicated?
-        // Another orphan inside something_computed
-
-        const PI = 3, DAYS_PER_YEAR = 365.25;
-        var x = 3 + 2;
-        x["bla"] = 50;
-        // This one to var few!
-        var few = new WhatEver();
-        x += Math.sin(3);
-        x--;
-        --x;
-        x++;
-        ++x;
-        for (var x = 0; x < 100; x++) {
-            x++;
-            console.log("Foo");
-        }
-        for (var x in [3, 2, 1]) {
-            y++;
-            console.log("Bar");
-        }
-        while (true)
-            console.log("Wee");
-
-        with (foo) {
-            bar;
-            x += 5;
-        } // This is related to with!
-        x3:
-        do {
-            console.log("Hello");
-        } while (3 == 0);
-        try {
-            dangerous();
-        } catch (e) {
-            console.log(e);
-        } finally {
-            dangerous();
-        }
-        switch (x) {
-        case 0:
-            x = 1;
-            break;
-        case 1:
-            x = 5;
-            break;
-        case 4:
-            x = 100;
-            break;
-        }
-        if (x == 50)
-            console.log("true");
-        else if (x == 50)
-            console.log("other thing");
-        else
-            console.log("false");
-        if (x == 50) {
-            console.log("true");
-        } else if (x == 50) {
-            console.log("other thing");
-            x--;
-        } else {
-            console.log("false");
-        }
-        return "foobar";
-    }()
-    default property bool some_default_bool: 500 % 5 !== 0 // some_default_bool
-    // some_read_only_bool
-    readonly property bool some_read_only_bool: Math.sin(3) && (aFunc()[30] + 5) | 2 != 0
-
-    signal say(string name, bool caps)
-
-    // This one to aFunc()
-    function aFunc() {
-        var x = 3;
-        return x;
-    }
-
-    x: 3 // Very cool
-    Component.onCompleted: console.log("Foo!")
-    myFavouriteThings: [
-        // This is an orphan
-
-        // This is a cool text
-        Text {
-        },
-        // This is a cool rectangle
-        Rectangle {
-        }
-    ]
-
-    Text {
-        required property string batman
-
-        signal boo(int count, int times, real duration)
-
-        text: "Bla"
-    }
-
-    // This comment is related to the property animation
-    PropertyAnimation on x {
-        id: foo
-
-        x: 3
-        y: x + 3
-    }
-
-}
diff --git a/tests/auto/qml/qmlformat/data/Example1.formatted.qml b/tests/auto/qml/qmlformat/data/Example1.formatted.qml
index 5f12517781..fdc0b1f68c 100644
--- a/tests/auto/qml/qmlformat/data/Example1.formatted.qml
+++ b/tests/auto/qml/qmlformat/data/Example1.formatted.qml
@@ -3,19 +3,19 @@
 	2. Goodbye
 */
 
-import A.B.B.A
-import A.LLOHA
 // Importing this is very important
 import QtQuick 5.15
 // Muddling the waters!
 import QtQuick.Models 3.14 as muddle
+// Importing that is important too
+import Z
 import That
 import This // THIS IS VERY IMPORTANT!
-import X.Y
-import X.Z
 import Y
-// Importing that is important too
-import Z
+import X.Z
+import X.Y
+import A.LLOHA
+import A.B.B.A
 
 // This comment is related to Item
 Item {
diff --git a/tests/auto/qml/qmlformat/data/Example1.formatted.tabs.qml b/tests/auto/qml/qmlformat/data/Example1.formatted.tabs.qml
index d569c414d2..d1c17c4c6b 100644
--- a/tests/auto/qml/qmlformat/data/Example1.formatted.tabs.qml
+++ b/tests/auto/qml/qmlformat/data/Example1.formatted.tabs.qml
@@ -3,19 +3,19 @@
 	2. Goodbye
 */
 
-import A.B.B.A
-import A.LLOHA
 // Importing this is very important
 import QtQuick 5.15
 // Muddling the waters!
 import QtQuick.Models 3.14 as muddle
+// Importing that is important too
+import Z
 import That
 import This // THIS IS VERY IMPORTANT!
-import X.Y
-import X.Z
 import Y
-// Importing that is important too
-import Z
+import X.Z
+import X.Y
+import A.LLOHA
+import A.B.B.A
 
 // This comment is related to Item
 Item {
diff --git a/tests/auto/qml/qmlformat/tst_qmlformat.cpp b/tests/auto/qml/qmlformat/tst_qmlformat.cpp
index f114704b0a..165be7e973 100644
--- a/tests/auto/qml/qmlformat/tst_qmlformat.cpp
+++ b/tests/auto/qml/qmlformat/tst_qmlformat.cpp
@@ -201,21 +201,17 @@ void TestQmlformat::testFormat_data()
     QTest::addColumn<QString>("fileFormatted");
     QTest::addColumn<QStringList>("args");
 
-    QTest::newRow("example1 (sorted)") << "Example1.qml"
-                                       << "Example1.formatted.qml" << QStringList {};
-    QTest::newRow("example1 (not sorted)")
+    QTest::newRow("example1")
             << "Example1.qml"
-            << "Example1.formatted.nosort.qml" << QStringList { "-n" };
+            << "Example1.formatted.qml" << QStringList {};
     QTest::newRow("example1 (tabs)") << "Example1.qml"
                                      << "Example1.formatted.tabs.qml" << QStringList { "-t" };
     QTest::newRow("example1 (two spaces)")
             << "Example1.qml"
             << "Example1.formatted.2spaces.qml" << QStringList { "-w", "2" };
-    QTest::newRow("annotation (sorted)") << "Annotations.qml"
-                                         << "Annotations.formatted.qml" << QStringList {};
-    QTest::newRow("annotation (not sorted)")
+    QTest::newRow("annotation")
             << "Annotations.qml"
-            << "Annotations.formatted.nosort.qml" << QStringList { "-n" };
+            << "Annotations.formatted.qml" << QStringList {};
     QTest::newRow("front inline") << "FrontInline.qml"
                                   << "FrontInline.formatted.qml" << QStringList {};
     QTest::newRow("if blocks") << "IfBlocks.qml"