From f6cd8f06f46ae70e18779de476f23e9c7909eb0a Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Mon, 5 Jan 2015 13:50:56 +0100 Subject: [PATCH] libnm/tests: fix GetAccessPoints() in test-networkmanager-service.py for hidden APs There was a bug checking for "if a.ssid():". Refactor the whole line to use a list comprehension. (cherry picked from commit 954c744bc0ac1b3260c0dfedb4ee7da8c29cb964) --- tools/test-networkmanager-service.py | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/tools/test-networkmanager-service.py b/tools/test-networkmanager-service.py index c3c2347138..50803c8871 100755 --- a/tools/test-networkmanager-service.py +++ b/tools/test-networkmanager-service.py @@ -356,11 +356,7 @@ class WifiDevice(Device): @dbus.service.method(dbus_interface=IFACE_WIFI, in_signature='', out_signature='ao') def GetAccessPoints(self): # only include non-hidden APs - array = [] - for a in self.aps: - if a.ssid(): - array.append(a) - return to_path_array(array) + return to_path_array([a for a in self.aps if a.ssid]) @dbus.service.method(dbus_interface=IFACE_WIFI, in_signature='', out_signature='ao') def GetAllAccessPoints(self):