Regression in "I synchronize keys in Seahorse" recovery_proc

889411fdc54afbbd56ba4ef4858806d13690d220 did:

@@ -120,7 +117,7 @@ Then /^I synchronize keys in Seahorse$/ do
     if @screen.exists('GnomeCloseButton.png') || !$vm.has_process?('seahorse')
       step 'I kill the process "seahorse"' if $vm.has_process?('seahorse')
       debug_log('Restarting Seahorse.')
-      start_or_restart_seahorse(withapplet = @withgpgapplet)
+      start_or_restart_seahorse(withgpgapplet)
     end
   end

… but withgpgapplet is not defined in this scope. I think this was exactly why kytv used something with broader scope than a local variable in the first place.

In practice, I think it means that if the initial sync fails, and we meant to test with Seahorse started from the OpenPGP applet, then the retries will be run from a Seahorse started differently, which is not what we want to test in this case.

Feature Branch: test/10782-restart-seahorse-in-same-manner

Parent Task: #7563 (closed)

Original created by @intrigeri on 10782 (Redmine)