Converted Mojo Common Unittests to run as Apptests.

mojo/common/BUILD.gn

Index: mojo/common/BUILD.gn
diff --git a/mojo/common/BUILD.gn b/mojo/common/BUILD.gn
index 3410af218085d87e93e29fe18a3393b6b9bd7434..527c4a133467e1f883bc1a409de5fe0f9b5fe79b 100644
--- a/mojo/common/BUILD.gn
+++ b/mojo/common/BUILD.gn
@@ -2,6 +2,7 @@
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
+import("//mojo/public/mojo_application.gni")
 import("//mojo/public/tools/bindings/mojom.gni")
 import("//testing/test.gni")
 
@@ -40,13 +41,13 @@ source_set("tests") {
   ]
 }
 
-test("mojo_common_unittests") {
+mojo_native_application("mojo_common_apptests") {
+  testonly = true
+
   deps = [
     ":tests",
-    "//mojo/data_pipe_utils:tests",

[viettrungluu, 2016/05/19 22:18:49]

This target also contains tests. Arguably it should get its own apptest binary
(mojo_data_pipe_utils_apptests, I guess).

[Sean Klein, 2016/05/19 22:52:27]

Ahhh yikes. Sorry about that. Apptest added for data pipe utils.

-    "//mojo/edk/test:run_all_unittests",
-    "//mojo/environment:chromium",
-    "//mojo/message_pump:tests",

[viettrungluu, 2016/05/19 22:18:49]

"

[Sean Klein, 2016/05/19 22:52:28]

Apptest added for message pump too. I promise I'm not just trying to delete
tests :P

+    "//mojo/application",
+    "//mojo/application:test_support",
   ]
 }