test_lists: Create test_list_common.py for test list constants.
Move some test list constants, such as file paths, to a separate file,
so that we don't need to import manager.py just to get these variables.
BUG=chromium:910037, chromium:899136
TEST=make test
Change-Id: Ife293743b539a45bf50a5d3a43a83b2ee1228718
Reviewed-on: https://chromium-review.googlesource.com/1560950
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Cheng-Han Yang <chenghan@chromium.org>
Reviewed-by: Yong Hong <yhong@google.com>
diff --git a/py/toolkit/installer.py b/py/toolkit/installer.py
index 437b7e9..fd77a99 100755
--- a/py/toolkit/installer.py
+++ b/py/toolkit/installer.py
@@ -24,7 +24,7 @@
import factory_common # pylint: disable=unused-import
from cros.factory.test.env import paths
-from cros.factory.test.test_lists import manager
+from cros.factory.test.test_lists import test_list_common
from cros.factory.tools import install_symlinks
from cros.factory.utils import file_utils
from cros.factory.utils import json_utils
@@ -201,8 +201,10 @@
"""Set the active test list for Goofy."""
if self._active_test_list is not None:
path = os.path.join(self._usr_local_dest, 'factory',
- manager.ACTIVE_TEST_LIST_CONFIG_RELPATH)
- json_utils.DumpFile(path, {'id': self._active_test_list})
+ test_list_common.ACTIVE_TEST_LIST_CONFIG_RELPATH)
+ json_utils.DumpFile(
+ path,
+ test_list_common.GenerateActiveTestListConfig(self._active_test_list))
def _EnableApp(self, app, enabled):
"""Enable / disable @app.