Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(432)

Side by Side Diff: chrome/browser/media/router/mojo/media_route_controller_unittest.cc

Issue 2727123002: [Media Router] Custom Controls 1 - Add MediaStatus, MediaRouteController, and mojo interfaces (Closed)
Patch Set: Address Derek's comments Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/media/router/mojo/media_route_controller.h"
6
7 #include <string>
8 #include <utility>
9
10 #include "base/run_loop.h"
11 #include "chrome/browser/media/router/mojo/media_router_mojo_test.h"
12 #include "testing/gmock/include/gmock/gmock.h"
13 #include "testing/gtest/include/gtest/gtest.h"
14
15 using ::testing::StrictMock;
16
17 namespace media_router {
18
19 class MockMediaController : public mojom::MediaController {
20 public:
21 MOCK_METHOD0(Play, void());
22 MOCK_METHOD0(Pause, void());
23 MOCK_METHOD1(SetMute, void(bool mute));
24 MOCK_METHOD1(SetVolume, void(float volume));
25 MOCK_METHOD1(Seek, void(base::TimeDelta time));
26 };
27
28 class MockMediaRouteControllerObserver : public MediaRouteController::Observer {
29 public:
30 MockMediaRouteControllerObserver(
31 scoped_refptr<MediaRouteController> controller)
32 : MediaRouteController::Observer(controller) {}
33
34 MOCK_METHOD1(OnMediaStatusUpdated, void(const MediaStatus& status));
35 };
36
37 class MediaRouteControllerTest : public ::testing::Test {
38 public:
39 MediaRouteControllerTest() {}
40 ~MediaRouteControllerTest() override {}
41
42 void SetUp() override {
43 mojom::MediaControllerPtr media_controller_ptr;
44 mojom::MediaControllerRequest media_controller_request =
45 mojo::MakeRequest(&media_controller_ptr);
46 media_controller_binding_ =
47 base::MakeUnique<mojo::Binding<mojom::MediaController>>(
48 &mock_media_controller_, std::move(media_controller_request));
49
50 observer_ = base::MakeUnique<MockMediaRouteControllerObserver>(
51 new MediaRouteController("routeId", std::move(media_controller_ptr)));
52 }
53
54 scoped_refptr<MediaRouteController> GetController() const {
55 return observer_->controller();
56 }
57
58 protected:
59 // This must be called only after |observer_| is set.
60 std::unique_ptr<StrictMock<MockMediaRouteControllerObserver>> CreateObserver()
61 const {
62 return base::MakeUnique<StrictMock<MockMediaRouteControllerObserver>>(
63 GetController());
64 }
65
66 MockMediaController mock_media_controller_;
67 std::unique_ptr<mojo::Binding<mojom::MediaController>>
68 media_controller_binding_;
69 std::unique_ptr<MockMediaRouteControllerObserver> observer_;
70
71 content::TestBrowserThreadBundle test_thread_bundle_;
72
73 private:
74 DISALLOW_COPY_AND_ASSIGN(MediaRouteControllerTest);
75 };
76
77 TEST_F(MediaRouteControllerTest, ForwardControllerCommands) {
78 const float volume = 0.5;
79 const base::TimeDelta time = base::TimeDelta::FromSeconds(42);
80
81 EXPECT_CALL(mock_media_controller_, Play());
82 GetController()->Play();
83 EXPECT_CALL(mock_media_controller_, Pause());
84 GetController()->Pause();
85 EXPECT_CALL(mock_media_controller_, SetMute(true));
86 GetController()->SetMute(true);
87 EXPECT_CALL(mock_media_controller_, SetVolume(volume));
88 GetController()->SetVolume(volume);
89 EXPECT_CALL(mock_media_controller_, Seek(time));
90 GetController()->Seek(time);
91 }
92
93 TEST_F(MediaRouteControllerTest, NotifyMediaRouteControllerObservers) {
94 auto observer1 = CreateObserver();
95 auto observer2 = CreateObserver();
mark a. foltz 2017/03/22 00:41:44 Much nicer :-)
96
97 MediaStatus status;
98 status.title = "test media status";
99
100 EXPECT_CALL(*observer1, OnMediaStatusUpdated(status));
101 EXPECT_CALL(*observer2, OnMediaStatusUpdated(status));
102 // TODO(takumif): Use a mojom::MediaStatusObserverPtr bound to the controller.
103 GetController()->OnMediaStatusUpdated(status);
104
105 observer1.reset();
106 auto observer3 = CreateObserver();
107
108 EXPECT_CALL(*observer2, OnMediaStatusUpdated(status));
109 EXPECT_CALL(*observer3, OnMediaStatusUpdated(status));
110 // TODO(takumif): Use a mojom::MediaStatusObserverPtr bound to the controller.
111 GetController()->OnMediaStatusUpdated(status);
112 }
113
114 } // namespace media_router
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698