search

canonical / checkbox

Checkbox
https://checkbox.readthedocs.io
GNU General Public License v3.0
30 stars 47 forks source link

Some issues with test case power-management/lid_close_suspend_open #1080

Closed eugene-yujinwu closed 1 month ago

eugene-yujinwu commented 6 months ago

Bug Description

There are some issues related to the test case power-management/lid_close_suspend_open:

  1. A typo that make the test steps jump from 3 to 5.

  2. There is a HoldoffTimeoutSec which will "Specifies a period of time after system startup or system resume in which systemd will hold off on reacting to lid events....Defaults to 30s. " So if we run this test just after a resume from suspend, it will failed. Please see LP bugs: LP:2042406, LP:2047136. Maybe we need add some notes to make the tester aware of this information.

  3. This test case seems to be redundant with another test case power-management/lid. Each of them listed in the test plan after-suspend-graphics-integrated-gpu-cert-manual , so it's recursively included in the test plan client-cert-desktop-22-04-manual. Maybe we can remove one of them.

  4. A improvement suggestion from Kent: Maybe we could move this test case away from the suspend_advanced_auto test case, to avoid the situation that run this test case just after the resume from suspend.

To Reproduce

checbox-cli run com.canonical.certification::power-management/lid_close_suspend_open

Environment

checkbox: 4.0.0.dev139. Lenovo Desktop OEM image 1016-oem kernel

Relevant log output

No response

Additional context

No response

syncronize-issues-to-jira[bot] commented 6 months ago

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/CHECKBOX-1313.

This message was autogenerated

zongminl commented 1 month ago

This has been fixed by #1093