commit 13b515bb06c1dea9666026597ae4e3f387a46b1a
author Wei-Han Chen <stimim@google.com> Fri Feb 26 03:46:29 2016
committer chrome-bot <chrome-bot@chromium.org> Fri Feb 26 16:09:51 2016
tree 0cc18b6277db4ff8539edd2254cd8a125eae87b0
parent fef5de9c27508b3519f0a04791ff9c59256f29e5

goofy: correctly receive new return value of BuildAllTestLists

BuildAllTestLists now returns a tuple, test_lists and failed_files.
goofy/invocation.py and tools/factory.py didn't reflect the change.
This CL fix this bug.

TEST=make test, manual test
BUG=None

Change-Id: I20ec89b99c1c96d0c68ab48121a4b2e83c2ab49c
Reviewed-on: https://chromium-review.googlesource.com/329440
Commit-Ready: Wei-Han Chen <stimim@chromium.org>
Tested-by: Wei-Han Chen <stimim@chromium.org>
Reviewed-by: Joel Kitching <kitching@chromium.org>

py/goofy/invocation.py

diff --git a/py/goofy/invocation.py b/py/goofy/invocation.py
index aeefa44..2ed545a 100755
--- a/py/goofy/invocation.py
+++ b/py/goofy/invocation.py
@@ -171,7 +171,7 @@
       # It's a path pointing to an old-style test list; use it.
       return factory.read_test_list(self.test_list)
     else:
-      all_test_lists = BuildAllTestLists(
+      all_test_lists, _ = BuildAllTestLists(
           force_generic=(self.automation_mode is not None))
       test_list = all_test_lists[self.test_list]
       if isinstance(test_list, OldStyleTestList):

py/tools/factory.py

diff --git a/py/tools/factory.py b/py/tools/factory.py
index 52c1204..634bae2 100755
--- a/py/tools/factory.py
+++ b/py/tools/factory.py
@@ -314,7 +314,7 @@
       print test_lists.GetActiveTestListId()
 
     if self.args.list:
-      all_test_lists = test_lists.BuildAllTestLists(force_generic=True)
+      all_test_lists, _ = test_lists.BuildAllTestLists(force_generic=True)
       active_id = test_lists.GetActiveTestListId()
 
       line_format = '%-8s %-20s %s'