OLD | NEW |
---|---|
(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 MediaRouteController* GetController() { | |
55 return observer_ ? observer_->controller() : nullptr; | |
mark a. foltz
2017/03/20 23:47:03
Still not fixed - observer_ is always set in SetUp
takumif
2017/03/21 04:00:07
I had this because I was resetting observer_ in on
| |
56 } | |
57 | |
58 protected: | |
59 MockMediaController mock_media_controller_; | |
60 std::unique_ptr<mojo::Binding<mojom::MediaController>> | |
61 media_controller_binding_; | |
62 std::unique_ptr<MockMediaRouteControllerObserver> observer_; | |
63 | |
64 content::TestBrowserThreadBundle test_thread_bundle_; | |
65 | |
66 private: | |
67 DISALLOW_COPY_AND_ASSIGN(MediaRouteControllerTest); | |
68 }; | |
69 | |
70 TEST_F(MediaRouteControllerTest, ForwardControllerCommands) { | |
71 const float volume = 0.5; | |
72 const base::TimeDelta time = base::TimeDelta::FromSeconds(42); | |
73 | |
74 EXPECT_CALL(mock_media_controller_, Play()); | |
75 GetController()->Play(); | |
76 EXPECT_CALL(mock_media_controller_, Pause()); | |
77 GetController()->Pause(); | |
78 EXPECT_CALL(mock_media_controller_, SetMute(true)); | |
79 GetController()->SetMute(true); | |
80 EXPECT_CALL(mock_media_controller_, SetVolume(volume)); | |
81 GetController()->SetVolume(volume); | |
82 EXPECT_CALL(mock_media_controller_, Seek(time)); | |
83 GetController()->Seek(time); | |
84 } | |
85 | |
86 TEST_F(MediaRouteControllerTest, NotifyMediaRouteControllerObservers) { | |
87 std::unique_ptr<StrictMock<MockMediaRouteControllerObserver>> observer1 = | |
88 base::MakeUnique<StrictMock<MockMediaRouteControllerObserver>>( | |
89 GetController()); | |
90 std::unique_ptr<StrictMock<MockMediaRouteControllerObserver>> observer2 = | |
91 base::MakeUnique<StrictMock<MockMediaRouteControllerObserver>>( | |
92 GetController()); | |
93 | |
94 MediaStatus status; | |
95 status.title = "test media status"; | |
96 | |
97 EXPECT_CALL(*observer1, OnMediaStatusUpdated(status)); | |
98 EXPECT_CALL(*observer2, OnMediaStatusUpdated(status)); | |
99 // TODO(takumif): Use a mojom::MediaStatusObserverPtr bound to the controller. | |
mark a. foltz
2017/03/20 23:47:03
Still don't quite follow this, but as long as it m
takumif
2017/03/21 04:00:07
Here, instead of calling OnMediaStatusUpdated() on
| |
100 GetController()->OnMediaStatusUpdated(status); | |
101 | |
102 observer1.reset(); | |
103 std::unique_ptr<StrictMock<MockMediaRouteControllerObserver>> observer3 = | |
104 base::MakeUnique<StrictMock<MockMediaRouteControllerObserver>>( | |
105 GetController()); | |
106 | |
107 EXPECT_CALL(*observer2, OnMediaStatusUpdated(status)); | |
108 EXPECT_CALL(*observer3, OnMediaStatusUpdated(status)); | |
109 // TODO(takumif): Use a mojom::MediaStatusObserverPtr bound to the controller. | |
110 GetController()->OnMediaStatusUpdated(status); | |
111 } | |
112 | |
takumif
2017/03/20 22:10:46
I'll be adding the two tests that I removed here i
| |
113 } // namespace media_router | |
OLD | NEW |