[DeviceService] Add service tests for VibrationManager.

services/device/device_service_test_base.cc

Index: services/device/device_service_test_base.cc
diff --git a/services/device/device_service_test_base.cc b/services/device/device_service_test_base.cc
new file mode 100644
index 0000000000000000000000000000000000000000..6c9c0e5d865b7bb23bb0f03eb354d6d1661a7487
--- /dev/null
+++ b/services/device/device_service_test_base.cc
@@ -0,0 +1,91 @@
+// Copyright 2017 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 "services/device/device_service_test_base.h"
+
+#include <memory>
+
+#include "base/memory/ptr_util.h"
+#include "base/threading/thread.h"
+#include "mojo/public/cpp/bindings/binding_set.h"
+#include "services/device/device_service.h"
+#include "services/device/public/interfaces/constants.mojom.h"
+#include "services/service_manager/public/cpp/interface_factory.h"
+#include "services/service_manager/public/cpp/interface_registry.h"
+#include "services/service_manager/public/cpp/service_context.h"
+#include "services/service_manager/public/interfaces/service_factory.mojom.h"
+
+namespace device {
+
+namespace {
+
+const char kTestServiceName[] = "device_unittests";
+
+// The test service responsible to package Device Service.
+class ServiceTestClient : public service_manager::test::ServiceTestClient,
+                          public service_manager::mojom::ServiceFactory,
+                          public service_manager::InterfaceFactory<
+                              service_manager::mojom::ServiceFactory> {
+ public:
+  explicit ServiceTestClient(service_manager::test::ServiceTest* test)
+      : service_manager::test::ServiceTestClient(test),
+        io_thread_("DeviceServiceTestIOThread"),
+        file_thread_("DeviceServiceTestFileThread") {}
+  ~ServiceTestClient() override {}
+
+ protected:
+  bool OnConnect(const service_manager::ServiceInfo& remote_info,
+                 service_manager::InterfaceRegistry* registry) override {
+    registry->AddInterface<service_manager::mojom::ServiceFactory>(this);
+    return true;
+  }
+
+  void CreateService(service_manager::mojom::ServiceRequest request,
+                     const std::string& name) override {
+    if (name == device::mojom::kServiceName) {
+      io_thread_.Start();
+      file_thread_.Start();
+#if defined(OS_ANDROID)
+      device_service_context_.reset(new service_manager::ServiceContext(
+          CreateDeviceService(file_thread_.task_runner(),
+                              io_thread_.task_runner(),
+                              wake_lock_context_callback_),
+          std::move(request)));
+#else
+      device_service_context_.reset(new service_manager::ServiceContext(
+          CreateDeviceService(file_thread_.task_runner(),
+                              io_thread_.task_runner()),
+          std::move(request)));
+#endif
+    }
+  }
+
+  void Create(const service_manager::Identity& remote_identity,
+              service_manager::mojom::ServiceFactoryRequest request) override {
+    service_factory_bindings_.AddBinding(this, std::move(request));
+  }
+
+ private:
+  mojo::BindingSet<service_manager::mojom::ServiceFactory>
+      service_factory_bindings_;
+  std::unique_ptr<service_manager::ServiceContext> device_service_context_;
+
+  base::Thread io_thread_;

[blundell, 2017/03/31 11:22:30]

nit: These should be moved above line 72 so that they outlive
|device_service_context_|.

[leonhsl(Using Gerrit), 2017/03/31 12:14:45]

Acknowledged.

[leonhsl(Using Gerrit), 2017/04/11 08:32:02]

Done.

+  base::Thread file_thread_;
+  WakeLockContextCallback wake_lock_context_callback_;
+};
+
+}  // namespace
+
+DeviceServiceTestBase::DeviceServiceTestBase()
+    : ServiceTest(kTestServiceName) {}
+
+DeviceServiceTestBase::~DeviceServiceTestBase() {}
+
+std::unique_ptr<service_manager::Service>
+DeviceServiceTestBase::CreateService() {
+  return base::MakeUnique<ServiceTestClient>(this);
+}
+
+}  // namespace device