test: removed obsolete tst_maketestselftest::tests_auto_pro
authorRohan McGovern <rohan.mcgovern@nokia.com>
Sun, 28 Aug 2011 23:24:37 +0000 (09:24 +1000)
committerJo Asplin <jo.asplin@nokia.com>
Mon, 29 Aug 2011 07:29:43 +0000 (09:29 +0200)
This test attempted to ensure that tests/auto/auto.pro would only refer
to other .pro files, attempting to guarantee that the set of all tests
could be cleanly broken up.  The purpose of this was to enable CI
optimizations (e.g. running the autotests for different modules on
different machines in parallel).

This test is invalidated by pending commits which rearrange the
autotests, and we never made use of this property for optimization
anyway, so drop this part of the test.

Change-Id: I23e712fb8ec1dbe7ac65fe66015e1f060f3dcd41
Reviewed-on: http://codereview.qt.nokia.com/3681
Reviewed-by: Qt Sanity Bot <qt_sanity_bot@ovi.com>
Reviewed-by: Jo Asplin <jo.asplin@nokia.com>
tests/auto/maketestselftest/tst_maketestselftest.cpp

index 07c68ac..59dd43c 100644 (file)
@@ -58,8 +58,6 @@ class tst_MakeTestSelfTest: public QObject
     Q_OBJECT
 
 private slots:
-    void tests_auto_pro();
-
     void tests_pro_files();
     void tests_pro_files_data();
 
@@ -78,30 +76,6 @@ bool looks_like_testcase(QString const&,QString*);
 bool looks_like_subdirs(QString const&);
 QStringList find_test_class(QString const&);
 
-/*
-    Verify that auto.pro only contains other .pro files (and not directories).
-    We enforce this so that we can process every .pro file other than auto.pro
-    independently and get all the tests.
-    If tests were allowed to appear directly in auto.pro, we'd have the problem
-    that we need to somehow run these tests from auto.pro while preventing
-    recursion into the other .pro files.
-*/
-void tst_MakeTestSelfTest::tests_auto_pro()
-{
-    QStringList subdirsList = find_subdirs(SRCDIR "/../auto.pro", Flat);
-    if (QTest::currentTestFailed()) {
-        return;
-    }
-
-    foreach (QString const& subdir, subdirsList) {
-        QVERIFY2(subdir.endsWith(".pro"), qPrintable(QString(
-            "auto.pro contains a subdir `%1'.\n"
-            "auto.pro must _only_ contain other .pro files, not actual subdirs.\n"
-            "Please move `%1' into some other .pro file referenced by auto.pro."
-        ).arg(subdir)));
-    }
-}
-
 /* Verify that all tests are listed somewhere in one of the autotest .pro files */
 void tst_MakeTestSelfTest::tests_pro_files()
 {