Skip to content
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

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

Open
eugene-yujinwu opened this issue Mar 18, 2024 · 1 comment
Open
Assignees
Labels
bug Something isn't working

Comments

@eugene-yujinwu
Copy link
Contributor

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

@eugene-yujinwu eugene-yujinwu added the bug Something isn't working label Mar 18, 2024
Copy link

Thank you for reporting us your feedback!

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

This message was autogenerated

eugene-yujinwu added a commit to eugene-yujinwu/checkbox that referenced this issue Mar 20, 2024
…_suspend_open (bugfix) (canonical#1080)

- Include a note to ensure the test is executed at the appropriate time
- Fix the typo in the test order number
- Remove functionally similar test cases power-management/lid from the test plan
- Adjust the place of test case to avoid running it just after the suspend case
eugene-yujinwu added a commit to eugene-yujinwu/checkbox that referenced this issue Mar 22, 2024
…_suspend_open (bugfix) (canonical#1080)

- Include a note to ensure the test is executed at the appropriate time
- Fix the typo in the test order number
- Adjust the place of test case to avoid running it just after the suspend case
@eugene-yujinwu eugene-yujinwu self-assigned this Mar 29, 2024
eugene-yujinwu added a commit to eugene-yujinwu/checkbox that referenced this issue May 24, 2024
…_suspend_open (bugfix) (canonical#1080)

- Include a note to ensure the test is executed at the appropriate time
- Fix the typo in the test order number
- Remove functionally similar test cases power-management/lid from the test plan
- Adjust the place of test case to avoid running it just after the suspend case
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant