Bug #10783

Test that clicks the roadmap URL in Pidgin is fragile

Added by intrigeri 2015-12-21 05:18:33 . Updated 2016-03-08 19:03:04 .

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Test suite
Target version:
Tails_2.2
Start date:
2015-12-21
Due date:
% Done:

100%

Feature Branch:
test/10783-pidgin-roadmap-url
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Instant Messaging
Deliverable for:
270

Description

End result: the “inserting text” mouse cursor (vertical bar) is on the line after the roadmap URL (“If the website has no answer […]”), vertically aligned between the “o” and the first “a” of the “roadmap” word that’s on the line above. And Tor Browser never opens.

My guess: there is a race between when Sikuli finds the “roadmap” link and when it clicks it, because at this time the topic text is still being displayed, scrolling line after line.

Subtasks

History

#1 Updated by intrigeri 2016-02-13 04:07:05

  • Assignee changed from kytv to anonym
  • Target version changed from Tails_2.4 to Tails_2.2
  • QA Check set to Ready for QA

I’ve seen it again today. I’m pretty sure this would be workaround’ed by:

--- a/features/pidgin.feature
+++ b/features/pidgin.feature
@@ -67,7 +67,8 @@ Feature: Chatting anonymously using Pidgin
     When I type "/topic"
     And I press the "ENTER" key
     Then I see the Tails roadmap URL
-    When I click on the Tails roadmap URL
+    When I wait 10 seconds
+    And I click on the Tails roadmap URL
     Then the Tor Browser has started and loaded the Tails roadmap
     And the "irc.oftc.net" account only responds to PING and VERSION CTCP requests

I know that we want to remove these ugly sleep:s, but right now I just want it to work, somehow, and we can make it nice later (something that doesn’t work cannot possibly be nice).

#2 Updated by anonym 2016-02-20 13:56:09

I filed a duplicate of this one (Bug #10995, now closed) and pushed a WIP branch (test/10995-robust-pidgin-url-clicking) with the idea of PMing the URL to yourself. I’ll look at that one too.

#3 Updated by anonym 2016-02-20 13:56:29

  • Target version changed from Tails_2.2 to Tails_2.4

#4 Updated by intrigeri 2016-02-20 14:02:48

  • Status changed from Confirmed to In Progress
  • Assignee changed from anonym to intrigeri
  • Target version changed from Tails_2.4 to Tails_2.2
  • QA Check deleted (Ready for QA)
  • Feature Branch set to test/10783-pidgin-roadmap-url

#5 Updated by intrigeri 2016-02-20 14:48:16

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

#6 Updated by anonym 2016-03-01 19:59:37

  • 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

I have reluctantly merged this. Reluctantly! :)

(On a more serious note, I agree; we need robustness even if that means more static sleep:s.)

#7 Updated by anonym 2016-03-08 19:03:04

  • Status changed from Fix committed to Resolved