Regression in "I synchronize keys in Seahorse" recovery_proc
@@ -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
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)