commit 6a89b51be457598aea5ca97a4f5f9a47af537885
author Yu-Ju Hong <yjhong@chromium.org> Fri Jan 10 16:23:59 2014 -0800
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Sat Jan 11 18:48:17 2014 +0000
tree 5716b10b1667591986fa6efa6d2ff8813847efd5
parent 29111983c9b9010879f8da3e60fa7d8889afdba4

Fix AU test getting incorrect devserver URL

AU test should use keyword arguments (e.g. port, sub_dir) when
calling dev_server_wrapper.DevServerWrapper methods.

BUG=chromium:321684
CQ-DEPEND=CL:181361
TEST=x86-mario-paladin (vmtest) trybot

Change-Id: Ied965a8f8788f6c625d19e28f677dfcadfee6521
Reviewed-on: https://chromium-review.googlesource.com/182240
Reviewed-by: Yu-Ju Hong <yjhong@chromium.org>
Tested-by: Yu-Ju Hong <yjhong@chromium.org>
Commit-Queue: Yu-Ju Hong <yjhong@chromium.org>

au_test_harness/au_worker.py

diff --git a/au_test_harness/au_worker.py b/au_test_harness/au_worker.py
index 981cde5..7b2c3c6 100644
--- a/au_test_harness/au_worker.py
+++ b/au_test_harness/au_worker.py
@@ -192,7 +192,7 @@
     cache_path = self.update_cache.get(update_id)
     if cache_path:
       update_url = dev_server_wrapper.DevServerWrapper.GetDevServerURL(
-          proxy_port, cache_path)
+          port=proxy_port, sub_dir=cache_path)
       cmd.append('--update_url=%s' % update_url)
     else:
       raise update_exception.UpdateException(

au_test_harness/cros_au_test_harness.py

diff --git a/au_test_harness/cros_au_test_harness.py b/au_test_harness/cros_au_test_harness.py
index 014884b..54bee7f 100755
--- a/au_test_harness/cros_au_test_harness.py
+++ b/au_test_harness/cros_au_test_harness.py
@@ -215,7 +215,7 @@
       # Only start a devserver if we'll need it.
       if update_cache:
         my_server = dev_server_wrapper.DevServerWrapper(
-            options.test_results_root)
+            log_dir=options.test_results_root)
         my_server.Start()
 
       if options.type == 'vm' and options.parallel: