Comment 4 for bug 1297500

Revision history for this message
Daniel Manrique (roadmr) wrote :

Once plainbox is exporting the summary, this patch to testitemmodel.cpp uses the summary if the test has one (decided by whether the summary is different from the id), otherwise it uses partial_id.

=== modified file 'plainbox-gui/canonical-driver-test-suite/testitemmodel.cpp'
--- plainbox-gui/canonical-driver-test-suite/testitemmodel.cpp 2014-03-26 18:12:32 +0000
+++ plainbox-gui/canonical-driver-test-suite/testitemmodel.cpp 2014-03-26 18:56:08 +0000
@@ -178,9 +178,18 @@
                         description = variant.toString();
                     }

- variant = *iface->properties.find("partial_id");
- if (variant.isValid() && variant.canConvert(QMetaType::QString) ) {
- testname = variant.toString();
+ variant = *iface->properties.find("summary");
+ QVariant test_id;
+ test_id = *iface->properties.find("id");
+ if (variant.isValid() && variant.canConvert(QMetaType::QString) &&
+ test_id.isValid() && test_id.canConvert(QMetaType::QString)) {
+ if (variant.toString().compare(test_id.toString()) != 0){
+ testname = variant.toString();
+ } else {
+ variant = *iface->properties.find("partial_id");
+ if (variant.isValid() && variant.canConvert(QMetaType::QString))
+ testname = variant.toString();
+ }
                     }

                     variant = *iface->properties.find("requires");