Autopilot proxy types should use pythonic reprs
Bug #1237039 reported by
Thomi Richards
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Autopilot |
Fix Released
|
High
|
Unassigned | ||
autopilot (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Saucy |
Won't Fix
|
Undecided
|
Unassigned |
Bug Description
The types used in a proxy class all derive from dbus type classes, but we should override their __repr__ methods, to make them use more pythonic representations. This is to assertion failures like this:
MismatchError: After 35.0 seconds test on Label.text failed: u'00:00:25' != dbus.String(u'', variant_level=1)
instead look like this:
MismatchError: After 35.0 seconds test on Label.text failed: u'00:00:25' != u''
Related branches
lp:~thomir-deactivatedaccount/autopilot/trunk-fix-type-reprs
- Christopher Lee (community): Approve
- PS Jenkins bot: Approve (continuous-integration)
-
Diff: 205 lines (+94/-11)4 files modifiedautopilot/introspection/types.py (+64/-5)
autopilot/tests/functional/__init__.py (+1/-2)
autopilot/tests/unit/test_types.py (+24/-3)
debian/control (+5/-1)
Changed in autopilot: | |
assignee: | nobody → Thomi Richards (thomir) |
Changed in autopilot: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fix committed into lp:autopilot at revision 351, scheduled for release in autopilot, milestone 1.4