Add unit test for mus_demo.

services/ui/demo/mus_demo_unittests.cc

 // Copyright 2016 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.
 
 #include "base/bind.h"
 #include "base/command_line.h"
+#include "base/macros.h"
 #include "base/run_loop.h"
 #include "services/shell/public/cpp/service_test.h"
 #include "services/ui/public/interfaces/window_server_test.mojom.h"
+#include "testing/gtest/include/gtest/gtest.h"
 
-namespace ash {
-namespace mus {
+namespace ui {
+namespace demo {
+
+namespace {
+
+const char kTestAppName[] = "exe:mus_demo_unittests";
 
 void RunCallback(bool* success, const base::Closure& callback, bool result) {
   *success = result;
   callback.Run();
 }
 
-class AppLaunchTest : public shell::test::ServiceTest {
+class MusDemoTest : public shell::test::ServiceTest {

[rjkroege, 2016/07/08 21:38:32]

Given that we renamed mus to services/ui -- maybe this needs a different name.
Challenge what should this different name be? I started a thread.

[kylechar, 2016/07/13 13:55:32]

I see the thread. Hmmm, UIServiceDemo doesn't have quite the same ring to it.

  public:
-  AppLaunchTest() : ServiceTest("exe:mash_unittests") {}
-  ~AppLaunchTest() override {}
+  MusDemoTest() : shell::test::ServiceTest(kTestAppName) {}
+  ~MusDemoTest() override {}
 
- private:
   void SetUp() override {
     base::CommandLine::ForCurrentProcess()->AppendSwitch("use-test-config");
     ServiceTest::SetUp();
   }
 
-  DISALLOW_COPY_AND_ASSIGN(AppLaunchTest);
+ private:
+  DISALLOW_COPY_AND_ASSIGN(MusDemoTest);
 };
 
-TEST_F(AppLaunchTest, TestQuickLaunch) {
-  connector()->Connect("mojo:ash");
-  connector()->Connect("mojo:quick_launch");
+}  // namespace
+
+TEST_F(MusDemoTest, CheckMusDemoDraws) {
+  connector()->Connect("mojo:mus_demo");
 
   ::ui::mojom::WindowServerTestPtr test_interface;
   connector()->ConnectToInterface("mojo:ui", &test_interface);
 
   base::RunLoop run_loop;
   bool success = false;
+  // TODO(kylechar): Fix WindowServer::CreateTreeForWindowManager so that the
+  // WindowTree has the correct name instead of an empty name.
   test_interface->EnsureClientHasDrawnWindow(
-      "mojo:quick_launch",
+      "",  // WindowTree name is empty.
       base::Bind(&RunCallback, &success, run_loop.QuitClosure()));
   run_loop.Run();
   EXPECT_TRUE(success);
 }
 
-}  // namespace mus
-}  // namespace ash
+}  // namespace demo
+}  // namespace ui