MediaPerceptionPrivate API impl and testing.

chromeos/dbus/fake_upstart_client.h

 // 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.
 
 #ifndef CHROMEOS_DBUS_FAKE_UPSTART_CLIENT_H_
 #define CHROMEOS_DBUS_FAKE_UPSTART_CLIENT_H_
 
 #include "base/macros.h"
 
 #include "chromeos/dbus/upstart_client.h"
 
 namespace chromeos {
 
 class CHROMEOS_EXPORT FakeUpstartClient : public UpstartClient {
  public:
   FakeUpstartClient();
   ~FakeUpstartClient() override;
 
   // DBusClient overrides.
   void Init(dbus::Bus* bus) override;
 
   // UpstartClient overrides.
   void StartAuthPolicyService() override;
   void RestartAuthPolicyService() override;
   void StartMediaAnalytics(const UpstartCallback& callback) override;
   void StopMediaAnalytics() override;
 
+  void SetStartMediaAnalyticsWillSucceed(bool will_succeed);
+  void SetStartMediaAnalyticsWillStall(bool will_stall);
+
  private:
+  // Sets whether the StartMediaAnalytics method will succeed.
+  bool start_media_analytics_will_succeed_;

[tbarzic, 2017/05/18 19:28:01]

So, I'd do this differently (my main concern is that we don';t really want to
test start-up stalling - we want the request to be eventually handled) :)

I'd change StartMediaAnalytics() to enqueue "start requests",
and provide a hook for a test to verify the request was received, and to run
response:
(though, instead of adding this to FakeUpstartClient, I'd consider creating a
test FakeUpstartClient override specific to your tests - if you bake this here,
you'll probably want to guard this behavior behind a bool variable)

i.e.

std::queue<UpstartCallback> pending_upstart_requests_;

void FakeUpstartClient::StartMediaAnalyticsClient(callback) {
  peding_upstart_requets_.push(callback);
}

bool FakeUpstartClient::HandleNextUpstart(bool succeed) {
  if (pending_upstart_requests_.empty())
    return false;

  UpstartCallback callback = pending_upstart_requests_.front();
  pending_upstart_requests_.pop();

  if (!succeed) {
    callback.Run(succeed);
    return true;
  }

  //default StartMediaAnalytics(callback); 
}

[Luke Sorenson, 2017/05/18 21:24:16]

Done.

+
+  // Sets whether the StartMediaAnalytics method will stall.
+  bool start_media_analytics_will_stall_;
+
   DISALLOW_COPY_AND_ASSIGN(FakeUpstartClient);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_FAKE_UPSTART_CLIENT_H_