-
Notifications
You must be signed in to change notification settings - Fork 270
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] Ensure unlocked desktop #6729
[WIP] Ensure unlocked desktop #6729
Conversation
SergioAtSUSE
commented
Feb 6, 2019
- Related ticket: https://progress.opensuse.org/issues/45710
- Verification run: http://openqa.slindomansilla-vm.qa.suse.de/tests/1219
dff040e
to
8929826
Compare
lib/x11utils.pm
Outdated
# if stay under tty console for long time, then check | ||
# screen lock is necessary when switch back to x11 | ||
# all possible options should be handled within loop to get unlocked desktop | ||
# When there is an application openede, generic-desktop won't match, | ||
# you can pass the tag name as parameter C<$needle_tag> to match that open application |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When turning into pod please remove the leading #. Also typo in openede
lib/x11utils.pm
Outdated
sub ensure_unlocked_desktop { | ||
my $needle_tag = shift @_; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you mind using my ($needle_tag) = @_;
?
@@ -35,6 +36,7 @@ sub run { | |||
# needle cannot match so we rely on the assert_and_click from | |||
# connect_to_network | |||
select_console 'x11', await_console => 0; | |||
ensure_unlocked_desktop; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't you want to call it with the parameter here?
We just type a few lines within a minute and return to the x11 session. The session shouldn't even be locked. And I doubt it's a good idea we sprinkle in seemingly random "ensure_unlocked_desktop" everywhere. Seems you have missed all my suggestions in the ticket description. |