Code review comment for lp:~facundo/magicicada-gui/show-md

Revision history for this message
Facundo Batista (facundo) wrote :

I just make it to return the DBus types to the GUI.

Naty does not want it, but I thought duck typing should work... even thinking that, I tried to make tests to check that the types were converted, but...

>>> isinstance(dbus.String(u'a'), unicode)
True

and

>>> isinstance(dbus.Dictionary({dbus.String('a'): dbus.String(3)}), dict)
True

So, actually, it would be too tough and too fake to assert that, and it should work rightaway.

« Back to merge proposal