blob: ac1c862f5f37552e3290853ccca4c60a52cbab08 [file] [log] [blame]
Taylor Brandstettera1c30352016-05-13 08:15:11 -07001/*
2 * Copyright 2016 The WebRTC Project Authors. All rights reserved.
3 *
4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree.
9 */
10
11#include <memory>
12
13#include "webrtc/base/gunit.h"
14#include "webrtc/base/thread.h"
15#include "webrtc/p2p/base/fakeportallocator.h"
16#include "webrtc/p2p/base/portallocator.h"
17
18static const char kContentName[] = "test content";
19// Based on ICE_UFRAG_LENGTH
zhihuang6d0d4bf2016-05-24 10:13:32 -070020static const char kIceUfrag[] = "UF00";
Taylor Brandstettera1c30352016-05-13 08:15:11 -070021// Based on ICE_PWD_LENGTH
22static const char kIcePwd[] = "TESTICEPWD00000000000000";
23static const char kTurnUsername[] = "test";
24static const char kTurnPassword[] = "test";
25
26class PortAllocatorTest : public testing::Test, public sigslot::has_slots<> {
27 public:
28 PortAllocatorTest() {
29 allocator_.reset(
30 new cricket::FakePortAllocator(rtc::Thread::Current(), nullptr));
31 }
32
33 protected:
34 void SetConfigurationWithPoolSize(int candidate_pool_size) {
deadbeef6de92f92016-12-12 18:49:32 -080035 EXPECT_TRUE(allocator_->SetConfiguration(
36 cricket::ServerAddresses(), std::vector<cricket::RelayServerConfig>(),
37 candidate_pool_size, false));
38 }
39
40 void SetConfigurationWithPoolSizeExpectFailure(int candidate_pool_size) {
41 EXPECT_FALSE(allocator_->SetConfiguration(
42 cricket::ServerAddresses(), std::vector<cricket::RelayServerConfig>(),
43 candidate_pool_size, false));
Taylor Brandstettera1c30352016-05-13 08:15:11 -070044 }
45
Taylor Brandstetter417eebe2016-05-23 16:02:19 -070046 std::unique_ptr<cricket::FakePortAllocatorSession> CreateSession(
Taylor Brandstetter417eebe2016-05-23 16:02:19 -070047 const std::string& content_name,
48 int component,
49 const std::string& ice_ufrag,
50 const std::string& ice_pwd) {
51 return std::unique_ptr<cricket::FakePortAllocatorSession>(
52 static_cast<cricket::FakePortAllocatorSession*>(
53 allocator_
johanfe1ffb12016-08-11 12:37:42 -070054 ->CreateSession(content_name, component, ice_ufrag, ice_pwd)
Taylor Brandstetter417eebe2016-05-23 16:02:19 -070055 .release()));
56 }
57
Taylor Brandstettera1c30352016-05-13 08:15:11 -070058 const cricket::FakePortAllocatorSession* GetPooledSession() const {
59 return static_cast<const cricket::FakePortAllocatorSession*>(
60 allocator_->GetPooledSession());
61 }
62
63 std::unique_ptr<cricket::FakePortAllocatorSession> TakePooledSession() {
64 return std::unique_ptr<cricket::FakePortAllocatorSession>(
65 static_cast<cricket::FakePortAllocatorSession*>(
66 allocator_->TakePooledSession(kContentName, 0, kIceUfrag, kIcePwd)
67 .release()));
68 }
69
70 int GetAllPooledSessionsReturnCount() {
71 int count = 0;
72 while (GetPooledSession()) {
73 TakePooledSession();
74 ++count;
75 }
76 return count;
77 }
78
79 std::unique_ptr<cricket::FakePortAllocator> allocator_;
80 rtc::SocketAddress stun_server_1{"11.11.11.11", 3478};
81 rtc::SocketAddress stun_server_2{"22.22.22.22", 3478};
82 cricket::RelayServerConfig turn_server_1{"11.11.11.11", 3478,
83 kTurnUsername, kTurnPassword,
84 cricket::PROTO_UDP, false};
85 cricket::RelayServerConfig turn_server_2{"22.22.22.22", 3478,
86 kTurnUsername, kTurnPassword,
87 cricket::PROTO_UDP, false};
88};
89
90TEST_F(PortAllocatorTest, TestDefaults) {
91 EXPECT_EQ(0UL, allocator_->stun_servers().size());
92 EXPECT_EQ(0UL, allocator_->turn_servers().size());
93 EXPECT_EQ(0, allocator_->candidate_pool_size());
94 EXPECT_EQ(0, GetAllPooledSessionsReturnCount());
95}
96
Taylor Brandstetter417eebe2016-05-23 16:02:19 -070097// Call CreateSession and verify that the parameters passed in and the
98// candidate filter are applied as expected.
99TEST_F(PortAllocatorTest, CreateSession) {
100 allocator_->set_candidate_filter(cricket::CF_RELAY);
johanfe1ffb12016-08-11 12:37:42 -0700101 auto session = CreateSession(kContentName, 1, kIceUfrag, kIcePwd);
Taylor Brandstetter417eebe2016-05-23 16:02:19 -0700102 ASSERT_NE(nullptr, session);
103 EXPECT_EQ(cricket::CF_RELAY, session->candidate_filter());
104 EXPECT_EQ(kContentName, session->content_name());
105 EXPECT_EQ(1, session->component());
106 EXPECT_EQ(kIceUfrag, session->ice_ufrag());
107 EXPECT_EQ(kIcePwd, session->ice_pwd());
108}
109
Taylor Brandstettera1c30352016-05-13 08:15:11 -0700110TEST_F(PortAllocatorTest, SetConfigurationUpdatesIceServers) {
111 cricket::ServerAddresses stun_servers_1 = {stun_server_1};
112 std::vector<cricket::RelayServerConfig> turn_servers_1 = {turn_server_1};
deadbeef6de92f92016-12-12 18:49:32 -0800113 EXPECT_TRUE(
114 allocator_->SetConfiguration(stun_servers_1, turn_servers_1, 0, false));
Taylor Brandstettera1c30352016-05-13 08:15:11 -0700115 EXPECT_EQ(stun_servers_1, allocator_->stun_servers());
116 EXPECT_EQ(turn_servers_1, allocator_->turn_servers());
117
118 // Update with a different set of servers.
119 cricket::ServerAddresses stun_servers_2 = {stun_server_2};
120 std::vector<cricket::RelayServerConfig> turn_servers_2 = {turn_server_2};
deadbeef6de92f92016-12-12 18:49:32 -0800121 EXPECT_TRUE(
122 allocator_->SetConfiguration(stun_servers_2, turn_servers_2, 0, false));
Taylor Brandstettera1c30352016-05-13 08:15:11 -0700123 EXPECT_EQ(stun_servers_2, allocator_->stun_servers());
124 EXPECT_EQ(turn_servers_2, allocator_->turn_servers());
125}
126
127TEST_F(PortAllocatorTest, SetConfigurationUpdatesCandidatePoolSize) {
128 SetConfigurationWithPoolSize(2);
129 EXPECT_EQ(2, allocator_->candidate_pool_size());
130 SetConfigurationWithPoolSize(3);
131 EXPECT_EQ(3, allocator_->candidate_pool_size());
132 SetConfigurationWithPoolSize(1);
133 EXPECT_EQ(1, allocator_->candidate_pool_size());
134 SetConfigurationWithPoolSize(4);
135 EXPECT_EQ(4, allocator_->candidate_pool_size());
136}
137
138// A negative pool size should just be treated as zero.
deadbeef6de92f92016-12-12 18:49:32 -0800139TEST_F(PortAllocatorTest, SetConfigurationWithNegativePoolSizeFails) {
140 SetConfigurationWithPoolSizeExpectFailure(-1);
Taylor Brandstettera1c30352016-05-13 08:15:11 -0700141}
142
143// Test that if the candidate pool size is nonzero, pooled sessions are
144// created, and StartGettingPorts is called on them.
145TEST_F(PortAllocatorTest, SetConfigurationCreatesPooledSessions) {
146 SetConfigurationWithPoolSize(2);
147 auto session_1 = TakePooledSession();
148 auto session_2 = TakePooledSession();
149 ASSERT_NE(nullptr, session_1.get());
150 ASSERT_NE(nullptr, session_2.get());
151 EXPECT_EQ(1, session_1->port_config_count());
152 EXPECT_EQ(1, session_2->port_config_count());
153 EXPECT_EQ(0, GetAllPooledSessionsReturnCount());
154}
155
156// Test that if the candidate pool size is increased, pooled sessions are
157// created as necessary.
158TEST_F(PortAllocatorTest, SetConfigurationCreatesMorePooledSessions) {
159 SetConfigurationWithPoolSize(1);
160 SetConfigurationWithPoolSize(2);
161 EXPECT_EQ(2, GetAllPooledSessionsReturnCount());
162}
163
164// Test that if the candidate pool size is reduced, extra sessions are
165// destroyed.
166TEST_F(PortAllocatorTest, SetConfigurationDestroysPooledSessions) {
167 SetConfigurationWithPoolSize(2);
168 SetConfigurationWithPoolSize(1);
169 EXPECT_EQ(1, GetAllPooledSessionsReturnCount());
170}
171
deadbeef6de92f92016-12-12 18:49:32 -0800172// According to JSEP, existing pooled sessions should be destroyed and new
Taylor Brandstettera1c30352016-05-13 08:15:11 -0700173// ones created when the ICE servers change.
174TEST_F(PortAllocatorTest,
175 SetConfigurationRecreatesPooledSessionsWhenIceServersChange) {
176 cricket::ServerAddresses stun_servers_1 = {stun_server_1};
177 std::vector<cricket::RelayServerConfig> turn_servers_1 = {turn_server_1};
Honghai Zhangb9e7b4a2016-06-30 20:52:02 -0700178 allocator_->SetConfiguration(stun_servers_1, turn_servers_1, 1, false);
Taylor Brandstettera1c30352016-05-13 08:15:11 -0700179 EXPECT_EQ(stun_servers_1, allocator_->stun_servers());
180 EXPECT_EQ(turn_servers_1, allocator_->turn_servers());
181
182 // Update with a different set of servers (and also change pool size).
183 cricket::ServerAddresses stun_servers_2 = {stun_server_2};
184 std::vector<cricket::RelayServerConfig> turn_servers_2 = {turn_server_2};
Honghai Zhangb9e7b4a2016-06-30 20:52:02 -0700185 allocator_->SetConfiguration(stun_servers_2, turn_servers_2, 2, false);
Taylor Brandstettera1c30352016-05-13 08:15:11 -0700186 EXPECT_EQ(stun_servers_2, allocator_->stun_servers());
187 EXPECT_EQ(turn_servers_2, allocator_->turn_servers());
188 auto session_1 = TakePooledSession();
189 auto session_2 = TakePooledSession();
190 ASSERT_NE(nullptr, session_1.get());
191 ASSERT_NE(nullptr, session_2.get());
192 EXPECT_EQ(stun_servers_2, session_1->stun_servers());
193 EXPECT_EQ(turn_servers_2, session_1->turn_servers());
194 EXPECT_EQ(stun_servers_2, session_2->stun_servers());
195 EXPECT_EQ(turn_servers_2, session_2->turn_servers());
196 EXPECT_EQ(0, GetAllPooledSessionsReturnCount());
197}
198
deadbeef42a42632017-03-10 15:18:00 -0800199// According to JSEP, after SetLocalDescription, setting different ICE servers
200// will not cause the pool to be refilled. This is implemented by the
201// PeerConnection calling FreezeCandidatePool when a local description is set.
202TEST_F(PortAllocatorTest,
203 SetConfigurationDoesNotRecreatePooledSessionsAfterFreezeCandidatePool) {
204 cricket::ServerAddresses stun_servers_1 = {stun_server_1};
205 std::vector<cricket::RelayServerConfig> turn_servers_1 = {turn_server_1};
206 allocator_->SetConfiguration(stun_servers_1, turn_servers_1, 1, false);
207 EXPECT_EQ(stun_servers_1, allocator_->stun_servers());
208 EXPECT_EQ(turn_servers_1, allocator_->turn_servers());
209
210 // Update with a different set of servers, but first freeze the pool.
211 allocator_->FreezeCandidatePool();
212 cricket::ServerAddresses stun_servers_2 = {stun_server_2};
213 std::vector<cricket::RelayServerConfig> turn_servers_2 = {turn_server_2};
214 allocator_->SetConfiguration(stun_servers_2, turn_servers_2, 2, false);
215 EXPECT_EQ(stun_servers_2, allocator_->stun_servers());
216 EXPECT_EQ(turn_servers_2, allocator_->turn_servers());
217 auto session = TakePooledSession();
218 ASSERT_NE(nullptr, session.get());
219 EXPECT_EQ(stun_servers_1, session->stun_servers());
220 EXPECT_EQ(turn_servers_1, session->turn_servers());
221 EXPECT_EQ(0, GetAllPooledSessionsReturnCount());
222}
223
Taylor Brandstettera1c30352016-05-13 08:15:11 -0700224TEST_F(PortAllocatorTest, GetPooledSessionReturnsNextSession) {
225 SetConfigurationWithPoolSize(2);
226 auto peeked_session_1 = GetPooledSession();
227 auto session_1 = TakePooledSession();
228 EXPECT_EQ(session_1.get(), peeked_session_1);
229 auto peeked_session_2 = GetPooledSession();
230 auto session_2 = TakePooledSession();
231 EXPECT_EQ(session_2.get(), peeked_session_2);
232}
233
234// Verify that subclasses of PortAllocatorSession are given a chance to update
235// ICE parameters when TakePooledSession is called, and the base class updates
236// the info itself.
237TEST_F(PortAllocatorTest, TakePooledSessionUpdatesIceParameters) {
238 SetConfigurationWithPoolSize(1);
239 auto peeked_session = GetPooledSession();
240 ASSERT_NE(nullptr, peeked_session);
241 EXPECT_EQ(0, peeked_session->transport_info_update_count());
242 std::unique_ptr<cricket::FakePortAllocatorSession> session(
243 static_cast<cricket::FakePortAllocatorSession*>(
244 allocator_->TakePooledSession(kContentName, 1, kIceUfrag, kIcePwd)
245 .release()));
246 EXPECT_EQ(1, session->transport_info_update_count());
247 EXPECT_EQ(kContentName, session->content_name());
248 EXPECT_EQ(1, session->component());
249 EXPECT_EQ(kIceUfrag, session->ice_ufrag());
250 EXPECT_EQ(kIcePwd, session->ice_pwd());
251}
Taylor Brandstetter417eebe2016-05-23 16:02:19 -0700252
253// According to JSEP, candidate filtering should be done when the pooled
254// candidates are surfaced to the application. This means when a pooled
255// session is taken. So a pooled session should gather candidates
256// unfiltered until it's returned by TakePooledSession.
257TEST_F(PortAllocatorTest, TakePooledSessionUpdatesCandidateFilter) {
258 allocator_->set_candidate_filter(cricket::CF_RELAY);
259 SetConfigurationWithPoolSize(1);
260 auto peeked_session = GetPooledSession();
261 ASSERT_NE(nullptr, peeked_session);
262 EXPECT_EQ(cricket::CF_ALL, peeked_session->candidate_filter());
263 auto session = TakePooledSession();
264 EXPECT_EQ(cricket::CF_RELAY, session->candidate_filter());
265}
deadbeef42a42632017-03-10 15:18:00 -0800266
267// Verify that after DiscardCandidatePool, TakePooledSession doesn't return
268// anything.
269TEST_F(PortAllocatorTest, DiscardCandidatePool) {
270 SetConfigurationWithPoolSize(1);
271 allocator_->DiscardCandidatePool();
272 EXPECT_EQ(0, GetAllPooledSessionsReturnCount());
273}