touchpad_test shouldn't change scroll method unless specified
Bug #1259987 reported by
Daniel Manrique
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Checkbox |
Fix Released
|
Undecided
|
Daniel Manrique |
Bug Description
The touchpad test has code to force horizontal scrolling and to change the scrolling method to edge scrolling. This makes it useless for testing e.g. two-finger scrolling. See the run() and quit() methods in the script.
Instead, it should only enable edge-scrolling if a specific switch (i.e. --edge-scrolling) is passed, otherwise it should leave the scroll-method setting alone.
None of the tests we have explicitly says it tests edge scrolling; instead, our tests generically verify that the touchpad is able to scroll in any way.
Related branches
lp:~roadmr/checkbox/1259987-scroll-method
- Chris Gregan (community): Approve
- Checkbox Developers: Pending requested
-
Diff: 81 lines (+13/-6)3 files modifiedcheckbox-old/debian/changelog (+3/-0)
checkbox-old/jobs/touchpad.txt.in (+2/-2)
checkbox-old/scripts/touchpad_test (+8/-4)
Changed in checkbox: | |
assignee: | nobody → Daniel Manrique (roadmr) |
status: | New → In Progress |
Changed in checkbox: | |
status: | In Progress → Fix Committed |
Changed in checkbox: | |
milestone: | none → 2013-dec-20 |
status: | Fix Committed → Fix Released |
To post a comment you must log in.