From: Patrik Flykt <[email protected]>
Fixes BMC#25723
---
Please review,
Patrik
test/test-connman | 19 ++++++++++++++-----
1 file changed, 14 insertions(+), 5 deletions(-)
diff --git a/test/test-connman b/test/test-connman
index 67b0c85..293bd03 100755
--- a/test/test-connman
+++ b/test/test-connman
@@ -145,28 +145,35 @@ elif sys.argv[1] in ["remove"]:
print "%s: %s" % (error._dbus_error_name, error.message)
elif sys.argv[1] == "scan":
- if len(sys.argv) > 2:
+ if len(sys.argv) == 3:
path = "/net/connman/technology/" + sys.argv[2]
technology = dbus.Interface(bus.get_object("net.connman", path),
"net.connman.Technology")
technology.Scan()
+ else:
+ print "'%s' takes two arguments" % sys.argv[1]
elif sys.argv[1] == "enable":
- if len(sys.argv) > 2:
+ if len(sys.argv) == 3:
path = "/net/connman/technology/" + sys.argv[2]
technology = dbus.Interface(bus.get_object("net.connman", path),
"net.connman.Technology")
technology.SetProperty("Powered", True)
+ else:
+ print "'%s' takes two arguments" % sys.argv[1]
elif sys.argv[1] == "disable":
- if len(sys.argv) > 2:
+ if len(sys.argv) == 3:
path = "/net/connman/technology/" + sys.argv[2]
technology = dbus.Interface(bus.get_object("net.connman", path),
"net.connman.Technology")
technology.SetProperty("Powered", False)
+ else:
+ print "'%s' takes two arguments" % sys.argv[1]
+
elif sys.argv[1] in ["offlinemode", "flightmode"]:
- if len(sys.argv) > 2:
+ if len(sys.argv) == 3:
if sys.argv[2] == "on":
active = dbus.Boolean(1)
elif sys.argv[2] == "off":
@@ -174,9 +181,11 @@ elif sys.argv[1] in ["offlinemode", "flightmode"]:
else:
active = dbus.Boolean(sys.argv[2])
manager.SetProperty("OfflineMode", active)
- else:
+ elif len(sys.argv) == 2:
properties = manager.GetProperties()
print "Offline mode is %s" % (properties["OfflineMode"])
+ else:
+ print "'%s' takes max. two arguments" % sys.argv[1]
else:
print "Unknown command"
--
1.7.10.4
_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman
