Re: [PATCH] test: add disable-call-forwarding

[Denis Kenzior <denkenz@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 2357 bytes --]

Hi Jarko,

On 01/10/2011 06:06 AM, Jarko Poutiainen wrote:
> ---
>  Makefile.am                  |    3 +-
>  test/disable-call-forwarding |   55 ++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 57 insertions(+), 1 deletions(-)
>  create mode 100755 test/disable-call-forwarding
> 
> diff --git a/Makefile.am b/Makefile.am
> index 8ad01cd..7908467 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -440,7 +440,8 @@ test_scripts = test/backtrace \
>  		test/set-tty \
>  		test/set-gsm-band \
>  		test/set-umts-band \
> -		test/lockdown-modem
> +		test/lockdown-modem \
> +		test/disable-call-forwarding
>  
>  if TEST
>  testdir = $(pkglibdir)/test
> diff --git a/test/disable-call-forwarding b/test/disable-call-forwarding
> new file mode 100755
> index 0000000..3c51d06
> --- /dev/null
> +++ b/test/disable-call-forwarding
> @@ -0,0 +1,55 @@
> +#!/usr/bin/python
> +
> +import sys
> +import gobject
> +
> +import dbus
> +import dbus.mainloop.glib
> +
> +def property_changed(property, value):
> +

Please remove this empty line.

> +	if len(value.__str__()) > 0:
> +		print "CallForwarding property %s changed to %s" % (property, value)
> +	else:
> +		print "CallForwarding property %s changed to disabled" % (property)
> +

Please don't go over 80 characters / line here

> +	if canexit:
> +		mainloop.quit();
> +
> +if __name__ == "__main__":
> +

Please remove this empty line.

> +	if len(sys.argv) < 2:
> +		print "Usage: %s <type>" % (sys.argv[0])
> +		print "Type can be: all, conditional"
> +		sys.exit(1)
> +
> +	canexit = False
> +
> +	type = sys.argv[1]
> +
> +	dbus.mainloop.glib.DBusGMainLoop(set_as_default=True)
> +
> +	bus = dbus.SystemBus()
> +
> +	manager = dbus.Interface(bus.get_object('org.ofono', '/'),
> +							'org.ofono.Manager')
> +
> +	modems = manager.GetModems()
> +
> +	cf = dbus.Interface(bus.get_object('org.ofono', modems[0][0]),
> +				'org.ofono.CallForwarding')
> +
> +	cf.connect_to_signal("PropertyChanged", property_changed)
> +
> +	try:
> +		cf.DisableAll(type, timeout = 100)
> +	except dbus.DBusException, e:
> +		print "Unable to DisableAll", e
> +		sys.exit(1);
> +
> +	print "DisableAll successful"
> +
> +	canexit = True
> +
> +	mainloop = gobject.MainLoop()
> +	mainloop.run()

Regards,
-Denis