NetworkManager template: Fix type of 'State' property

Most places got it right already, but AddActiveConnection() didn't set it as unit32 but as a normal int. This caused a bunch of GLib-GIO-WARNING **: Trying to set property State of type i but according to the expected interface the type is u warnings.
martinpitt · 763d0b3ef3bb50fb990ef6bc1e3f688056f8373e · martinpitt committed Jun 19, 2017 · Jun 19, 2017 · 763d0b3
1 parent 6c351f8
commit 763d0b3ef3bb50fb990ef6bc1e3f688056f8373e
Unified Split

Showing with 5 additions and 1 deletion.

+4 −0 NEWS

+1 −1 dbusmock/templates/networkmanager.py
diff --git a/NEWS b/NEWS
@@ -1,3 +1,7 @@
+0.16.9 (UNRELEASED)
+-------------------
+ - NetworkManager template: Fix type of 'State' property in AddActiveConnection()
+
 0.16.8 (2017-06-12)
 -------------------
  - bluez5 template tests: Fix failure of test_no_adapters with BlueZ 5.45.
diff --git a/dbusmock/templates/networkmanager.py b/dbusmock/templates/networkmanager.py
@@ -577,7 +577,7 @@ def AddActiveConnection(self, devices, connection_device, specific_object, name,
                        'Master': dbus.ObjectPath('/'),
                        'SpecificObject': dbus.ObjectPath(specific_object),
                        'Uuid': conn_uuid,
-                       'State': state,
+                       'State': dbus.UInt32(state),
                    },
                    [])