Merge lp:~sylvain-pineau/checkbox/937715_submit_disabled_at_startup into lp:checkbox

[Sylvain Pineau]

This MR tries to solve a new feature only available with the Qt UI. The ability to switch from one tab to another is quite complex to handle with checkbox events. So I prefer to disable the submission buttons until the end of the test run.

[TienFu Chen (ctf)]

Sylvain,
The behavior after the patch looks reasonable to me.
But I still see the warning text in the content of results tab.
------------
Warning: Not all tests have been run yet.
You can send the results now, but the submission won't make it to Ubuntu Friendly.
------------
After the buttons being disabled, user has no way to submit the result before all tests are finished.
For now, to prevent confusing users, should we change the texts to something like "You have to finish all the tests before you can submit the result"?

[Sylvain Pineau (sylvain-pineau)]

I think we could keep this feature (submit at any time) for the future as the Qt ui allows it. But for now and if we want the certification client to also benefit asap from the good stuff it's a good idea to simply disable the buttons. In addition to this, disabling the final tab click/navigation could be also something to implement.
I will change the warning message to match the new bahaviour. thank you.
Again I keep in mind that submit at any time could be a good addition, perhaps should we target it for 14.x ?

[Jeff Lane  (bladernr)]

Setting to needs fixing per your comments above... I agree with Tim, once the warning message is changed (hopefully that only changes it while the buttons are disabled and will revert to the standard warning after testing is done) this can probably be pushed.

I also like the idea of leaving the facility there for 14.x when we have more time to sort out the complexity and maybe get "Submit at any time" working. That could be quite useful for debugging specific tests or issues, or other uses.

review: Needs Fixing

[Jeff Lane  (bladernr)]

Per discussion w/ ara this morning, merging. The code is fine, the message can be handled elsewhere (there's already a different bug open about the warning message).

review: Approve

=== modified file 'debian/changelog'
--- debian/changelog	2012-04-03 11:38:10 +0000
+++ debian/changelog	2012-04-04 14:28:26 +0000
@@ -19,6 +19,8 @@
   * qt/frontend/qtfront.cpp, qt/frontend/qtfront.h, plugins/suites_prompt.py,
     checkbox_qt/qt_interface.py, plugins/jobs_prompt.py: The selection tree is
     now updated when recovering from a previous run (LP: #937696)
+  * qt/frontend/qtfront.cpp: Submit/View results buttons are disabled until
+    every selected test has been run (LP: #937715)
 
   [Brendan Donegan]
   * [FEATURE] Added touchpad tests from CE QA Checkbox to allow touchpad

=== modified file 'qt/frontend/qtfront.cpp'
--- qt/frontend/qtfront.cpp	2012-04-03 11:38:10 +0000
+++ qt/frontend/qtfront.cpp	2012-04-04 14:28:26 +0000
@@ -73,6 +73,9 @@
     ui->treeView->setContextMenuPolicy(Qt::CustomContextMenu);
     connect(ui->treeView, SIGNAL(customContextMenuRequested(const QPoint&)), this, SLOT(onSelectAllContextMenu(const QPoint&)));
     ui->stepsFrame->setFixedHeight(0);
+    ui->buttonSubmitResults->setEnabled(false);
+    ui->lineEditEmailAddress->setEnabled(false);
+    ui->buttonViewResults->setEnabled(false);
 
     // comment box
     ui->commentTestButton->setMenu(new QMenu());
@@ -285,6 +288,7 @@
 
     ui->buttonSubmitResults->setEnabled(true);
     ui->lineEditEmailAddress->setEnabled(true);
+    ui->buttonViewResults->setEnabled(true);
 
 }