Checkbox-qt: When checkbox is running an automated test, previous test interface should not appear.

Bug #1012377 reported by Jeff Marcom
22
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Checkbox
Fix Released
Medium
Unassigned

Bug Description

Checkbox-qt: When checkbox is running an automated test, previous test interface should not appear.

Problem:
Automated test is running in the top of test window pane with progress bar, while un-related test comments/instructions/selection options are displayed in the remaining window pane area:

Example:

cpu_test is running while audio/microphone test instructions and result buttons are still present.

Steps to Reproduce: (Issue was seen with checkbox-qt interface):

1.) Start checkbox
2.) run through tests

Expected Result:

Automated tests should run in their own window, or refresh/restore active window.

Related branches

Revision history for this message
Marc Tardif (cr3) wrote :

Agreed, seeing the previous manual tests while one or more shell tests are running doesn't make sense. However, I'm not clear on the solution you propose.

You first suggest "tests should run in their own window", do you mean a popup window? If so, should a new popup window open and close for each shell test or should one popup window refresh itself as long as there are shell tests running?

You also suggest "refresh/restore active window", what should it contain?

While answering these questions, I'd appreciate if you could consider the two extreme use cases where you have few shell tests that take a lot of time or you have many shell tests that take little time. So, for a popup window that opens and closes, the user experience might just look like windows flashing by if the tests take little time.

Changed in checkbox:
status: New → Incomplete
importance: Undecided → Low
assignee: nobody → Marc Tardif (cr3)
Revision history for this message
Jeff Marcom (jeffmarcom) wrote :

Hmm, I see your point.

Would a feasible solution be to remove any instructions from the display box, and "grey-out"/lock out the radio buttons, comment area, next, and instruction area?

Changed in checkbox:
status: Incomplete → New
Ara Pulido (ara)
Changed in checkbox:
importance: Low → Medium
Changed in checkbox:
status: New → Incomplete
assignee: Marc Tardif (cr3) → nobody
Revision history for this message
Marc Tardif (cr3) wrote :

Removing the instructions from the display box might look a bit empty though. Now that descriptions are required for shell tests, we might consider showing that in the display box to give more information about the test currently running. These descriptions might flash by more clickly than users could reasonably read them though, so that might be a usability concern. It might be worth see what people think about this on the ubuntu-friendly-squad mailing list.

Revision history for this message
Jeff Marcom (jeffmarcom) wrote :

email sent. Waiting for responses.

Revision history for this message
Jeff Marcom (jeffmarcom) wrote :

based on the feedback, the proposal is to do the following:

1.) Disable all buttons from being "click-able."
2.) Remove the previous test description text and set the container of text to an inactive "grey" state.
3.) Clear the Test name area so that the previous test name is no longer shown.

A merge proposal is under way.

Changed in checkbox:
status: Incomplete → In Progress
Jeff Marcom (jeffmarcom)
Changed in checkbox:
status: In Progress → Fix Committed
Changed in checkbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.