Closed Hook25 closed 1 month ago
Proof of this working on the original hardware in the lab:
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 47.56%. Comparing base (
17ceef1
) to head (5a432d2
). Report is 10 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
When running the touchscreen tests via remote, the user is promted to rotate the rectangle on a window but the
user-interact-verify
job (silently) fails to start the window at all. When the stderr is un-silenced, the following error is printed in the console:Timer is a builtin type of QT and this happens for any other type (removing the timer just makes it fail on the next builtin type).
After much investigation, and thanks to the reported fact that the test starts fine if launched from the
checkbox.shell
or locally, I found that the problem is the missing envvar in this PR. This fixes the propagation of the envvar. I have whitelisted it instead of explicitly requiring it in the test because I believe that one doesn't have to understand exactly how QT works when writing a test, so they don't need to reason about which QT envvars are needed.Resolved issues
Fixes: CHECKBOX-1511 Fixes: https://github.com/canonical/checkbox/issues/1371
Documentation
N/A
Tests
I have built and ran the test via remote with the following launcher. The window now shows up even via remote when before it did not.
To test it yourself use this checkbox24 snap, built from this branch (or build it yourself on arm64):