xpathselect attribute matching doesn't match anything other than object Ids
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Unity |
Fix Released
|
High
|
Marco Trevisan (Treviño) | ||
7.2 |
Fix Released
|
High
|
Marco Trevisan (Treviño) | ||
unity (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
I came across this issue while testing an autopilot branch of mine.
The xpathselect server-side matching (as opposed to matching in the autopilot python code) is not working as expected.
To reproduce:
- Save this code as test_selection_
- (with the unity-autopilot package installed) run: `autopilot run test_selection_
This script contains 2 tests, one passes to prove that it can get a Launcher that has the monitor attribute of 0, the 2nd test attempts to select the Launcher object with attribute monitor=0 and fails.
We see in the unity logs:
- "Unable to match 'monitor' against property of unknown integer type."
A bit of digging suggests that GetPropertyValue is returning an array of values (Probably: Object type id, Value?) which means that, in MatchIntegerPro
Related branches
- Christopher Lee (community): Approve
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 59 lines (+24/-4)1 file modifiedunity-shared/DebugDBusInterface.cpp (+24/-4)
- Brandon Schaefer (community): Approve
-
Diff: 3275 lines (+2231/-373)31 files modifiedAUTHORS (+6/-1)
CMakeLists.txt (+1/-1)
ChangeLog (+1159/-0)
dash/DashController.cpp (+30/-25)
dash/DashController.h (+3/-5)
dash/DashView.cpp (+2/-2)
dash/ResultRendererTile.cpp (+1/-1)
debian/changelog (+41/-0)
debian/rules (+9/-5)
launcher/LauncherController.cpp (+0/-12)
lockscreen/CMakeLists.txt (+2/-0)
lockscreen/LockScreenAbstractShield.h (+5/-1)
lockscreen/LockScreenAcceleratorController.cpp (+146/-0)
lockscreen/LockScreenAcceleratorController.h (+46/-0)
lockscreen/LockScreenAccelerators.cpp (+528/-0)
lockscreen/LockScreenAccelerators.h (+94/-0)
lockscreen/LockScreenController.cpp (+14/-1)
lockscreen/LockScreenController.h (+4/-1)
lockscreen/LockScreenPanel.cpp (+4/-237)
lockscreen/LockScreenPanel.h (+2/-50)
lockscreen/LockScreenShield.cpp (+23/-6)
lockscreen/LockScreenShield.h (+2/-1)
lockscreen/LockScreenShieldFactory.cpp (+2/-2)
lockscreen/LockScreenShieldFactory.h (+2/-2)
lockscreen/UserPromptView.cpp (+2/-0)
plugins/unityshell/src/unityshell.cpp (+45/-10)
plugins/unityshell/src/unityshell.h (+1/-0)
tests/data/external.gschema.xml (+21/-0)
tests/test_lockscreen_controller.cpp (+3/-2)
unity-shared/DebugDBusInterface.cpp (+24/-4)
unity-shared/PluginAdapter.cpp (+9/-4)
Changed in unity: | |
status: | New → In Progress |
importance: | Undecided → High |
assignee: | nobody → Marco Trevisan (Treviño) (3v1n0) |
Changed in unity: | |
milestone: | none → 7.2.1 |
Changed in unity: | |
milestone: | 7.2.1 → 7.3.0 |
status: | In Progress → Fix Committed |
Changed in unity: | |
status: | Fix Committed → Fix Released |
A simplified single test that should work with multiple screens: http:// paste.ubuntu. com/7252628/