Code review comment for lp:~josharenson/unity8/fix-session-icon
Revision history for this message
| Josh Arenson (josharenson) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
> Tested, seems to work well.
>
> Is this something that we can autotest?
There are some existing tests for the session icons. I think the level of effort to test this small issue would be very high, if even possible. The information for this field is delivered from AccountService via LightDM and thus, would likely require substantial updates to the mocks. Perhaps I'm overthinking this...
It may be a good candidate for an autopilot test, however. If you disagree, lets discuss in person.