Bug #9344
Sometimes Seahorse's Find keys dialog cannot be found during tests
100%
Description
The failure log is:
Scenario: Fetching OpenPGP keys using Seahorse via the Tails OpenPGP Applet should work and be done over Tor. # features/torified_gnupg.feature:30
When I fetch the "10CC5BC7" OpenPGP key using Seahorse via the Tails OpenPGP Applet # features/step_definitions/torified_gnupg.rb:97
FindFailed: can not find SeahorseFindKeysWindow.png on the screen.
It happens before any network interaction, so it’s purely a Sikuli vs GUI issue.
Subtasks
Related issues
Blocked by Tails - |
Resolved | 2015-05-07 |
History
#1 Updated by anonym 2015-05-05 14:29:49
- blocks #8538 added
#2 Updated by kytv 2015-05-15 17:59:40
- Status changed from Confirmed to In Progress
I’ve now seen this in two different situations:
- menu selection failure (looks like
Bug #8928, I can’t reproduce this in my nested VM withunclutter
disabled). I solved this in my workspace. - the menu selection is successful, the find window appears, but then it quickly disappears causing the test to fail. (Of course this was seen when I wasn’t capturing video.)
As I did for the problem of the keyboard layout not being activated when selected at the greeter (Bug #7912) I’ll try reproducing the second bullet point with a custom test scenario.
#3 Updated by kytv 2015-05-27 22:39:19
- Assignee changed from kytv to anonym
- % Done changed from 0 to 20
- QA Check set to Ready for QA
- Feature Branch set to kytv:test/9344-seahorse-findkey
#4 Updated by kytv 2015-05-27 22:39:40
- blocked by
Bug #9355: More robust accessing the gpgApplet in the test suite added
#5 Updated by anonym 2015-05-28 15:30:30
- Assignee changed from anonym to kytv
- % Done changed from 20 to 40
- QA Check changed from Ready for QA to Dev Needed
First of all, I like the addition of the OpenPGPKeyserverCommunicationError
exception. I may be that we at some point start to classifying transient network errors like this and then (lower down) make retries when we catch such exceptions (see Bug #9478).
+def menu_click_helper(main, sub, verify = nil)
Defining it like this (which we do all over the place, so no harm there) will add the function to the global namespace, and it is a bit to generic (=> risk for collisions) for that IMHO. What about seahorse_menu_click_helper
(similar to the seahorse_wait_helper
you add later)? Related ticket: Feature #9030.
Otherwise, the tests runs great for me, and the changes make sense. I’ll merge once you’ve done the rename.
#6 Updated by anonym 2015-05-28 15:42:07
anonym wrote:
> Defining it like this (which we do all over the place, so no harm there) will add the function to the global namespace, and it is a bit to generic (=> risk for collisions) for that IMHO. What about seahorse_menu_click_helper
(similar to the seahorse_wait_helper
you add later)? Related ticket: Feature #9030.
Also, please move it into features/step_definitions/encryption.rb
.
#7 Updated by kytv 2015-05-28 18:55:29
- Subject changed from Sometime Seahorse's Find keys dialog cannot be found during tests to Sometimes Seahorse's Find keys dialog cannot be found during tests
- Assignee changed from kytv to anonym
- QA Check changed from Dev Needed to Ready for QA
Renames done here and Bug #9355.
#8 Updated by kytv 2015-06-02 11:48:41
Applied in changeset commit:ddd58e9a37bdf97ae499d3c574a354851e615353.
#9 Updated by anonym 2015-06-02 11:48:44
- Status changed from In Progress to Fix committed
- % Done changed from 40 to 100
Applied in changeset commit:621fa0a6e47886eb9fb0397e19d96fb014561d87.
#10 Updated by anonym 2015-06-02 12:05:49
- Assignee deleted (
anonym) - QA Check changed from Ready for QA to Pass
#11 Updated by intrigeri 2015-07-03 03:40:02
- Status changed from Fix committed to Resolved