Project

General

Profile

Bug #16969

"Electrum starts" test step is broken on Buster

Added by intrigeri 3 months ago. Updated about 1 month ago.

Status:
In Progress
Priority:
Normal
Assignee:
-
Category:
Test suite
Target version:
-
Start date:
Due date:
% Done:

0%

Feature Branch:
Type of work:
Code
Blueprint:
Starter:
Affected tool:
Electrum

Description

I've been so used to see the "Using a persistent Electrum configuration" fail since months that I did not notice that it failed much earlier on Buster than on Stretch:

  • on Stretch, it fails to connect to the Electrum servers (known issue wrt. the version we're shipping)
  • on Buster, it fails at the "Electrum starts" step: Dogtail finds no "electrum" application when the "Electrum - Install Wizard" dialog is displayed; I've quickly tried the obvious, i.e. asking Dogtail to look for "electrum" and "Electrum - Install Wizard" instead, but none of these work

recommended-packages.txt View (4.66 KB) segfault, 10/04/2019 11:33 AM


Related issues

Related to Tails - Bug #16281: Update the test suite for Buster Resolved 01/05/2019
Related to Tails - Bug #9732: Orca cannot work with Electrum Confirmed 07/13/2015
Blocks Tails - Feature #16209: Core work: Foundations Team Confirmed
Blocked by Tails - Bug #16421: Electrum Phishing Attack - Upstream Fix Committed Resolved 02/05/2019

Associated revisions

Revision 3bba768b (diff)
Added by segfault about 2 months ago

Revert "Test suite: remove totally broken Electrum scenario" (refs: #16969)

This reverts commit f092b0d6f268a12550283e3a510f0455055ca1d9.

We have a working Electrum version again.

Revision 80d04f8f (diff)
Added by intrigeri about 1 month ago

Revert "Revert "Test suite: remove totally broken Electrum scenario" (refs: #16969)"

This reverts commit 3bba768b940f498d66215a16c4c4c95f98d62be6.

Unfortunately, this test is still broken on Buster as Dogtail can't see
Electrum's widgets, and even ORCA can only read part of its interface.

History

#1 Updated by intrigeri 3 months ago

#2 Updated by intrigeri 3 months ago

  • Blocked by Bug #16421: Electrum Phishing Attack - Upstream Fix Committed added

#3 Updated by intrigeri 3 months ago

  • Assignee deleted (anonym)
  • Priority changed from Elevated to Normal
  • Target version deleted (Tails_4.0)

As per discussion on the CI list, let's not bother until we have a working Electrum (#16421). Meanwhile, the affected scenario will be removed.

#4 Updated by intrigeri 3 months ago

  • Parent task deleted (#16281)

#5 Updated by intrigeri 3 months ago

  • Related to Bug #16281: Update the test suite for Buster added

#6 Updated by intrigeri 2 months ago

  • Target version set to Tails_4.0

#7 Updated by segfault about 2 months ago

  • Status changed from Confirmed to In Progress

#8 Updated by segfault about 2 months ago

  • Assignee set to segfault
  • Feature Branch set to bugfix/16421-fix-electrum

#9 Updated by segfault about 1 month ago

The "Electrum starts" step fails because Electrum is not visible to Dogtail. It doesn't seem to register with the accessibility interface.

#10 Updated by segfault about 1 month ago

It seems like all qt applications are not registering with the accessibility interface. Affected are: electrum, keepassxc, onionshare.

I tried the same on a Debian sid and there I can see those applications in Dogtail - but the Electrum app is called "python3" and doesn't contain any information about its widgets.

#11 Updated by segfault about 1 month ago

I built Tails with recommended packages [1], as suggested by intrigeri, and now I see the same as on Sid: The Electrum window is displayed in Dogtail's sniff tool as "python3" with no child nodes. So the tests still won't work.

[1] FTR, I had to delete those local-hooks to get Tails to build with --apt-recommends true in auto/config:

config/chroot_local-hooks/04-change-gids-and-uids
config/chroot_local-hooks/05-adduser_tails-persistence-setup
config/chroot_local-hooks/06-adduser_clearnet
config/chroot_local-hooks/06-adduser_htp
config/chroot_local-hooks/06-adduser_tails-iuk-get-target-file
config/chroot_local-hooks/06-adduser_tails-upgrade-frontend
config/chroot_local-hooks/06-adduser_tor-launcher
config/chroot_local-hooks/07-adduser_tails-install-iuk
config/chroot_local-hooks/99-zzz_check_uids_and_gids

#12 Updated by intrigeri about 1 month ago

and now I see the same as on Sid: The Electrum window is displayed in Dogtail's sniff tool as "python3" with no child nodes. So the tests still won't work.

On my sid (GNOME 3.34 + Wayland session), ORCA can read the Electrum UI.
But indeed, with Dogtail's sniff tool I see the as you.
So it seems that Electrum is accessible for users who need this, but somehow it's incompatible with Dogtail?

#13 Updated by segfault about 1 month ago

The Electrum window is displayed in Dogtail's sniff tool as "python3" with no child nodes

Same for OnionShare. KeepassXC shows up as "keepassxc" but also without child nodes.

I didn't find out yet which package resulted in at least something showing up for Qt apps in Dogtail - and I don't want to spend a lot of time on this, since the result doesn't seem to be useful. I don't see any obvious candidates in the diff of the packages installed in my old build and the one with --apt-recommends true. I'm attaching this list.

#14 Updated by segfault about 1 month ago

segfault wrote:

I didn't find out yet which package resulted in at least something showing up for Qt apps in Dogtail

It's qt5-gtk-platformtheme

#15 Updated by segfault about 1 month ago

  • Assignee deleted (segfault)

#16 Updated by intrigeri about 1 month ago

  • Related to Bug #9732: Orca cannot work with Electrum added

#17 Updated by intrigeri about 1 month ago

  • Feature Branch deleted (bugfix/16421-fix-electrum)

#18 Updated by intrigeri about 1 month ago

  • Target version deleted (Tails_4.0)

This was added to 4.0 because we thought that it would be easier to fix once we upgraded Electrum on #16421. Unfortunately, that's not the case, and given the amount of other work left to do for 4.0, I don't think we can reasonably treat this as a release blocker.

Also available in: Atom PDF