action #40094
closed[functional][u] xscreensaver disabled in updates_packagekit_gpk.pm is back in x11 tests
0%
Description
'xscreensaver-command -exit' done in tests/update/updates_packagekit_gpk.pm#L29 is lost (apparently) after a relogin (at least in LXDE): https://openqa.opensuse.org/tests/739615#step/midori/17
We should find a way to disable it for good.
Possible solutions are:
1) add 'xscreensaver-command -exit' also in tests/x11/user_gui_login.pm
2) disable it for good in tests/update/updates_packagekit_gpk.pm (mask a service, change the configuration, remove the package, ...)
Updated by okurz over 6 years ago
- Subject changed from xscreensaver disabled in updates_packagekit_gpk.pm is back in x11 tests to [functional][u] xscreensaver disabled in updates_packagekit_gpk.pm is back in x11 tests
- Category set to Bugs in existing tests
- Target version set to Milestone 20
I guess the best solution is finally and after all make the screensaver check explicit, e.g. in consoletest_finish only detect anything graphical, locked or not, and in an explicit module detect the screensaver lock, unlock it once, disable it for good and then no further test module should care about a potentially locked screen
Updated by okurz over 6 years ago
- Related to action #39827: [sle][functional][y] test fails in nis_client - gnome screen lock was not turned off added
Updated by ggardet_arm over 6 years ago
Not sure if consoletest_finish is the best place to do that, because we have graphical tests before consoletest_finish, e.g. updates_packagekit_gpk, keymap_or_locale.
Anyway, we must disable lock screen for good.
Updated by okurz about 6 years ago
This is an autogenerated message for openQA integration by the openqa_review script:
This bug is still referenced in a failing openQA test: lxde
https://openqa.opensuse.org/tests/758433
Updated by okurz about 6 years ago
https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/5808 disables screensaver on lxde already. I guess this should be made consistent for all DEs.
Updated by ggardet_arm about 6 years ago
I agree it should be made consistent across DEs, but this is not working properly yet, since screen is still locked on glxgears.
Maybe we should just use the gfx interface for all DEs, as done for KDE.
Updated by ggardet_arm about 6 years ago
The problem was it was disabled in system config file, but there was also a user config file. So, disable it there too.
MR: https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/5828
Updated by okurz about 6 years ago
- Target version changed from Milestone 20 to future
Updated by okurz almost 6 years ago
- Related to action #40490: [qe-core][functional] Implement test for screensaver and disable it in other scenarios added
Updated by SLindoMansilla almost 5 years ago
- Status changed from New to Rejected
- Assignee set to SLindoMansilla
Link already cleaned up.
I think this is already handled by ensure_unlocked_desktop