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

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 Mark, Devlin, Derek, and Daniel'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 "base/time/time.h"
12 #include "chrome/browser/media/router/media_route_status_observer.h"
13 #include "chrome/browser/media/router/mojo/media_router_mojo_test.h"
14 #include "testing/gmock/include/gmock/gmock.h"
15 #include "testing/gtest/include/gtest/gtest.h"
16
17 using ::testing::_;
18 using ::testing::Invoke;
19 using ::testing::StrictMock;
20
21 namespace media_router {
22
23 class MockMediaController : public mojom::MediaController {
24 public:
25 MOCK_METHOD0(Play, void());
26 MOCK_METHOD0(Pause, void());
27 MOCK_METHOD1(SetMute, void(bool mute));
28 MOCK_METHOD1(SetVolume, void(float volume));
29 MOCK_METHOD1(Seek, void(base::TimeDelta time));
30 };
31
32 class MockMediaRouteStatusObserver : public MediaRouteStatusObserver {
33 public:
34 MOCK_METHOD1(OnMediaStatusUpdated, void(const MediaStatus& status));
35 };
36
37 class MediaRouteControllerUnitTest : public ::testing::Test {
38 public:
39 MediaRouteControllerUnitTest() {}
40 ~MediaRouteControllerUnitTest() override {}
41
42 void SetUp() override {
43 mojom::MediaControllerPtr media_controller_ptr;
44 mock_media_controller_ = base::MakeUnique<MockMediaController>();
45 media_controller_binding_ =
46 base::MakeUnique<mojo::Binding<mojom::MediaController>>(
47 mock_media_controller_.get(),
48 mojo::MakeRequest(&media_controller_ptr));
49
50 controller_ = base::MakeUnique<MediaRouteController>(
51 std::move(media_controller_ptr),
52 base::Bind(&MediaRouteControllerUnitTest::DestroyController,
53 base::Unretained(this)));
54 }
55
56 void DestroyController() {
57 controller_.reset();
58 OnDestroyController();
59 }
60 MOCK_METHOD0(OnDestroyController, void());
61
62 protected:
63 std::unique_ptr<MockMediaController> mock_media_controller_;
64 std::unique_ptr<mojo::Binding<mojom::MediaController>>
65 media_controller_binding_;
66 std::unique_ptr<MediaRouteController> controller_;
67
68 content::TestBrowserThreadBundle test_thread_bundle_;
69
70 private:
71 DISALLOW_COPY_AND_ASSIGN(MediaRouteControllerUnitTest);
72 };
73
74 TEST_F(MediaRouteControllerUnitTest, ForwardControllerCommands) {
75 const float volume = 0.5;
76 const base::TimeDelta time = base::TimeDelta::FromSeconds(42);
77
78 EXPECT_CALL(*mock_media_controller_, Play());
79 controller_->Play();
80 EXPECT_CALL(*mock_media_controller_, Pause());
81 controller_->Pause();
82 EXPECT_CALL(*mock_media_controller_, SetMute(true));
83 controller_->SetMute(true);
84 EXPECT_CALL(*mock_media_controller_, SetVolume(volume));
85 controller_->SetVolume(volume);
86 EXPECT_CALL(*mock_media_controller_, Seek(time));
87 controller_->Seek(time);
88 }
89
90 TEST_F(MediaRouteControllerUnitTest, NotifyMediaRouteStatusObservers) {
91 StrictMock<MockMediaRouteStatusObserver> observer1;
92 StrictMock<MockMediaRouteStatusObserver> observer2;
93 StrictMock<MockMediaRouteStatusObserver> observer3;
94
95 MediaStatus status;
96 status.title = "test media status";
97
98 controller_->AddObserver(&observer1);
99 controller_->AddObserver(&observer2);
100
101 // Get a mojo pointer for |controller_|, so that we can notify it of status
102 // updates via mojo.
103 mojom::MediaStatusObserverPtr mojo_observer = controller_->GetObserverPtr();
104
105 base::RunLoop run_loop1;
106 EXPECT_CALL(observer1, OnMediaStatusUpdated(status));
107 EXPECT_CALL(observer2, OnMediaStatusUpdated(status));
108 mojo_observer->OnMediaStatusUpdated(status);
109 run_loop1.RunUntilIdle();
110
111 controller_->RemoveObserver(&observer1);
112 controller_->AddObserver(&observer3);
113
114 base::RunLoop run_loop2;
115 EXPECT_CALL(observer2, OnMediaStatusUpdated(status));
116 EXPECT_CALL(observer3, OnMediaStatusUpdated(status));
117 mojo_observer->OnMediaStatusUpdated(status);
118 run_loop2.RunUntilIdle();
119 }
120
121 TEST_F(MediaRouteControllerUnitTest, DestroyControllerOnNoObservers) {
122 MockMediaRouteStatusObserver observer1;
123 MockMediaRouteStatusObserver observer2;
124 controller_->AddObserver(&observer1);
125 controller_->AddObserver(&observer2);
126
127 EXPECT_CALL(*this, OnDestroyController()).Times(0);
128 controller_->RemoveObserver(&observer1);
129
130 // OnDestroyController() should be called when |controller_| no longer has any
131 // observers.
132 EXPECT_CALL(*this, OnDestroyController()).Times(1);
133 controller_->RemoveObserver(&observer2);
134 }
135
136 } // namespace media_router
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698