[PATCH 09/15] WPS: service's wps property support in test scripts.

Samuel Ortiz sameo at linux.intel.com
Fri Jan 21 16:12:23 PST 2011


Hi Tomasz,

Some nitpicking:

On Tue, Jan 11, 2011 at 03:54:02PM +0200, Tomasz Bursztyka wrote:
> ---
>  test/get-services  |    2 +-
>  test/list-services |    2 +-
>  test/test-manager  |    2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/test/get-services b/test/get-services
> index 29d4ea0..605e278 100755
> --- a/test/get-services
> +++ b/test/get-services
> @@ -43,7 +43,7 @@ for entry in services:
>  					"Domains", "Domains.Configuration"]:
>  			val = extract_list(properties[key])
>  		elif key in ["Favorite", "Immutable", "AutoConnect",
> -				"SetupRequired", "PassphraseRequired"]:
> +				"SetupRequired", "PassphraseRequired", "WPS"]:
>  			if properties[key] == dbus.Boolean(1):
>  				val = "true"
>  			else:
> diff --git a/test/list-services b/test/list-services
> index 66dfe5c..f1b89b6 100755
> --- a/test/list-services
> +++ b/test/list-services
> @@ -49,7 +49,7 @@ for path in properties["Services"]:
>  			val = extract_list(properties[key])
>  		elif key in ["Favorite", "Immutable", "AutoConnect",
>  					"LoginRequired", "SetupRequired",
> -							"PassphraseRequired"]:
> +						"PassphraseRequired", "WPS"]:
>  			if properties[key] == dbus.Boolean(1):
>  				val = "true"
>  			else:
> diff --git a/test/test-manager b/test/test-manager
> index b5642d0..9f35105 100755
> --- a/test/test-manager
> +++ b/test/test-manager
> @@ -56,7 +56,7 @@ def print_properties(key, value):
>  					"Available", "Remember", "Default",
>  					"Favorite", "Immutable", "AutoConnect",
>  						"LoginRequired", "SetupRequired",
> -							"PassphraseRequired"]:
> +							"PassphraseRequired", "WPS"]:
>
more than 80 chars here.

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/



More information about the connman mailing list