canonical / checkbox

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

Simplify check-display test case (New) #1279

Closed pseudocc closed 3 months ago

pseudocc commented 3 months ago

Description

Simplify code.

Resolved issues

Documentation

Tests

xanthein commented 3 months ago

test job report that uwake can't be found with your modification

sudo: uwake: command not found
Has disaplay attached and screen on
/sys/class/drm/card0-eDP-1/status:connected
/sys/class/drm/card0-eDP-1/dpms:On
pseudocc commented 3 months ago

Perhaps, needs #1292 first.

xanthein commented 3 months ago

With plainbox-provider-pc-sanity 4.0.0~dev313~ubuntu22.04.1 which include the fix in #1292. Test job will still report command not found

=========[ Running job 1 / 1. Estimated time left (at least): 0:00:00 ]=========
---------------[ Check if display is available for the machine ]----------------
ID: com.canonical.certification::miscellanea/check-display_All In One
Category: com.canonical.plainbox::miscellanea
... 8< -------------------------------------------------------------------------
sudo: uwake: command not found
Has disaplay attached and screen on
/sys/class/drm/card0-eDP-1/status:connected
/sys/class/drm/card0-eDP-1/dpms:On
pseudocc commented 3 months ago

Okay, ignore me.