Converts resource_provider apptests to unit tests

components/resource_provider/BUILD.gn

 # Copyright 2015 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/mojo_application_manifest.gni")
+import("//mojo/public/tools/bindings/mojom.gni")
 import("//testing/test.gni")
 
 if (is_android) {
   import("//build/config/android/rules.gni")
 
   java_library_path = "$target_out_dir/java_library.dex.jar"
 
   mojo_android_application("resource_provider") {
     input_so =
         "$root_shlib_dir/${shlib_prefix}resource_provider_lib$shlib_extension"
@@ skipping 85 matching lines @@
     "//base",
     "//components/resource_provider/public/interfaces",
     "//mojo/common:common_base",
     "//mojo/platform_handle",
     "//mojo/services/tracing/public/cpp",
     "//mojo/shell/public/cpp",
     "//url",
   ]
 }
 
-test("resource_provider_unittests") {
+test("unittests") {
+  output_name = "resource_provider_unittests"

[Dirk Pranke, 2016/03/21 22:57:31]

GN style is to have the target_name match the binary name for test binaries,
rather than setting the output_name; you should really only use output_name when
you have to for some reason.

[sky, 2016/03/21 23:05:28]

Is that decision up for discussion? It seems to me typing
components/resource_provider:unittests makes a lot more sense than the redundant
components/resource_provider:resource_provider_unittests.

[sky, 2016/03/21 23:15:03]

I talked with Brett and he convinced me to go_with_the_long_name. Updated.

+
   sources = [
     "file_utils_unittest.cc",
+    "resource_provider_unittest.cc",
   ]
 
   deps = [
     ":lib",
+    ":test_interfaces",
     "//base",
-    "//base/test:test_config",
     "//components/resource_provider/public/interfaces",
-    "//mojo/edk/test:run_all_unittests",
     "//mojo/platform_handle:for_shared_library",
+    "//mojo/shell/background:main",
+    "//mojo/shell/public/cpp:shell_test_support",
+    "//mojo/shell/public/cpp/test:run_all_shelltests",
     "//testing/gtest",
     "//url",
   ]
+
+  data_deps = [
+    ":test_app",
+    ":unittests_manifest",
+  ]
 }
 
-mojo_native_application("apptests") {
-  output_name = "resource_provider_apptests"
+mojo_native_application("test_app") {
+  output_name = "resource_provider_test_app"
   testonly = true
 
   sources = [
-    "resource_provider_apptest.cc",
+    "resource_provider_test_app.cc",
   ]
 
   resources = [
     "//components/test/data/resource_provider/sample",
     "//components/test/data/resource_provider/dir",
   ]
 
   deps = [
+    ":test_interfaces",
     "//base",
-    "//base/test:test_config",
     "//components/resource_provider/public/cpp",
     "//components/resource_provider/public/interfaces",
-    "//mojo/common",
-    "//mojo/shell/public/cpp:test_support",
+    "//mojo/shell/public/cpp",
+    "//mojo/shell/public/interfaces",
   ]
 
   data_deps = [
-    ":apptest_manifest",
     ":resource_provider",
+    ":test_manifest",
   ]
 }
 
-mojo_application_manifest("apptest_manifest") {
-  application_name = "resource_provider_apptests"
-  source = "apptest_manifest.json"
+mojo_application_manifest("test_manifest") {
+  application_name = "resource_provider_test_app"
+  source = "test_manifest.json"
 }
+
+mojo_application_manifest("unittests_manifest") {
+  type = "exe"
+  application_name = "resource_provider_unittests"
+  source = "unittests_manifest.json"
+}
+
+mojom("test_interfaces") {
+  sources = [
+    "test.mojom",
+  ]
+}