Adds the ability to run pure Go unit tests in the Mojo test suite.

mojo/tools/get_test_list.py

 #!/usr/bin/env python
 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Central list of tests to run (as appropriate for a given config). Add tests
 to run by modifying this file.
 
 Note that this file is both imported (by mojob.py) and run directly (via a
 recipe)."""
@@ skipping 99 matching lines @@
       # TODO(smklein): Add for Android once tests are pexes.
       AddXvfbEntry("NaCl app tests",
                    [os.path.join("mojo", "tools", "apptest_runner.py"),
                     os.path.join("mojo", "tools", "data", "nacl_apptests"),
                     build_dir] + verbose_flags)
 
   # Go unit tests (Linux-only):
   if (target_os == Config.OS_LINUX and
       config.sanitizer != Config.SANITIZER_ASAN and
       ShouldRunTest(Config.TEST_TYPE_DEFAULT, Config.TEST_TYPE_UNIT, "go")):
+    # Go system tests:
+    AddEntry("Go system tests",
+             [os.path.join(build_dir, "obj", "mojo", "go", "system_test")])
+
+    # Pure Go unit tests:
+    if "go_build_tool" in config.values:
+      go_tool = config.values["go_build_tool"]
+    else:
+      # Note(rudominer) This branch is necessary for when this script is
+      # run on the bots. In that case the config.values dictionary is built in
+      # the bot's recipe script and does not contain a value for
+      # "go_build_tool." Consider adding that value there instead.
+      go_tool = os.path.join(Paths().src_root, "third_party", "go", "tool",

[viettrungluu, 2015/10/06 21:06:28]

In particular, if the go_build_tool directory should be derived from other
configuration parameters, then something should be added to Paths to figure that
out.

Moreover, using Paths() here is wrong. You should be using |paths|, which
properly takes into account the config.

[rudominer, 2015/10/09 18:30:30]

Done.

+                             "linux_amd64", "bin", "go")
     AddEntry("Go unit tests",
-             [os.path.join(build_dir, "obj", "mojo", "go", "system_test")])
+             ["python", os.path.join("mojo", "tools", "run_pure_go_tests.py"),
+              go_tool, os.path.join("mojo", "tools", "data", "gotests")])
 
   # Python unit tests:
   if ShouldRunTest(Config.TEST_TYPE_DEFAULT, Config.TEST_TYPE_UNIT, "python"):
     AddEntry("Python unit tests",
              ["python", os.path.join("mojo", "tools",
               "run_mojo_python_tests.py")])
 
   # Python bindings tests (Linux-only):
   # See http://crbug.com/438781 for details on asan exclusion.
   if (target_os == Config.OS_LINUX and
@@ skipping 109 matching lines @@
   config = Config(**json.load(args.config_file))
   test_list = GetTestList(config)
   json.dump(test_list, args.test_list_file, indent=2)
   args.test_list_file.write("\n")
 
   return 0
 
 
 if __name__ == "__main__":
   sys.exit(main())