Bug #8875

Make running applications from the GNOME menu more robust, again

Added by intrigeri 2015-02-06 09:23:31 . Updated 2015-02-24 22:42:13 .

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Test suite
Target version:
Start date:
2015-02-06
Due date:
% Done:

100%

Feature Branch:
test/8875-more-robust-apps-menu
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Deliverable for:

Description

test/8140-more-robust-apps-menu improved things, but one commit on that branch wasn’t merged. Let’s check if it’s working and merge it if yes.


Subtasks


Related issues

Related to Tails - Bug #8140: Running applications via the GNOME menu is fragile Resolved 2014-10-16
Related to Tails - Bug #8928: "unclutter" can cause spurious errors with some configurations Resolved 2015-03-03

History

#1 Updated by intrigeri 2015-02-06 09:23:49

#2 Updated by intrigeri 2015-02-06 09:23:59

  • related to Bug #8140: Running applications via the GNOME menu is fragile added

#3 Updated by intrigeri 2015-02-06 17:48:07

  • blocks #8538 added

#4 Updated by intrigeri 2015-02-07 12:57:52

  • Feature Branch set to test/8875-more-robust-apps-menu

#5 Updated by intrigeri 2015-02-07 13:55:31

  • Assignee changed from intrigeri to anonym
  • % Done changed from 10 to 50
  • QA Check set to Ready for QA

#6 Updated by kytv 2015-02-08 19:13:14

It may not completely resolve the problem but it should help, and certainly won’t hurt.

+1 on this change.

(Not sure if I can (=should) change the QA Check status so I’m leaving it alone.)

#7 Updated by kytv 2015-02-08 20:09:41

I’d like to see this added to the step below it too, e.g.

diff --git a/features/step_definitions/common_steps.rb b/features/step_definitions/common_steps.rb
index 0ec498c..19c82c1 100644
--- a/features/step_definitions/common_steps.rb
+++ b/features/step_definitions/common_steps.rb
@@ -685,6 +685,9 @@ Given /^I start "([^"]+)" via the GNOME "([^"]+)"\/"([^"]+)" applications menu$/
     prefix = 'Gnome'
   end
   @screen.wait_and_click(prefix + "ApplicationsMenu.png", 10)
+  @screen.hide_cursor
+  # Wait for the menu to be displayed, by waiting for one of its last entries
+  @screen.wait("GnomeApplicationsTails.png", 40)
   @screen.wait_and_hover(prefix + "Applications" + submenu + ".png", 20)
   @screen.wait_and_hover(prefix + "Applications" + subsubmenu + ".png", 20)
   @screen.wait_and_click(prefix + "Applications" + app + ".png", 20)

Without these changes the tests in Feature #7821 were really fragile for me.

#8 Updated by intrigeri 2015-02-09 09:34:57

kytv wrote:
> I’d like to see this added to the step below it too, e.g.

Done in the branch, now testing. Should be safe enough a change => shouldn’t block the review’n’merge process IMO.

#9 Updated by anonym 2015-02-09 13:28:10

  • Assignee changed from anonym to intrigeri
  • QA Check changed from Ready for QA to Dev Needed

kytv wrote:
> + screen.wait(“GnomeApplicationsTails.png”, 40)@

This will break the “I start the Tor Browser” step when run in windows_camouflage.feature. You need to do the prefix stuff here too, and add an appropriate image for when the camouflage is enabled.

#10 Updated by intrigeri 2015-02-09 13:45:47

> This will break the “I start the Tor Browser” step when run in
> windows_camouflage.feature. You need to do the prefix stuff here too, and add an
> appropriate image for when the camouflage is enabled.

Good catch, will fix that today.

#11 Updated by intrigeri 2015-02-09 13:58:21

  • Assignee changed from intrigeri to anonym
  • QA Check changed from Dev Needed to Ready for QA

Fixed with commit:44cd921, please review’n’merge. Thanks again!

#12 Updated by anonym 2015-02-11 10:44:35

  • Status changed from In Progress to Fix committed
  • Assignee deleted (anonym)
  • % Done changed from 50 to 100
  • QA Check changed from Ready for QA to Pass

#13 Updated by intrigeri 2015-02-24 15:16:30

  • related to Bug #8928: "unclutter" can cause spurious errors with some configurations added

#14 Updated by BitingBird 2015-02-24 22:42:13

  • Status changed from Fix committed to Resolved