On Tue, 2011-04-19 at 13:43 -0700, Eric Li(李咏竹) wrote:
> Fix unit test for server/autotest.py for change @5317.

Oops, missed that out (wonder why, my patch checker should've caught
it). Anyway, here it is:

http://autotest.kernel.org/changeset/5323

Please consider using git to generate your patches (this one was a -p0,
not a -p1, which makes it hard for automated checkers to use - maybe
that's why your previous change didn't warn me about the unit test
failure). Any method to generate a -p1 patch would be OK as well.

> Index: server/autotest_unittest.py
> ===================================================================
> --- server/autotest_unittest.py (revision 5316)
> +++ server/autotest_unittest.py (working copy)
> @@ -205,7 +205,7 @@
>  
>          c = autotest.global_config.global_config
>          c.get_config_value.expect_call("PACKAGES",
> -            'fetch_location', type=list).and_return(['repo'])
> +            'fetch_location', type=list,
> default=[]).and_return(['repo'])
>          pkgmgr = packages.PackageManager.expect_new('autotest',
> 
> repo_urls=['repo'],
> 
> hostname='hostname')
> 
> 
> -- 
> Eric Li
> 李咏竹
> Google Kirkland
> 
> 
> 


_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to