Converted Mojo Common Unittests to run as Apptests.

mojo/common/BUILD.gn

 # 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.
 
+import("//mojo/public/mojo_application.gni")
 import("//mojo/public/tools/bindings/mojom.gni")
 import("//testing/test.gni")
 
 source_set("common") {
   sources = [
     "binding_set.h",
     "interface_ptr_set.h",
     "strong_binding_set.h",
   ]
 
@@ skipping 18 matching lines @@
     ":test_interfaces",
     "//base",
     "//mojo/message_pump",
     "//mojo/public/cpp/bindings",
     "//mojo/public/cpp/bindings:callback",
     "//mojo/public/cpp/system",
     "//testing/gtest",
   ]
 }
 
-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",
   ]
 }
 
 mojom("test_interfaces") {
   testonly = true
   sources = [
     "test_interfaces.mojom",
   ]
 }
 
 source_set("tracing_impl") {
   sources = [
     "trace_provider_impl.cc",
     "trace_provider_impl.h",
     "tracing_impl.cc",
     "tracing_impl.h",
   ]
 
   deps = [
     "//base",
     "//mojo/public/cpp/application",
     "//mojo/public/cpp/bindings",
     "//mojo/public/interfaces/application",
     "//mojo/services/tracing/interfaces",
   ]
 }