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, 21 insertions, 21 deletions

diff --git a/tests/auto/testlib/selftests/expected_verifyexceptionthrown.junitxml b/tests/auto/testlib/selftests/expected_verifyexceptionthrown.junitxml
index 003a31a1cc..38fcfed109 100644
--- a/tests/auto/testlib/selftests/expected_verifyexceptionthrown.junitxml
+++ b/tests/auto/testlib/selftests/expected_verifyexceptionthrown.junitxml
@@ -1,32 +1,32 @@
 <?xml version="1.0" encoding="UTF-8" ?>
-<testsuite errors="0" failures="6" tests="11" name="tst_VerifyExceptionThrown">
+<testsuite name="tst_VerifyExceptionThrown" tests="11" failures="6" errors="0">
   <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="pass" name="testCorrectStdTypes"/>
-  <testcase result="pass" name="testCorrectStdExceptions"/>
-  <testcase result="pass" name="testCorrectMyExceptions"/>
-  <testcase result="fail" name="testFailInt">
-    <failure message="Expected exception of type double to be thrown but unknown exception caught" result="fail"/>
+  <testcase name="initTestCase" result="pass"/>
+  <testcase name="testCorrectStdTypes" result="pass"/>
+  <testcase name="testCorrectStdExceptions" result="pass"/>
+  <testcase name="testCorrectMyExceptions" result="pass"/>
+  <testcase name="testFailInt" result="fail">
+    <failure result="fail" message="Expected exception of type double to be thrown but unknown exception caught"/>
   </testcase>
-  <testcase result="fail" name="testFailStdString">
-    <failure message="Expected exception of type char* to be thrown but unknown exception caught" result="fail"/>
+  <testcase name="testFailStdString" result="fail">
+    <failure result="fail" message="Expected exception of type char* to be thrown but unknown exception caught"/>
   </testcase>
-  <testcase result="fail" name="testFailStdRuntimeError">
-    <failure message="Expected exception of type std::runtime_error to be thrown but std::exception caught with message: logic error" result="fail"/>
+  <testcase name="testFailStdRuntimeError" result="fail">
+    <failure result="fail" message="Expected exception of type std::runtime_error to be thrown but std::exception caught with message: logic error"/>
   </testcase>
-  <testcase result="fail" name="testFailMyException">
-    <failure message="Expected exception of type MyBaseException to be thrown but std::exception caught with message: logic error" result="fail"/>
+  <testcase name="testFailMyException" result="fail">
+    <failure result="fail" message="Expected exception of type MyBaseException to be thrown but std::exception caught with message: logic error"/>
   </testcase>
-  <testcase result="fail" name="testFailMyDerivedException">
-    <failure message="Expected exception of type std::runtime_error to be thrown but std::exception caught with message: MyDerivedException" result="fail"/>
+  <testcase name="testFailMyDerivedException" result="fail">
+    <failure result="fail" message="Expected exception of type std::runtime_error to be thrown but std::exception caught with message: MyDerivedException"/>
   </testcase>
-  <testcase result="fail" name="testFailNoException">
-    <failure message="Expected exception of type std::exception to be thrown but no exception caught" result="fail"/>
+  <testcase name="testFailNoException" result="fail">
+    <failure result="fail" message="Expected exception of type std::exception to be thrown but no exception caught"/>
   </testcase>
-  <testcase result="pass" name="cleanupTestCase"/>
+  <testcase name="cleanupTestCase" result="pass"/>
   <system-err/>
 </testsuite>