Enable PipelineIntegrationTests in mojo!

media/mojo/services/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")
 
 # Target naming conventions:
 # - converters: C++/Mojo type converters.
 # - proxy: C++ implementations supported by mojo services.
 # - service: Mojo interface implementations.
@@ skipping 91 matching lines @@
       "mojo_media_application.cc",
     ]
 
     deps = [
       ":renderer_service",
       "//mojo/application",
       "//mojo/public/c/system:for_shared_library",
     ]
   }
 
-  # Not a 'test' because this is loaded via mojo_shell as an app.
-  # To run the test:
-  #   out/Debug/mojo_shell mojo:media_test
-  # You can get "mojo_shell" using mojo/public/tools/download_shell_binary.py
+  # Note, this 'test' must be loaded via mojo_shell as an app:
+  #
+  #   out/Debug/mojo_shell mojo://media_test

[xhwang, 2014/12/17 05:25:03]

OOC, is mojo:media_test not working, or is mojo://* recommended?

[DaleCurtis, 2014/12/17 21:40:49]

It works. I thought a thread encouraged the :// behavior, but it seems that's
less clear now:

https://groups.google.com/a/chromium.org/forum/#!searchin/mojo-dev/PSA/mojo-d...

[xhwang, 2014/12/17 21:53:38]

jamesr: Do you know what's the recommended style now?

+  #
+  # To get "mojo_shell" you must specify "use_prebuilt_mojo_shell=true" in GN.
   mojo_native_application("media_test") {
     testonly = true
-    sources = [
-      "media_renderer_apptest.cc",
-    ]
 
     deps = [
-      ":media",
-      ":renderer_proxy",
-      "//base",
-      "//media",
-      "//media/mojo/interfaces",
-      "//media:shared_memory_support",
-      "//mojo/application",
-      "//mojo/application:test_support",
-      "//mojo/common",
-      "//mojo/environment:chromium",
-      "//mojo/public/c/system:for_shared_library",
+      "//media/test:mojo_pipeline_integration_tests",
     ]
   }
 }
 
 group("services") {
   deps = [
     ":renderer_proxy",
   ]
 
   if (!is_component_build) {
     deps += [ ":media" ]
   }
 }
 
 group("tests") {
   testonly = true
   deps = [
     ":media_mojo_unittests",
   ]
 
   if (!is_component_build) {
     deps += [ ":media_test" ]
   }
 }