testlib: Output JUnitXML attributes in right order

The attributes are, like the elements, maintained in reverse
order in the underlying QTestCoreList, so we need to iterate
them backwards when printing out the resulting XML to reflect
the order they were added.

This results in e.g.:

  <testcase name="passingBenchmark" result="pass">

Instead of:

  <testcase result="pass" name="passingBenchmark">

Change-Id: Ic2eeab8de05ffedd0c41977358d5b40ff77878b1
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>

1 files changed, 37 insertions, 37 deletions

diff --git a/tests/auto/testlib/selftests/expected_expectfail.junitxml b/tests/auto/testlib/selftests/expected_expectfail.junitxml
index 47483d71bf..03d9b2c32d 100644
--- a/tests/auto/testlib/selftests/expected_expectfail.junitxml
+++ b/tests/auto/testlib/selftests/expected_expectfail.junitxml
@@ -1,54 +1,54 @@
 <?xml version="1.0" encoding="UTF-8" ?>
-<testsuite errors="13" failures="6" tests="14" name="tst_ExpectFail">
+<testsuite name="tst_ExpectFail" tests="14" failures="6" errors="13">
   <properties>
-    <property value="@INSERT_QT_VERSION_HERE@" name="QTestVersion"/>
-    <property value="@INSERT_QT_VERSION_HERE@" name="QtVersion"/>
-    <property value="" name="QtBuild"/>
+    <property name="QTestVersion" value="@INSERT_QT_VERSION_HERE@"/>
+    <property name="QtVersion" value="@INSERT_QT_VERSION_HERE@"/>
+    <property name="QtBuild" value=""/>
   </properties>
-  <testcase result="pass" name="initTestCase"/>
-  <testcase result="xfail" name="xfailAndContinue">
-    <!-- message="begin" type="qdebug" -->
-    <!-- message="This should xfail" type="info" -->
-    <!-- message="after" type="qdebug" -->
+  <testcase name="initTestCase" result="pass"/>
+  <testcase name="xfailAndContinue" result="xfail">
+    <!-- type="qdebug" message="begin" -->
+    <!-- type="info" message="This should xfail" -->
+    <!-- type="qdebug" message="after" -->
   </testcase>
-  <testcase result="xfail" name="xfailAndAbort">
-    <!-- message="begin" type="qdebug" -->
-    <!-- message="This should xfail" type="info" -->
+  <testcase name="xfailAndAbort" result="xfail">
+    <!-- type="qdebug" message="begin" -->
+    <!-- type="info" message="This should xfail" -->
   </testcase>
-  <testcase result="fail" name="xfailTwice">
-    <failure message="Already expecting a fail" result="fail"/>
+  <testcase name="xfailTwice" result="fail">
+    <failure result="fail" message="Already expecting a fail"/>
   </testcase>
-  <testcase result="xfail" name="xfailWithQString">
-    <!-- message="A string" type="info" -->
-    <!-- message="Bug 5 (The message)" type="info" -->
+  <testcase name="xfailWithQString" result="xfail">
+    <!-- type="info" message="A string" -->
+    <!-- type="info" message="Bug 5 (The message)" -->
   </testcase>
-  <testcase result="xfail" name="xfailDataDrivenWithQVerify">
-    <!-- tag="Abort" message="This test should xfail" type="info" -->
-    <!-- tag="Continue" message="This test should xfail" type="info" -->
+  <testcase name="xfailDataDrivenWithQVerify" result="xfail">
+    <!-- type="info" message="This test should xfail" tag="Abort" -->
+    <!-- type="info" message="This test should xfail" tag="Continue" -->
   </testcase>
-  <testcase result="xfail" name="xfailDataDrivenWithQCompare">
-    <!-- tag="Abort" message="This test should xfail" type="info" -->
-    <!-- tag="Continue" message="This test should xfail" type="info" -->
+  <testcase name="xfailDataDrivenWithQCompare" result="xfail">
+    <!-- type="info" message="This test should xfail" tag="Abort" -->
+    <!-- type="info" message="This test should xfail" tag="Continue" -->
   </testcase>
-  <testcase result="pass" name="xfailOnWrongRow"/>
-  <testcase result="xfail" name="xfailOnAnyRow">
-    <!-- tag="first row" message="This test should xfail" type="info" -->
-    <!-- tag="second row" message="This test should xfail" type="info" -->
+  <testcase name="xfailOnWrongRow" result="pass"/>
+  <testcase name="xfailOnAnyRow" result="xfail">
+    <!-- type="info" message="This test should xfail" tag="first row" -->
+    <!-- type="info" message="This test should xfail" tag="second row" -->
   </testcase>
-  <testcase result="fail" name="xfailWithoutVerify">
-    <failure tag="first row" message="QEXPECT_FAIL was called without any subsequent verification statements" result="fail"/>
-    <failure tag="second row" message="QEXPECT_FAIL was called without any subsequent verification statements" result="fail"/>
+  <testcase name="xfailWithoutVerify" result="fail">
+    <failure result="fail" message="QEXPECT_FAIL was called without any subsequent verification statements" tag="first row"/>
+    <failure result="fail" message="QEXPECT_FAIL was called without any subsequent verification statements" tag="second row"/>
   </testcase>
-  <testcase result="xpass" name="xpass">
-    <failure message="&apos;true&apos; returned TRUE unexpectedly. ()" result="xpass"/>
+  <testcase name="xpass" result="xpass">
+    <failure result="xpass" message="&apos;true&apos; returned TRUE unexpectedly. ()"/>
   </testcase>
-  <testcase result="xpass" name="xpassDataDrivenWithQVerify">
-    <failure tag="XPass" message="&apos;true&apos; returned TRUE unexpectedly. ()" result="xpass"/>
+  <testcase name="xpassDataDrivenWithQVerify" result="xpass">
+    <failure result="xpass" message="&apos;true&apos; returned TRUE unexpectedly. ()" tag="XPass"/>
   </testcase>
-  <testcase result="xpass" name="xpassDataDrivenWithQCompare">
-    <failure tag="XPass" message="QCOMPARE(1, 1) returned TRUE unexpectedly." result="xpass"/>
+  <testcase name="xpassDataDrivenWithQCompare" result="xpass">
+    <failure result="xpass" message="QCOMPARE(1, 1) returned TRUE unexpectedly." tag="XPass"/>
   </testcase>
-  <testcase result="pass" name="cleanupTestCase"/>
+  <testcase name="cleanupTestCase" result="pass"/>
   <system-err>
 <![CDATA[begin]]>
 <![CDATA[This should xfail]]>