blob: e7ac8c7f6161ef6b23753305cad48c5c46a74eeb [file] [log] [blame]
Henrik Boström933d8b02017-10-10 10:05:16 -07001/*
2 * Copyright 2017 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#include <vector>
13
Karl Wiberg1b0eae32017-10-17 14:48:54 +020014#include "api/audio_codecs/builtin_audio_decoder_factory.h"
15#include "api/audio_codecs/builtin_audio_encoder_factory.h"
Henrik Boström933d8b02017-10-10 10:05:16 -070016#include "api/jsep.h"
17#include "api/mediastreaminterface.h"
18#include "api/peerconnectioninterface.h"
19#include "pc/mediastream.h"
20#include "pc/mediastreamtrack.h"
21#include "pc/peerconnectionwrapper.h"
22#include "pc/test/fakeaudiocapturemodule.h"
23#include "pc/test/mockpeerconnectionobservers.h"
24#include "rtc_base/checks.h"
25#include "rtc_base/gunit.h"
26#include "rtc_base/ptr_util.h"
27#include "rtc_base/refcountedobject.h"
28#include "rtc_base/scoped_ref_ptr.h"
29#include "rtc_base/thread.h"
Steve Anton9158ef62017-11-27 13:01:52 -080030#include "test/gmock.h"
Henrik Boström933d8b02017-10-10 10:05:16 -070031
32// This file contains tests for RTP Media API-related behavior of
33// |webrtc::PeerConnection|, see https://w3c.github.io/webrtc-pc/#rtp-media-api.
34
Steve Anton9158ef62017-11-27 13:01:52 -080035namespace webrtc {
36
37using RTCConfiguration = PeerConnectionInterface::RTCConfiguration;
38using ::testing::ElementsAre;
39using ::testing::UnorderedElementsAre;
Henrik Boström933d8b02017-10-10 10:05:16 -070040
Henrik Boström31638672017-11-23 17:48:32 +010041const uint32_t kDefaultTimeout = 10000u;
42
43template <typename MethodFunctor>
44class OnSuccessObserver : public rtc::RefCountedObject<
45 webrtc::SetRemoteDescriptionObserverInterface> {
46 public:
47 explicit OnSuccessObserver(MethodFunctor on_success)
48 : on_success_(std::move(on_success)) {}
49
50 // webrtc::SetRemoteDescriptionObserverInterface implementation.
51 void OnSetRemoteDescriptionComplete(webrtc::RTCError error) override {
52 RTC_CHECK(error.ok());
53 on_success_();
54 }
55
56 private:
57 MethodFunctor on_success_;
58};
59
Henrik Boström933d8b02017-10-10 10:05:16 -070060class PeerConnectionRtpTest : public testing::Test {
61 public:
62 PeerConnectionRtpTest()
Steve Anton9158ef62017-11-27 13:01:52 -080063 : pc_factory_(
64 CreatePeerConnectionFactory(rtc::Thread::Current(),
65 rtc::Thread::Current(),
66 rtc::Thread::Current(),
67 FakeAudioCaptureModule::Create(),
68 CreateBuiltinAudioEncoderFactory(),
69 CreateBuiltinAudioDecoderFactory(),
70 nullptr,
71 nullptr)) {}
Henrik Boström933d8b02017-10-10 10:05:16 -070072
Steve Anton9158ef62017-11-27 13:01:52 -080073 std::unique_ptr<PeerConnectionWrapper> CreatePeerConnection() {
74 return CreatePeerConnection(RTCConfiguration());
75 }
76
77 std::unique_ptr<PeerConnectionWrapper> CreatePeerConnectionWithUnifiedPlan() {
78 RTCConfiguration config;
79 config.sdp_semantics = SdpSemantics::kUnifiedPlan;
80 return CreatePeerConnection(config);
81 }
82
83 std::unique_ptr<PeerConnectionWrapper> CreatePeerConnection(
84 const RTCConfiguration& config) {
85 auto observer = rtc::MakeUnique<MockPeerConnectionObserver>();
Henrik Boström933d8b02017-10-10 10:05:16 -070086 auto pc = pc_factory_->CreatePeerConnection(config, nullptr, nullptr,
87 observer.get());
Steve Anton9158ef62017-11-27 13:01:52 -080088 return rtc::MakeUnique<PeerConnectionWrapper>(pc_factory_, pc,
89 std::move(observer));
Henrik Boström933d8b02017-10-10 10:05:16 -070090 }
91
92 protected:
Steve Anton9158ef62017-11-27 13:01:52 -080093 rtc::scoped_refptr<PeerConnectionFactoryInterface> pc_factory_;
Henrik Boström933d8b02017-10-10 10:05:16 -070094};
95
Henrik Boström31638672017-11-23 17:48:32 +010096// These tests cover |webrtc::PeerConnectionObserver| callbacks firing upon
97// setting the remote description.
98class PeerConnectionRtpCallbacksTest : public PeerConnectionRtpTest {};
99
100TEST_F(PeerConnectionRtpCallbacksTest, AddTrackWithoutStreamFiresOnAddTrack) {
Henrik Boström933d8b02017-10-10 10:05:16 -0700101 auto caller = CreatePeerConnection();
102 auto callee = CreatePeerConnection();
103
Steve Anton9158ef62017-11-27 13:01:52 -0800104 rtc::scoped_refptr<AudioTrackInterface> audio_track(
Henrik Boström933d8b02017-10-10 10:05:16 -0700105 pc_factory_->CreateAudioTrack("audio_track", nullptr));
106 EXPECT_TRUE(caller->pc()->AddTrack(audio_track.get(), {}));
Henrik Boström31638672017-11-23 17:48:32 +0100107 ASSERT_TRUE(
108 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
109 static_cast<webrtc::RTCError*>(nullptr)));
Henrik Boström933d8b02017-10-10 10:05:16 -0700110
Henrik Boström31638672017-11-23 17:48:32 +0100111 ASSERT_EQ(callee->observer()->add_track_events_.size(), 1u);
112 // TODO(hbos): When "no stream" is handled correctly we would expect
Henrik Boström933d8b02017-10-10 10:05:16 -0700113 // |add_track_events_[0].streams| to be empty. https://crbug.com/webrtc/7933
Henrik Boström9e6fd2b2017-11-21 13:41:51 +0100114 auto& add_track_event = callee->observer()->add_track_events_[0];
Henrik Boström31638672017-11-23 17:48:32 +0100115 ASSERT_EQ(add_track_event.streams.size(), 1u);
Henrik Boström9e6fd2b2017-11-21 13:41:51 +0100116 EXPECT_TRUE(add_track_event.streams[0]->FindAudioTrack("audio_track"));
117 EXPECT_EQ(add_track_event.streams, add_track_event.receiver->streams());
Henrik Boström933d8b02017-10-10 10:05:16 -0700118}
119
Henrik Boström31638672017-11-23 17:48:32 +0100120TEST_F(PeerConnectionRtpCallbacksTest, AddTrackWithStreamFiresOnAddTrack) {
Henrik Boström933d8b02017-10-10 10:05:16 -0700121 auto caller = CreatePeerConnection();
122 auto callee = CreatePeerConnection();
123
Steve Anton9158ef62017-11-27 13:01:52 -0800124 rtc::scoped_refptr<AudioTrackInterface> audio_track(
Henrik Boström933d8b02017-10-10 10:05:16 -0700125 pc_factory_->CreateAudioTrack("audio_track", nullptr));
Steve Anton9158ef62017-11-27 13:01:52 -0800126 auto stream = MediaStream::Create("audio_stream");
Henrik Boström933d8b02017-10-10 10:05:16 -0700127 EXPECT_TRUE(caller->pc()->AddTrack(audio_track.get(), {stream.get()}));
Henrik Boström31638672017-11-23 17:48:32 +0100128 ASSERT_TRUE(
129 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
130 static_cast<webrtc::RTCError*>(nullptr)));
Henrik Boström933d8b02017-10-10 10:05:16 -0700131
Henrik Boström31638672017-11-23 17:48:32 +0100132 ASSERT_EQ(callee->observer()->add_track_events_.size(), 1u);
Henrik Boström9e6fd2b2017-11-21 13:41:51 +0100133 auto& add_track_event = callee->observer()->add_track_events_[0];
Henrik Boström31638672017-11-23 17:48:32 +0100134 ASSERT_EQ(add_track_event.streams.size(), 1u);
Henrik Boström9e6fd2b2017-11-21 13:41:51 +0100135 EXPECT_EQ("audio_stream", add_track_event.streams[0]->label());
136 EXPECT_TRUE(add_track_event.streams[0]->FindAudioTrack("audio_track"));
137 EXPECT_EQ(add_track_event.streams, add_track_event.receiver->streams());
Henrik Boström933d8b02017-10-10 10:05:16 -0700138}
139
Henrik Boström31638672017-11-23 17:48:32 +0100140TEST_F(PeerConnectionRtpCallbacksTest,
141 RemoveTrackWithoutStreamFiresOnRemoveTrack) {
Henrik Boström933d8b02017-10-10 10:05:16 -0700142 auto caller = CreatePeerConnection();
143 auto callee = CreatePeerConnection();
144
Steve Anton9158ef62017-11-27 13:01:52 -0800145 rtc::scoped_refptr<AudioTrackInterface> audio_track(
Henrik Boström933d8b02017-10-10 10:05:16 -0700146 pc_factory_->CreateAudioTrack("audio_track", nullptr));
147 auto sender = caller->pc()->AddTrack(audio_track.get(), {});
Henrik Boström31638672017-11-23 17:48:32 +0100148 ASSERT_TRUE(
149 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
150 static_cast<webrtc::RTCError*>(nullptr)));
151 ASSERT_EQ(callee->observer()->add_track_events_.size(), 1u);
Henrik Boström933d8b02017-10-10 10:05:16 -0700152 EXPECT_TRUE(caller->pc()->RemoveTrack(sender));
Henrik Boström31638672017-11-23 17:48:32 +0100153 ASSERT_TRUE(
154 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
155 static_cast<webrtc::RTCError*>(nullptr)));
Henrik Boström933d8b02017-10-10 10:05:16 -0700156
Henrik Boström31638672017-11-23 17:48:32 +0100157 ASSERT_EQ(callee->observer()->add_track_events_.size(), 1u);
Henrik Boström933d8b02017-10-10 10:05:16 -0700158 EXPECT_EQ(callee->observer()->GetAddTrackReceivers(),
159 callee->observer()->remove_track_events_);
160}
161
Henrik Boström31638672017-11-23 17:48:32 +0100162TEST_F(PeerConnectionRtpCallbacksTest,
163 RemoveTrackWithStreamFiresOnRemoveTrack) {
Henrik Boström933d8b02017-10-10 10:05:16 -0700164 auto caller = CreatePeerConnection();
165 auto callee = CreatePeerConnection();
166
Steve Anton9158ef62017-11-27 13:01:52 -0800167 rtc::scoped_refptr<AudioTrackInterface> audio_track(
Henrik Boström933d8b02017-10-10 10:05:16 -0700168 pc_factory_->CreateAudioTrack("audio_track", nullptr));
Steve Anton9158ef62017-11-27 13:01:52 -0800169 auto stream = MediaStream::Create("audio_stream");
Henrik Boström933d8b02017-10-10 10:05:16 -0700170 auto sender = caller->pc()->AddTrack(audio_track.get(), {stream.get()});
Henrik Boström31638672017-11-23 17:48:32 +0100171 ASSERT_TRUE(
172 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
173 static_cast<webrtc::RTCError*>(nullptr)));
174 ASSERT_EQ(callee->observer()->add_track_events_.size(), 1u);
Henrik Boström933d8b02017-10-10 10:05:16 -0700175 EXPECT_TRUE(caller->pc()->RemoveTrack(sender));
Henrik Boström31638672017-11-23 17:48:32 +0100176 ASSERT_TRUE(
177 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
178 static_cast<webrtc::RTCError*>(nullptr)));
Henrik Boström933d8b02017-10-10 10:05:16 -0700179
Henrik Boström31638672017-11-23 17:48:32 +0100180 ASSERT_EQ(callee->observer()->add_track_events_.size(), 1u);
Henrik Boström933d8b02017-10-10 10:05:16 -0700181 EXPECT_EQ(callee->observer()->GetAddTrackReceivers(),
182 callee->observer()->remove_track_events_);
183}
184
Henrik Boström31638672017-11-23 17:48:32 +0100185TEST_F(PeerConnectionRtpCallbacksTest,
186 RemoveTrackWithSharedStreamFiresOnRemoveTrack) {
Henrik Boström933d8b02017-10-10 10:05:16 -0700187 auto caller = CreatePeerConnection();
188 auto callee = CreatePeerConnection();
189
Steve Anton9158ef62017-11-27 13:01:52 -0800190 rtc::scoped_refptr<AudioTrackInterface> audio_track1(
Henrik Boström933d8b02017-10-10 10:05:16 -0700191 pc_factory_->CreateAudioTrack("audio_track1", nullptr));
Steve Anton9158ef62017-11-27 13:01:52 -0800192 rtc::scoped_refptr<AudioTrackInterface> audio_track2(
Henrik Boström933d8b02017-10-10 10:05:16 -0700193 pc_factory_->CreateAudioTrack("audio_track2", nullptr));
Steve Anton9158ef62017-11-27 13:01:52 -0800194 auto stream = MediaStream::Create("shared_audio_stream");
195 std::vector<MediaStreamInterface*> streams{stream.get()};
Henrik Boström933d8b02017-10-10 10:05:16 -0700196 auto sender1 = caller->pc()->AddTrack(audio_track1.get(), streams);
197 auto sender2 = caller->pc()->AddTrack(audio_track2.get(), streams);
Henrik Boström31638672017-11-23 17:48:32 +0100198 ASSERT_TRUE(
199 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
200 static_cast<webrtc::RTCError*>(nullptr)));
Henrik Boström933d8b02017-10-10 10:05:16 -0700201
Henrik Boström31638672017-11-23 17:48:32 +0100202 ASSERT_EQ(callee->observer()->add_track_events_.size(), 2u);
Henrik Boström933d8b02017-10-10 10:05:16 -0700203
204 // Remove "audio_track1".
205 EXPECT_TRUE(caller->pc()->RemoveTrack(sender1));
Henrik Boström31638672017-11-23 17:48:32 +0100206 ASSERT_TRUE(
207 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
208 static_cast<webrtc::RTCError*>(nullptr)));
209 ASSERT_EQ(callee->observer()->add_track_events_.size(), 2u);
Henrik Boström933d8b02017-10-10 10:05:16 -0700210 EXPECT_EQ(
Steve Anton9158ef62017-11-27 13:01:52 -0800211 std::vector<rtc::scoped_refptr<RtpReceiverInterface>>{
Henrik Boström933d8b02017-10-10 10:05:16 -0700212 callee->observer()->add_track_events_[0].receiver},
213 callee->observer()->remove_track_events_);
214
215 // Remove "audio_track2".
216 EXPECT_TRUE(caller->pc()->RemoveTrack(sender2));
Henrik Boström31638672017-11-23 17:48:32 +0100217 ASSERT_TRUE(
218 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
219 static_cast<webrtc::RTCError*>(nullptr)));
220 ASSERT_EQ(callee->observer()->add_track_events_.size(), 2u);
Henrik Boström933d8b02017-10-10 10:05:16 -0700221 EXPECT_EQ(callee->observer()->GetAddTrackReceivers(),
222 callee->observer()->remove_track_events_);
223}
224
Henrik Boström31638672017-11-23 17:48:32 +0100225// These tests examine the state of the peer connection as a result of
226// performing SetRemoteDescription().
227class PeerConnectionRtpObserverTest : public PeerConnectionRtpTest {};
228
229TEST_F(PeerConnectionRtpObserverTest, AddSenderWithoutStreamAddsReceiver) {
230 auto caller = CreatePeerConnection();
231 auto callee = CreatePeerConnection();
232
233 rtc::scoped_refptr<webrtc::AudioTrackInterface> audio_track(
234 pc_factory_->CreateAudioTrack("audio_track", nullptr));
235 EXPECT_TRUE(caller->pc()->AddTrack(audio_track.get(), {}));
236 ASSERT_TRUE(
237 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
238 static_cast<webrtc::RTCError*>(nullptr)));
239
240 EXPECT_EQ(callee->pc()->GetReceivers().size(), 1u);
241 auto receiver_added = callee->pc()->GetReceivers()[0];
242 EXPECT_EQ("audio_track", receiver_added->track()->id());
243 // TODO(hbos): When "no stream" is handled correctly we would expect
244 // |receiver_added->streams()| to be empty. https://crbug.com/webrtc/7933
245 EXPECT_EQ(receiver_added->streams().size(), 1u);
246 EXPECT_TRUE(receiver_added->streams()[0]->FindAudioTrack("audio_track"));
247}
248
249TEST_F(PeerConnectionRtpObserverTest, AddSenderWithStreamAddsReceiver) {
250 auto caller = CreatePeerConnection();
251 auto callee = CreatePeerConnection();
252
253 rtc::scoped_refptr<webrtc::AudioTrackInterface> audio_track(
254 pc_factory_->CreateAudioTrack("audio_track", nullptr));
255 auto stream = webrtc::MediaStream::Create("audio_stream");
256 EXPECT_TRUE(caller->pc()->AddTrack(audio_track.get(), {stream}));
257 ASSERT_TRUE(
258 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
259 static_cast<webrtc::RTCError*>(nullptr)));
260
261 EXPECT_EQ(callee->pc()->GetReceivers().size(), 1u);
262 auto receiver_added = callee->pc()->GetReceivers()[0];
263 EXPECT_EQ("audio_track", receiver_added->track()->id());
264 EXPECT_EQ(receiver_added->streams().size(), 1u);
265 EXPECT_EQ("audio_stream", receiver_added->streams()[0]->label());
266 EXPECT_TRUE(receiver_added->streams()[0]->FindAudioTrack("audio_track"));
267}
268
269TEST_F(PeerConnectionRtpObserverTest,
270 RemoveSenderWithoutStreamRemovesReceiver) {
271 auto caller = CreatePeerConnection();
272 auto callee = CreatePeerConnection();
273
274 rtc::scoped_refptr<webrtc::AudioTrackInterface> audio_track(
275 pc_factory_->CreateAudioTrack("audio_track", nullptr));
276 auto sender = caller->pc()->AddTrack(audio_track.get(), {});
277 ASSERT_TRUE(sender);
278 ASSERT_TRUE(
279 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
280 static_cast<webrtc::RTCError*>(nullptr)));
281 ASSERT_EQ(callee->pc()->GetReceivers().size(), 1u);
282 auto receiver = callee->pc()->GetReceivers()[0];
283 ASSERT_TRUE(caller->pc()->RemoveTrack(sender));
284 ASSERT_TRUE(
285 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
286 static_cast<webrtc::RTCError*>(nullptr)));
287
288 // TODO(hbos): When we implement Unified Plan, receivers will not be removed.
289 // Instead, the transceiver owning the receiver will become inactive.
290 EXPECT_EQ(callee->pc()->GetReceivers().size(), 0u);
291}
292
293TEST_F(PeerConnectionRtpObserverTest, RemoveSenderWithStreamRemovesReceiver) {
294 auto caller = CreatePeerConnection();
295 auto callee = CreatePeerConnection();
296
297 rtc::scoped_refptr<webrtc::AudioTrackInterface> audio_track(
298 pc_factory_->CreateAudioTrack("audio_track", nullptr));
299 auto stream = webrtc::MediaStream::Create("audio_stream");
300 auto sender = caller->pc()->AddTrack(audio_track.get(), {stream});
301 ASSERT_TRUE(sender);
302 ASSERT_TRUE(
303 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
304 static_cast<webrtc::RTCError*>(nullptr)));
305 ASSERT_EQ(callee->pc()->GetReceivers().size(), 1u);
306 auto receiver = callee->pc()->GetReceivers()[0];
307 ASSERT_TRUE(caller->pc()->RemoveTrack(sender));
308 ASSERT_TRUE(
309 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
310 static_cast<webrtc::RTCError*>(nullptr)));
311
312 // TODO(hbos): When we implement Unified Plan, receivers will not be removed.
313 // Instead, the transceiver owning the receiver will become inactive.
314 EXPECT_EQ(callee->pc()->GetReceivers().size(), 0u);
315}
316
317TEST_F(PeerConnectionRtpObserverTest,
318 RemoveSenderWithSharedStreamRemovesReceiver) {
319 auto caller = CreatePeerConnection();
320 auto callee = CreatePeerConnection();
321
322 rtc::scoped_refptr<webrtc::AudioTrackInterface> audio_track1(
323 pc_factory_->CreateAudioTrack("audio_track1", nullptr));
324 rtc::scoped_refptr<webrtc::AudioTrackInterface> audio_track2(
325 pc_factory_->CreateAudioTrack("audio_track2", nullptr));
326 auto stream = webrtc::MediaStream::Create("shared_audio_stream");
327 std::vector<webrtc::MediaStreamInterface*> streams{stream.get()};
328 auto sender1 = caller->pc()->AddTrack(audio_track1.get(), streams);
329 auto sender2 = caller->pc()->AddTrack(audio_track2.get(), streams);
330 ASSERT_TRUE(
331 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
332 static_cast<webrtc::RTCError*>(nullptr)));
333
334 ASSERT_EQ(callee->pc()->GetReceivers().size(), 2u);
335 rtc::scoped_refptr<webrtc::RtpReceiverInterface> receiver1;
336 rtc::scoped_refptr<webrtc::RtpReceiverInterface> receiver2;
337 if (callee->pc()->GetReceivers()[0]->track()->id() == "audio_track1") {
338 receiver1 = callee->pc()->GetReceivers()[0];
339 receiver2 = callee->pc()->GetReceivers()[1];
340 } else {
341 receiver1 = callee->pc()->GetReceivers()[1];
342 receiver2 = callee->pc()->GetReceivers()[0];
343 }
344 EXPECT_EQ("audio_track1", receiver1->track()->id());
345 EXPECT_EQ("audio_track2", receiver2->track()->id());
346
347 // Remove "audio_track1".
348 EXPECT_TRUE(caller->pc()->RemoveTrack(sender1));
349 ASSERT_TRUE(
350 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
351 static_cast<webrtc::RTCError*>(nullptr)));
352 // Only |receiver2| should remain.
353 // TODO(hbos): When we implement Unified Plan, receivers will not be removed.
354 // Instead, the transceiver owning the receiver will become inactive.
355 EXPECT_EQ(
356 std::vector<rtc::scoped_refptr<webrtc::RtpReceiverInterface>>{receiver2},
357 callee->pc()->GetReceivers());
358
359 // Remove "audio_track2".
360 EXPECT_TRUE(caller->pc()->RemoveTrack(sender2));
361 ASSERT_TRUE(
362 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(),
363 static_cast<webrtc::RTCError*>(nullptr)));
364 // TODO(hbos): When we implement Unified Plan, receivers will not be removed.
365 // Instead, the transceiver owning the receiver will become inactive.
366 EXPECT_EQ(callee->pc()->GetReceivers().size(), 0u);
367}
368
369// Invokes SetRemoteDescription() twice in a row without synchronizing the two
370// calls and examine the state of the peer connection inside the callbacks to
371// ensure that the second call does not occur prematurely, contaminating the
372// state of the peer connection of the first callback.
373TEST_F(PeerConnectionRtpObserverTest,
374 StatesCorrelateWithSetRemoteDescriptionCall) {
375 auto caller = CreatePeerConnection();
376 auto callee = CreatePeerConnection();
377
378 rtc::scoped_refptr<webrtc::AudioTrackInterface> audio_track(
379 pc_factory_->CreateAudioTrack("audio_track", nullptr));
380 // Create SDP for adding a track and for removing it. This will be used in the
381 // first and second SetRemoteDescription() calls.
382 auto sender = caller->pc()->AddTrack(audio_track.get(), {});
383 auto srd1_sdp = caller->CreateOfferAndSetAsLocal();
384 EXPECT_TRUE(caller->pc()->RemoveTrack(sender));
385 auto srd2_sdp = caller->CreateOfferAndSetAsLocal();
386
387 // In the first SetRemoteDescription() callback, check that we have a
388 // receiver for the track.
389 auto pc = callee->pc();
390 bool srd1_callback_called = false;
391 auto srd1_callback = [&srd1_callback_called, &pc]() {
392 EXPECT_EQ(pc->GetReceivers().size(), 1u);
393 srd1_callback_called = true;
394 };
395
396 // In the second SetRemoteDescription() callback, check that the receiver has
397 // been removed.
398 // TODO(hbos): When we implement Unified Plan, receivers will not be removed.
399 // Instead, the transceiver owning the receiver will become inactive.
400 // https://crbug.com/webrtc/7600
401 bool srd2_callback_called = false;
402 auto srd2_callback = [&srd2_callback_called, &pc]() {
403 EXPECT_TRUE(pc->GetReceivers().empty());
404 srd2_callback_called = true;
405 };
406
407 // Invoke SetRemoteDescription() twice in a row without synchronizing the two
408 // calls. The callbacks verify that the two calls are synchronized, as in, the
409 // effects of the second SetRemoteDescription() call must not have happened by
410 // the time the first callback is invoked. If it has then the receiver that is
411 // added as a result of the first SetRemoteDescription() call will already
412 // have been removed as a result of the second SetRemoteDescription() call
413 // when the first callback is invoked.
414 callee->pc()->SetRemoteDescription(
415 std::move(srd1_sdp),
416 new OnSuccessObserver<decltype(srd1_callback)>(srd1_callback));
417 callee->pc()->SetRemoteDescription(
418 std::move(srd2_sdp),
419 new OnSuccessObserver<decltype(srd2_callback)>(srd2_callback));
420 EXPECT_TRUE_WAIT(srd1_callback_called, kDefaultTimeout);
421 EXPECT_TRUE_WAIT(srd2_callback_called, kDefaultTimeout);
422}
423
424// Tests for the legacy SetRemoteDescription() function signature.
425class PeerConnectionRtpLegacyObserverTest : public PeerConnectionRtpTest {};
426
427// Sanity test making sure the callback is invoked.
428TEST_F(PeerConnectionRtpLegacyObserverTest, OnSuccess) {
429 auto caller = CreatePeerConnection();
430 auto callee = CreatePeerConnection();
431
432 std::string error;
433 ASSERT_TRUE(
434 callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal(), &error));
435}
436
437// Verifies legacy behavior: The observer is not called if if the peer
438// connection is destroyed because the asynchronous callback is executed in the
439// peer connection's message handler.
440TEST_F(PeerConnectionRtpLegacyObserverTest,
441 ObserverNotCalledIfPeerConnectionDereferenced) {
442 auto caller = CreatePeerConnection();
443 auto callee = CreatePeerConnection();
444
445 rtc::scoped_refptr<webrtc::MockSetSessionDescriptionObserver> observer =
446 new rtc::RefCountedObject<webrtc::MockSetSessionDescriptionObserver>();
447
448 auto offer = caller->CreateOfferAndSetAsLocal();
449 callee->pc()->SetRemoteDescription(observer, offer.release());
450 callee = nullptr;
451 rtc::Thread::Current()->ProcessMessages(0);
452 EXPECT_FALSE(observer->called());
453}
454
Steve Anton9158ef62017-11-27 13:01:52 -0800455// RtpTransceiver Tests
456
457// Test that by default there are no transceivers with Unified Plan.
458TEST_F(PeerConnectionRtpTest, PeerConnectionHasNoTransceivers) {
459 auto caller = CreatePeerConnectionWithUnifiedPlan();
460 EXPECT_THAT(caller->pc()->GetTransceivers(), ElementsAre());
461}
462
463// Test that a transceiver created with the audio kind has the correct initial
464// properties.
465TEST_F(PeerConnectionRtpTest, AddTransceiverHasCorrectInitProperties) {
466 auto caller = CreatePeerConnectionWithUnifiedPlan();
467
468 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
469 EXPECT_EQ(rtc::nullopt, transceiver->mid());
470 EXPECT_FALSE(transceiver->stopped());
471 EXPECT_EQ(RtpTransceiverDirection::kSendRecv, transceiver->direction());
472 EXPECT_EQ(rtc::nullopt, transceiver->current_direction());
473}
474
475// Test that adding a transceiver with the audio kind creates an audio sender
476// and audio receiver with the receiver having a live audio track.
477TEST_F(PeerConnectionRtpTest,
478 AddAudioTransceiverCreatesAudioSenderAndReceiver) {
479 auto caller = CreatePeerConnectionWithUnifiedPlan();
480
481 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
482
483 ASSERT_TRUE(transceiver->sender());
484 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO, transceiver->sender()->media_type());
485
486 ASSERT_TRUE(transceiver->receiver());
487 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO, transceiver->receiver()->media_type());
488
489 auto track = transceiver->receiver()->track();
490 ASSERT_TRUE(track);
491 EXPECT_EQ(MediaStreamTrackInterface::kAudioKind, track->kind());
492 EXPECT_EQ(MediaStreamTrackInterface::TrackState::kLive, track->state());
493}
494
495// Test that adding a transceiver with the video kind creates an video sender
496// and video receiver with the receiver having a live video track.
497TEST_F(PeerConnectionRtpTest,
498 AddAudioTransceiverCreatesVideoSenderAndReceiver) {
499 auto caller = CreatePeerConnectionWithUnifiedPlan();
500
501 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO);
502
503 ASSERT_TRUE(transceiver->sender());
504 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, transceiver->sender()->media_type());
505
506 ASSERT_TRUE(transceiver->receiver());
507 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, transceiver->receiver()->media_type());
508
509 auto track = transceiver->receiver()->track();
510 ASSERT_TRUE(track);
511 EXPECT_EQ(MediaStreamTrackInterface::kVideoKind, track->kind());
512 EXPECT_EQ(MediaStreamTrackInterface::TrackState::kLive, track->state());
513}
514
515// Test that after a call to AddTransceiver, the transceiver shows in
516// GetTransceivers(), the transceiver's sender shows in GetSenders(), and the
517// transceiver's receiver shows in GetReceivers().
518TEST_F(PeerConnectionRtpTest, AddTransceiverShowsInLists) {
519 auto caller = CreatePeerConnectionWithUnifiedPlan();
520
521 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
522 EXPECT_EQ(
523 std::vector<rtc::scoped_refptr<RtpTransceiverInterface>>{transceiver},
524 caller->pc()->GetTransceivers());
525 EXPECT_EQ(
526 std::vector<rtc::scoped_refptr<RtpSenderInterface>>{
527 transceiver->sender()},
528 caller->pc()->GetSenders());
529 EXPECT_EQ(
530 std::vector<rtc::scoped_refptr<RtpReceiverInterface>>{
531 transceiver->receiver()},
532 caller->pc()->GetReceivers());
533}
534
535// Test that the direction passed in through the AddTransceiver init parameter
536// is set in the returned transceiver.
537TEST_F(PeerConnectionRtpTest, AddTransceiverWithDirectionIsReflected) {
538 auto caller = CreatePeerConnectionWithUnifiedPlan();
539
540 RtpTransceiverInit init;
541 init.direction = RtpTransceiverDirection::kSendOnly;
542 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO, init);
543 EXPECT_EQ(RtpTransceiverDirection::kSendOnly, transceiver->direction());
544}
545
546TEST_F(PeerConnectionRtpTest, AddTransceiverWithInvalidKindReturnsError) {
547 auto caller = CreatePeerConnectionWithUnifiedPlan();
548
549 auto result = caller->pc()->AddTransceiver(cricket::MEDIA_TYPE_DATA);
550 EXPECT_EQ(RTCErrorType::INVALID_PARAMETER, result.error().type());
551}
552
553// Test that calling AddTransceiver with a track creates a transceiver which has
554// its sender's track set to the passed-in track.
555TEST_F(PeerConnectionRtpTest, AddTransceiverWithTrackCreatesSenderWithTrack) {
556 auto caller = CreatePeerConnectionWithUnifiedPlan();
557
558 auto audio_track = caller->CreateAudioTrack("audio track");
559 auto transceiver = caller->AddTransceiver(audio_track);
560
561 auto sender = transceiver->sender();
562 ASSERT_TRUE(sender->track());
563 EXPECT_EQ(audio_track, sender->track());
564
565 auto receiver = transceiver->receiver();
566 ASSERT_TRUE(receiver->track());
567 EXPECT_EQ(MediaStreamTrackInterface::kAudioKind, receiver->track()->kind());
568 EXPECT_EQ(MediaStreamTrackInterface::TrackState::kLive,
569 receiver->track()->state());
570}
571
572// Test that calling AddTransceiver twice with the same track creates distinct
573// transceivers, senders with the same track.
574TEST_F(PeerConnectionRtpTest,
575 AddTransceiverTwiceWithSameTrackCreatesMultipleTransceivers) {
576 auto caller = CreatePeerConnectionWithUnifiedPlan();
577
578 auto audio_track = caller->CreateAudioTrack("audio track");
579
580 auto transceiver1 = caller->AddTransceiver(audio_track);
581 auto transceiver2 = caller->AddTransceiver(audio_track);
582
583 EXPECT_NE(transceiver1, transceiver2);
584
585 auto sender1 = transceiver1->sender();
586 auto sender2 = transceiver2->sender();
587 EXPECT_NE(sender1, sender2);
588 EXPECT_EQ(audio_track, sender1->track());
589 EXPECT_EQ(audio_track, sender2->track());
590
591 EXPECT_THAT(caller->pc()->GetTransceivers(),
592 UnorderedElementsAre(transceiver1, transceiver2));
593 EXPECT_THAT(caller->pc()->GetSenders(),
594 UnorderedElementsAre(sender1, sender2));
595}
596
597} // namespace webrtc