blob: ae51c9e565c88dfdcb8e75629e7f0c2d6c12eb5f [file] [log] [blame]
Steve Antondcc3c022017-12-22 16:02:54 -08001/*
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 "api/audio_codecs/builtin_audio_decoder_factory.h"
12#include "api/audio_codecs/builtin_audio_encoder_factory.h"
Steve Antonfa2260d2017-12-28 16:38:23 -080013#include "media/engine/webrtcmediaengine.h"
14#include "modules/audio_processing/include/audio_processing.h"
Steve Antondcc3c022017-12-22 16:02:54 -080015#include "pc/mediasession.h"
Steve Antonfa2260d2017-12-28 16:38:23 -080016#include "pc/peerconnectionfactory.h"
Steve Antondcc3c022017-12-22 16:02:54 -080017#include "pc/peerconnectionwrapper.h"
18#include "pc/sdputils.h"
19#ifdef WEBRTC_ANDROID
20#include "pc/test/androidtestinitializer.h"
21#endif
22#include "pc/test/fakeaudiocapturemodule.h"
Steve Antonfa2260d2017-12-28 16:38:23 -080023#include "pc/test/fakesctptransport.h"
Steve Antondcc3c022017-12-22 16:02:54 -080024#include "rtc_base/gunit.h"
25#include "rtc_base/ptr_util.h"
26#include "rtc_base/virtualsocketserver.h"
27#include "test/gmock.h"
28
29// This file contains tests that ensure the PeerConnection's implementation of
30// CreateOffer/CreateAnswer/SetLocalDescription/SetRemoteDescription conform
31// to the JavaScript Session Establishment Protocol (JSEP).
32// For now these semantics are only available when configuring the
33// PeerConnection with Unified Plan, but eventually that will be the default.
34
35namespace webrtc {
36
37using cricket::MediaContentDescription;
38using RTCConfiguration = PeerConnectionInterface::RTCConfiguration;
39using ::testing::Values;
40using ::testing::Combine;
41using ::testing::ElementsAre;
Steve Antonef65ef12018-01-10 17:15:20 -080042using ::testing::UnorderedElementsAre;
Steve Antondcc3c022017-12-22 16:02:54 -080043
Steve Antonfa2260d2017-12-28 16:38:23 -080044class PeerConnectionFactoryForJsepTest : public PeerConnectionFactory {
45 public:
46 PeerConnectionFactoryForJsepTest()
47 : PeerConnectionFactory(
48 rtc::Thread::Current(),
49 rtc::Thread::Current(),
50 rtc::Thread::Current(),
51 rtc::WrapUnique(cricket::WebRtcMediaEngineFactory::Create(
52 FakeAudioCaptureModule::Create(),
53 CreateBuiltinAudioEncoderFactory(),
54 CreateBuiltinAudioDecoderFactory(),
55 nullptr,
56 nullptr,
57 nullptr,
Ivo Creusen62337e52018-01-09 14:17:33 +010058 AudioProcessingBuilder().Create())),
Steve Antonfa2260d2017-12-28 16:38:23 -080059 CreateCallFactory(),
60 nullptr) {}
61
62 std::unique_ptr<cricket::SctpTransportInternalFactory>
63 CreateSctpTransportInternalFactory() {
64 return rtc::MakeUnique<FakeSctpTransportFactory>();
65 }
66};
67
Steve Antondcc3c022017-12-22 16:02:54 -080068class PeerConnectionJsepTest : public ::testing::Test {
69 protected:
70 typedef std::unique_ptr<PeerConnectionWrapper> WrapperPtr;
71
72 PeerConnectionJsepTest()
73 : vss_(new rtc::VirtualSocketServer()), main_(vss_.get()) {
74#ifdef WEBRTC_ANDROID
75 InitializeAndroidObjects();
76#endif
Steve Antondcc3c022017-12-22 16:02:54 -080077 }
78
79 WrapperPtr CreatePeerConnection() {
80 RTCConfiguration config;
81 config.sdp_semantics = SdpSemantics::kUnifiedPlan;
82 return CreatePeerConnection(config);
83 }
84
85 WrapperPtr CreatePeerConnection(const RTCConfiguration& config) {
Steve Antonfa2260d2017-12-28 16:38:23 -080086 rtc::scoped_refptr<PeerConnectionFactory> pc_factory(
87 new rtc::RefCountedObject<PeerConnectionFactoryForJsepTest>());
88 RTC_CHECK(pc_factory->Initialize());
Steve Antondcc3c022017-12-22 16:02:54 -080089 auto observer = rtc::MakeUnique<MockPeerConnectionObserver>();
Steve Antonfa2260d2017-12-28 16:38:23 -080090 auto pc = pc_factory->CreatePeerConnection(config, nullptr, nullptr,
91 observer.get());
Steve Antondcc3c022017-12-22 16:02:54 -080092 if (!pc) {
93 return nullptr;
94 }
95
Steve Antonfa2260d2017-12-28 16:38:23 -080096 return rtc::MakeUnique<PeerConnectionWrapper>(pc_factory, pc,
Steve Antondcc3c022017-12-22 16:02:54 -080097 std::move(observer));
98 }
99
100 std::unique_ptr<rtc::VirtualSocketServer> vss_;
101 rtc::AutoSocketServerThread main_;
Steve Antondcc3c022017-12-22 16:02:54 -0800102};
103
104// Tests for JSEP initial offer generation.
105
106// Test that an offer created by a PeerConnection with no transceivers generates
107// no media sections.
108TEST_F(PeerConnectionJsepTest, EmptyInitialOffer) {
109 auto caller = CreatePeerConnection();
Steve Antonfa2260d2017-12-28 16:38:23 -0800110
Steve Antondcc3c022017-12-22 16:02:54 -0800111 auto offer = caller->CreateOffer();
Steve Antonfa2260d2017-12-28 16:38:23 -0800112 ASSERT_EQ(0u, offer->description()->contents().size());
Steve Antondcc3c022017-12-22 16:02:54 -0800113}
114
115// Test that an initial offer with one audio track generates one audio media
116// section.
117TEST_F(PeerConnectionJsepTest, AudioOnlyInitialOffer) {
118 auto caller = CreatePeerConnection();
119 caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
Steve Antondcc3c022017-12-22 16:02:54 -0800120
Steve Antonfa2260d2017-12-28 16:38:23 -0800121 auto offer = caller->CreateOffer();
Steve Antondcc3c022017-12-22 16:02:54 -0800122 auto contents = offer->description()->contents();
123 ASSERT_EQ(1u, contents.size());
124 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO, contents[0].media_description()->type());
125}
126
127// Test than an initial offer with one video track generates one video media
128// section
129TEST_F(PeerConnectionJsepTest, VideoOnlyInitialOffer) {
130 auto caller = CreatePeerConnection();
131 caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO);
Steve Antondcc3c022017-12-22 16:02:54 -0800132
Steve Antonfa2260d2017-12-28 16:38:23 -0800133 auto offer = caller->CreateOffer();
Steve Antondcc3c022017-12-22 16:02:54 -0800134 auto contents = offer->description()->contents();
135 ASSERT_EQ(1u, contents.size());
136 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, contents[0].media_description()->type());
137}
138
Steve Antonfa2260d2017-12-28 16:38:23 -0800139// Test that an initial offer with one data channel generates one data media
140// section.
141TEST_F(PeerConnectionJsepTest, DataOnlyInitialOffer) {
142 auto caller = CreatePeerConnection();
143 caller->CreateDataChannel("dc");
144
145 auto offer = caller->CreateOffer();
146 auto contents = offer->description()->contents();
147 ASSERT_EQ(1u, contents.size());
148 EXPECT_EQ(cricket::MEDIA_TYPE_DATA, contents[0].media_description()->type());
149}
150
151// Test that creating multiple data channels only results in one data section
152// generated in the offer.
153TEST_F(PeerConnectionJsepTest, MultipleDataChannelsCreateOnlyOneDataSection) {
154 auto caller = CreatePeerConnection();
155 caller->CreateDataChannel("first");
156 caller->CreateDataChannel("second");
157 caller->CreateDataChannel("third");
158
159 auto offer = caller->CreateOffer();
160 ASSERT_EQ(1u, offer->description()->contents().size());
161}
162
Steve Antondcc3c022017-12-22 16:02:54 -0800163// Test that multiple media sections in the initial offer are ordered in the
164// order the transceivers were added to the PeerConnection. This is required by
165// JSEP section 5.2.1.
166TEST_F(PeerConnectionJsepTest, MediaSectionsInInitialOfferOrderedCorrectly) {
167 auto caller = CreatePeerConnection();
168 caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO);
169 caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
170 RtpTransceiverInit init;
171 init.direction = RtpTransceiverDirection::kSendOnly;
172 caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO, init);
Steve Antondcc3c022017-12-22 16:02:54 -0800173
Steve Antonfa2260d2017-12-28 16:38:23 -0800174 auto offer = caller->CreateOffer();
Steve Antondcc3c022017-12-22 16:02:54 -0800175 auto contents = offer->description()->contents();
176 ASSERT_EQ(3u, contents.size());
177
178 const MediaContentDescription* media_description1 =
179 contents[0].media_description();
180 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, media_description1->type());
181 EXPECT_EQ(RtpTransceiverDirection::kSendRecv,
182 media_description1->direction());
183
184 const MediaContentDescription* media_description2 =
185 contents[1].media_description();
186 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO, media_description2->type());
187 EXPECT_EQ(RtpTransceiverDirection::kSendRecv,
188 media_description2->direction());
189
190 const MediaContentDescription* media_description3 =
191 contents[2].media_description();
192 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, media_description3->type());
193 EXPECT_EQ(RtpTransceiverDirection::kSendOnly,
194 media_description3->direction());
195}
196
197// Test that media sections in the initial offer have different mids.
198TEST_F(PeerConnectionJsepTest, MediaSectionsInInitialOfferHaveDifferentMids) {
199 auto caller = CreatePeerConnection();
200 caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
201 caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
Steve Antonfa2260d2017-12-28 16:38:23 -0800202
Steve Antondcc3c022017-12-22 16:02:54 -0800203 auto offer = caller->CreateOffer();
Steve Antondcc3c022017-12-22 16:02:54 -0800204 auto contents = offer->description()->contents();
205 ASSERT_EQ(2u, contents.size());
206 EXPECT_NE(contents[0].name, contents[1].name);
207}
208
209TEST_F(PeerConnectionJsepTest,
210 StoppedTransceiverHasNoMediaSectionInInitialOffer) {
211 auto caller = CreatePeerConnection();
212 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
213 transceiver->Stop();
214
215 auto offer = caller->CreateOffer();
216 EXPECT_EQ(0u, offer->description()->contents().size());
217}
218
219// Tests for JSEP SetLocalDescription with a local offer.
220
221TEST_F(PeerConnectionJsepTest, SetLocalEmptyOfferCreatesNoTransceivers) {
222 auto caller = CreatePeerConnection();
Steve Antonfa2260d2017-12-28 16:38:23 -0800223
Steve Antondcc3c022017-12-22 16:02:54 -0800224 ASSERT_TRUE(caller->SetLocalDescription(caller->CreateOffer()));
225
226 EXPECT_THAT(caller->pc()->GetTransceivers(), ElementsAre());
227 EXPECT_THAT(caller->pc()->GetSenders(), ElementsAre());
228 EXPECT_THAT(caller->pc()->GetReceivers(), ElementsAre());
229}
230
231TEST_F(PeerConnectionJsepTest, SetLocalOfferSetsTransceiverMid) {
232 auto caller = CreatePeerConnection();
233 auto audio_transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
234 auto video_transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO);
235
236 auto offer = caller->CreateOffer();
237 std::string audio_mid = offer->description()->contents()[0].name;
238 std::string video_mid = offer->description()->contents()[1].name;
239
240 ASSERT_TRUE(caller->SetLocalDescription(std::move(offer)));
241
242 EXPECT_EQ(audio_mid, audio_transceiver->mid());
243 EXPECT_EQ(video_mid, video_transceiver->mid());
244}
245
246// Tests for JSEP SetRemoteDescription with a remote offer.
247
248// Test that setting a remote offer with sendrecv audio and video creates two
249// transceivers, one for receiving audio and one for receiving video.
250TEST_F(PeerConnectionJsepTest, SetRemoteOfferCreatesTransceivers) {
251 auto caller = CreatePeerConnection();
252 auto caller_audio = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
253 auto caller_video = caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO);
254 auto callee = CreatePeerConnection();
255
256 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
257
258 auto transceivers = callee->pc()->GetTransceivers();
259 ASSERT_EQ(2u, transceivers.size());
Steve Anton69470252018-02-09 11:43:08 -0800260 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO, transceivers[0]->media_type());
Steve Antondcc3c022017-12-22 16:02:54 -0800261 EXPECT_EQ(caller_audio->mid(), transceivers[0]->mid());
262 EXPECT_EQ(RtpTransceiverDirection::kRecvOnly, transceivers[0]->direction());
Steve Anton69470252018-02-09 11:43:08 -0800263 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, transceivers[1]->media_type());
Steve Antondcc3c022017-12-22 16:02:54 -0800264 EXPECT_EQ(caller_video->mid(), transceivers[1]->mid());
265 EXPECT_EQ(RtpTransceiverDirection::kRecvOnly, transceivers[1]->direction());
266}
267
268// Test that setting a remote offer with an audio track will reuse the
269// transceiver created for a local audio track added by AddTrack.
270// This is specified in JSEP section 5.10 (Applying a Remote Description). The
271// intent is to preserve backwards compatibility with clients who only use the
272// AddTrack API.
273TEST_F(PeerConnectionJsepTest, SetRemoteOfferReusesTransceiverFromAddTrack) {
274 auto caller = CreatePeerConnection();
275 caller->AddAudioTrack("a");
276 auto caller_audio = caller->pc()->GetTransceivers()[0];
277 auto callee = CreatePeerConnection();
278 callee->AddAudioTrack("a");
279
280 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
281
282 auto transceivers = callee->pc()->GetTransceivers();
283 ASSERT_EQ(1u, transceivers.size());
284 EXPECT_EQ(MediaStreamTrackInterface::kAudioKind,
285 transceivers[0]->receiver()->track()->kind());
286 EXPECT_EQ(caller_audio->mid(), transceivers[0]->mid());
287}
288
289// Test that setting a remote offer with an audio track marked sendonly will not
290// reuse a transceiver created by AddTrack. JSEP only allows the transceiver to
291// be reused if the offer direction is sendrecv or recvonly.
292TEST_F(PeerConnectionJsepTest,
293 SetRemoteOfferDoesNotReuseTransceiverIfDirectionSendOnly) {
294 auto caller = CreatePeerConnection();
295 caller->AddAudioTrack("a");
296 auto caller_audio = caller->pc()->GetTransceivers()[0];
297 caller_audio->SetDirection(RtpTransceiverDirection::kSendOnly);
298 auto callee = CreatePeerConnection();
299 callee->AddAudioTrack("a");
300
301 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
302
303 auto transceivers = callee->pc()->GetTransceivers();
304 ASSERT_EQ(2u, transceivers.size());
305 EXPECT_EQ(rtc::nullopt, transceivers[0]->mid());
306 EXPECT_EQ(caller_audio->mid(), transceivers[1]->mid());
307}
308
309// Test that setting a remote offer with an audio track will not reuse a
310// transceiver added by AddTransceiver. The logic for reusing a transceiver is
311// specific to those added by AddTrack and is tested above.
312TEST_F(PeerConnectionJsepTest,
313 SetRemoteOfferDoesNotReuseTransceiverFromAddTransceiver) {
314 auto caller = CreatePeerConnection();
315 caller->AddAudioTrack("a");
316 auto callee = CreatePeerConnection();
317 auto transceiver = callee->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
318
319 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
320
321 auto transceivers = callee->pc()->GetTransceivers();
322 ASSERT_EQ(2u, transceivers.size());
323 EXPECT_EQ(rtc::nullopt, transceivers[0]->mid());
324 EXPECT_EQ(caller->pc()->GetTransceivers()[0]->mid(), transceivers[1]->mid());
325 EXPECT_EQ(MediaStreamTrackInterface::kAudioKind,
326 transceivers[1]->receiver()->track()->kind());
327}
328
329// Test that setting a remote offer with an audio track will not reuse a
330// transceiver created for a local video track added by AddTrack.
331TEST_F(PeerConnectionJsepTest,
332 SetRemoteOfferDoesNotReuseTransceiverOfWrongType) {
333 auto caller = CreatePeerConnection();
334 caller->AddAudioTrack("a");
335 auto callee = CreatePeerConnection();
336 auto video_sender = callee->AddVideoTrack("v");
337
338 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
339
340 auto transceivers = callee->pc()->GetTransceivers();
341 ASSERT_EQ(2u, transceivers.size());
342 EXPECT_EQ(rtc::nullopt, transceivers[0]->mid());
343 EXPECT_EQ(caller->pc()->GetTransceivers()[0]->mid(), transceivers[1]->mid());
344 EXPECT_EQ(MediaStreamTrackInterface::kAudioKind,
345 transceivers[1]->receiver()->track()->kind());
346}
347
348// Test that setting a remote offer with an audio track will not reuse a
349// stopped transceiver.
350TEST_F(PeerConnectionJsepTest, SetRemoteOfferDoesNotReuseStoppedTransceiver) {
351 auto caller = CreatePeerConnection();
352 caller->AddAudioTrack("a");
353 auto callee = CreatePeerConnection();
354 callee->AddAudioTrack("a");
355 callee->pc()->GetTransceivers()[0]->Stop();
356
357 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
358
359 auto transceivers = callee->pc()->GetTransceivers();
360 ASSERT_EQ(2u, transceivers.size());
361 EXPECT_EQ(rtc::nullopt, transceivers[0]->mid());
362 EXPECT_TRUE(transceivers[0]->stopped());
363 EXPECT_EQ(caller->pc()->GetTransceivers()[0]->mid(), transceivers[1]->mid());
364 EXPECT_FALSE(transceivers[1]->stopped());
365}
366
367// Test that audio and video transceivers created on the remote side with
368// AddTrack will all be reused if there is the same number of audio/video tracks
369// in the remote offer. Additionally, this tests that transceivers are
370// successfully matched even if they are in a different order on the remote
371// side.
372TEST_F(PeerConnectionJsepTest, SetRemoteOfferReusesTransceiversOfBothTypes) {
373 auto caller = CreatePeerConnection();
374 caller->AddVideoTrack("v");
375 caller->AddAudioTrack("a");
376 auto callee = CreatePeerConnection();
377 callee->AddAudioTrack("a");
378 callee->AddVideoTrack("v");
379
380 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
381
382 auto caller_transceivers = caller->pc()->GetTransceivers();
383 auto callee_transceivers = callee->pc()->GetTransceivers();
384 ASSERT_EQ(2u, callee_transceivers.size());
385 EXPECT_EQ(caller_transceivers[0]->mid(), callee_transceivers[1]->mid());
386 EXPECT_EQ(caller_transceivers[1]->mid(), callee_transceivers[0]->mid());
387}
388
389// Tests for JSEP initial CreateAnswer.
390
391// Test that the answer to a remote offer creates media sections for each
392// offered media in the same order and with the same mids.
393TEST_F(PeerConnectionJsepTest, CreateAnswerHasSameMidsAsOffer) {
394 auto caller = CreatePeerConnection();
395 auto first_transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO);
396 auto second_transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
397 auto third_transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO);
Steve Antonfa2260d2017-12-28 16:38:23 -0800398 caller->CreateDataChannel("dc");
Steve Antondcc3c022017-12-22 16:02:54 -0800399 auto callee = CreatePeerConnection();
400
Steve Antonfa2260d2017-12-28 16:38:23 -0800401 auto offer = caller->CreateOffer();
402 const auto* offer_data = cricket::GetFirstDataContent(offer->description());
403 ASSERT_TRUE(
404 caller->SetLocalDescription(CloneSessionDescription(offer.get())));
405 ASSERT_TRUE(callee->SetRemoteDescription(std::move(offer)));
Steve Antondcc3c022017-12-22 16:02:54 -0800406
407 auto answer = callee->CreateAnswer();
408 auto contents = answer->description()->contents();
Steve Antonfa2260d2017-12-28 16:38:23 -0800409 ASSERT_EQ(4u, contents.size());
Steve Antondcc3c022017-12-22 16:02:54 -0800410 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, contents[0].media_description()->type());
Steve Antonfa2260d2017-12-28 16:38:23 -0800411 EXPECT_EQ(first_transceiver->mid(), contents[0].name);
Steve Antondcc3c022017-12-22 16:02:54 -0800412 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO, contents[1].media_description()->type());
Steve Antonfa2260d2017-12-28 16:38:23 -0800413 EXPECT_EQ(second_transceiver->mid(), contents[1].name);
Steve Antondcc3c022017-12-22 16:02:54 -0800414 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, contents[2].media_description()->type());
Steve Antonfa2260d2017-12-28 16:38:23 -0800415 EXPECT_EQ(third_transceiver->mid(), contents[2].name);
416 EXPECT_EQ(cricket::MEDIA_TYPE_DATA, contents[3].media_description()->type());
417 EXPECT_EQ(offer_data->name, contents[3].name);
Steve Antondcc3c022017-12-22 16:02:54 -0800418}
419
420// Test that an answering media section is marked as rejected if the underlying
421// transceiver has been stopped.
422TEST_F(PeerConnectionJsepTest, CreateAnswerRejectsStoppedTransceiver) {
423 auto caller = CreatePeerConnection();
424 caller->AddAudioTrack("a");
425 auto callee = CreatePeerConnection();
426
427 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
428
429 callee->pc()->GetTransceivers()[0]->Stop();
430
431 auto answer = callee->CreateAnswer();
432 auto contents = answer->description()->contents();
433 ASSERT_EQ(1u, contents.size());
434 EXPECT_TRUE(contents[0].rejected);
435}
436
437// Test that CreateAnswer will generate media sections which will only send or
438// receive if the offer indicates it can do the reciprocating direction.
439// The full matrix is tested more extensively in MediaSession.
440TEST_F(PeerConnectionJsepTest, CreateAnswerNegotiatesDirection) {
441 auto caller = CreatePeerConnection();
442 RtpTransceiverInit init;
443 init.direction = RtpTransceiverDirection::kSendOnly;
444 caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO, init);
445 auto callee = CreatePeerConnection();
446 callee->AddAudioTrack("a");
447
448 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
449
450 auto answer = callee->CreateAnswer();
451 auto contents = answer->description()->contents();
452 ASSERT_EQ(1u, contents.size());
453 EXPECT_EQ(RtpTransceiverDirection::kRecvOnly,
454 contents[0].media_description()->direction());
455}
456
457// Tests for JSEP SetLocalDescription with a local answer.
458// Note that these test only the additional behaviors not covered by
459// SetLocalDescription with a local offer.
460
461// Test that SetLocalDescription with an answer sets the current_direction
462// property of the transceivers mentioned in the session description.
463TEST_F(PeerConnectionJsepTest, SetLocalAnswerUpdatesCurrentDirection) {
464 auto caller = CreatePeerConnection();
465 auto caller_audio = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
466 caller_audio->SetDirection(RtpTransceiverDirection::kRecvOnly);
467 auto callee = CreatePeerConnection();
468 callee->AddAudioTrack("a");
469
470 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
471 ASSERT_TRUE(callee->SetLocalDescription(callee->CreateAnswer()));
472
473 auto transceivers = callee->pc()->GetTransceivers();
474 ASSERT_EQ(1u, transceivers.size());
475 // Since the offer was recvonly and the transceiver direction is sendrecv,
476 // the negotiated direction will be sendonly.
477 EXPECT_EQ(RtpTransceiverDirection::kSendOnly,
478 transceivers[0]->current_direction());
479}
480
481// Tests for JSEP SetRemoteDescription with a remote answer.
482// Note that these test only the additional behaviors not covered by
483// SetRemoteDescription with a remote offer.
484
485TEST_F(PeerConnectionJsepTest, SetRemoteAnswerUpdatesCurrentDirection) {
486 auto caller = CreatePeerConnection();
487 caller->AddAudioTrack("a");
488 auto callee = CreatePeerConnection();
489 callee->AddAudioTrack("a");
490 auto callee_audio = callee->pc()->GetTransceivers()[0];
491 callee_audio->SetDirection(RtpTransceiverDirection::kSendOnly);
492
493 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
494 ASSERT_TRUE(
495 caller->SetRemoteDescription(callee->CreateAnswerAndSetAsLocal()));
496
497 auto transceivers = caller->pc()->GetTransceivers();
498 ASSERT_EQ(1u, transceivers.size());
499 // Since the remote transceiver was set to sendonly, the negotiated direction
500 // in the answer would be sendonly which we apply as recvonly to the local
501 // transceiver.
502 EXPECT_EQ(RtpTransceiverDirection::kRecvOnly,
503 transceivers[0]->current_direction());
504}
505
506// Tests for multiple round trips.
507
508// Test that setting a transceiver with the inactive direction does not stop it
509// on either the caller or the callee.
510TEST_F(PeerConnectionJsepTest, SettingTransceiverInactiveDoesNotStopIt) {
511 auto caller = CreatePeerConnection();
512 caller->AddAudioTrack("a");
513 auto callee = CreatePeerConnection();
514 callee->AddAudioTrack("a");
515 callee->pc()->GetTransceivers()[0]->SetDirection(
516 RtpTransceiverDirection::kInactive);
517
518 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
519 ASSERT_TRUE(
520 caller->SetRemoteDescription(callee->CreateAnswerAndSetAsLocal()));
521
522 EXPECT_FALSE(caller->pc()->GetTransceivers()[0]->stopped());
523 EXPECT_FALSE(callee->pc()->GetTransceivers()[0]->stopped());
524}
525
526// Test that if a transceiver had been associated and later stopped, then a
527// media section is still generated for it and the media section is marked as
528// rejected.
529TEST_F(PeerConnectionJsepTest,
530 ReOfferMediaSectionForAssociatedStoppedTransceiverIsRejected) {
531 auto caller = CreatePeerConnection();
532 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
533 auto callee = CreatePeerConnection();
534
535 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
536 ASSERT_TRUE(
537 caller->SetRemoteDescription(callee->CreateAnswerAndSetAsLocal()));
538
539 ASSERT_TRUE(transceiver->mid());
540 transceiver->Stop();
541
542 auto reoffer = caller->CreateOffer();
543 auto contents = reoffer->description()->contents();
544 ASSERT_EQ(1u, contents.size());
545 EXPECT_TRUE(contents[0].rejected);
546}
547
548// Test that stopping an associated transceiver on the caller side will stop the
549// corresponding transceiver on the remote side when the remote offer is
550// applied.
551TEST_F(PeerConnectionJsepTest,
552 StoppingTransceiverInOfferStopsTransceiverOnRemoteSide) {
553 auto caller = CreatePeerConnection();
554 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
555 auto callee = CreatePeerConnection();
556
557 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
558 ASSERT_TRUE(
559 caller->SetRemoteDescription(callee->CreateAnswerAndSetAsLocal()));
560
561 transceiver->Stop();
562
563 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
564
565 auto transceivers = callee->pc()->GetTransceivers();
566 EXPECT_TRUE(transceivers[0]->stopped());
567 EXPECT_TRUE(transceivers[0]->mid());
568}
569
570// Test that CreateOffer will only generate a recycled media section if the
571// transceiver to be recycled has been seen stopped by the other side first.
572TEST_F(PeerConnectionJsepTest,
573 CreateOfferDoesNotRecycleMediaSectionIfFirstStopped) {
574 auto caller = CreatePeerConnection();
575 auto first_transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
576 auto callee = CreatePeerConnection();
577
578 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
579 ASSERT_TRUE(
580 caller->SetRemoteDescription(callee->CreateAnswerAndSetAsLocal()));
581
582 auto second_transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
583 first_transceiver->Stop();
584
585 auto reoffer = caller->CreateOffer();
586 auto contents = reoffer->description()->contents();
587 ASSERT_EQ(2u, contents.size());
588 EXPECT_TRUE(contents[0].rejected);
589 EXPECT_FALSE(contents[1].rejected);
590}
591
592// Test that the offer/answer and transceivers for both the caller and callee
593// side are generated/updated correctly when recycling an audio/video media
594// section as a media section of either the same or opposite type.
595class RecycleMediaSectionTest
596 : public PeerConnectionJsepTest,
597 public testing::WithParamInterface<
598 std::tuple<cricket::MediaType, cricket::MediaType>> {
599 protected:
600 RecycleMediaSectionTest() {
601 first_type_ = std::get<0>(GetParam());
602 second_type_ = std::get<1>(GetParam());
603 }
604
605 cricket::MediaType first_type_;
606 cricket::MediaType second_type_;
607};
608
609TEST_P(RecycleMediaSectionTest, VerifyOfferAnswerAndTransceivers) {
610 auto caller = CreatePeerConnection();
611 auto first_transceiver = caller->AddTransceiver(first_type_);
612 auto callee = CreatePeerConnection();
613
614 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
615
616 std::string first_mid = *first_transceiver->mid();
617 first_transceiver->Stop();
618
619 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
620
621 auto second_transceiver = caller->AddTransceiver(second_type_);
622
623 // The offer should reuse the previous media section but allocate a new MID
624 // and change the media type.
625 auto offer = caller->CreateOffer();
626 auto offer_contents = offer->description()->contents();
627 ASSERT_EQ(1u, offer_contents.size());
628 EXPECT_FALSE(offer_contents[0].rejected);
629 EXPECT_EQ(second_type_, offer_contents[0].media_description()->type());
630 std::string second_mid = offer_contents[0].name;
631 EXPECT_NE(first_mid, second_mid);
632
633 // Setting the local offer will dissociate the previous transceiver and set
634 // the MID for the new transceiver.
635 ASSERT_TRUE(
636 caller->SetLocalDescription(CloneSessionDescription(offer.get())));
637 EXPECT_EQ(rtc::nullopt, first_transceiver->mid());
638 EXPECT_EQ(second_mid, second_transceiver->mid());
639
640 // Setting the remote offer will dissociate the previous transceiver and
641 // create a new transceiver for the media section.
642 ASSERT_TRUE(callee->SetRemoteDescription(std::move(offer)));
643 auto callee_transceivers = callee->pc()->GetTransceivers();
644 ASSERT_EQ(2u, callee_transceivers.size());
645 EXPECT_EQ(rtc::nullopt, callee_transceivers[0]->mid());
Steve Anton69470252018-02-09 11:43:08 -0800646 EXPECT_EQ(first_type_, callee_transceivers[0]->media_type());
Steve Antondcc3c022017-12-22 16:02:54 -0800647 EXPECT_EQ(second_mid, callee_transceivers[1]->mid());
Steve Anton69470252018-02-09 11:43:08 -0800648 EXPECT_EQ(second_type_, callee_transceivers[1]->media_type());
Steve Antondcc3c022017-12-22 16:02:54 -0800649
650 // The answer should have only one media section for the new transceiver.
651 auto answer = callee->CreateAnswer();
652 auto answer_contents = answer->description()->contents();
653 ASSERT_EQ(1u, answer_contents.size());
654 EXPECT_FALSE(answer_contents[0].rejected);
655 EXPECT_EQ(second_mid, answer_contents[0].name);
656 EXPECT_EQ(second_type_, answer_contents[0].media_description()->type());
657
658 // Setting the local answer should succeed.
659 ASSERT_TRUE(
660 callee->SetLocalDescription(CloneSessionDescription(answer.get())));
661
662 // Setting the remote answer should succeed.
663 ASSERT_TRUE(caller->SetRemoteDescription(std::move(answer)));
664}
665
666// Test all combinations of audio and video as the first and second media type
667// for the media section. This is needed for full test coverage because
668// MediaSession has separate functions for processing audio and video media
669// sections.
670INSTANTIATE_TEST_CASE_P(
671 PeerConnectionJsepTest,
672 RecycleMediaSectionTest,
673 Combine(Values(cricket::MEDIA_TYPE_AUDIO, cricket::MEDIA_TYPE_VIDEO),
674 Values(cricket::MEDIA_TYPE_AUDIO, cricket::MEDIA_TYPE_VIDEO)));
675
Steve Antonfa2260d2017-12-28 16:38:23 -0800676// Test that a new data channel section will not reuse a recycleable audio or
677// video media section. Additionally, tests that the new section is added to the
678// end of the session description.
679TEST_F(PeerConnectionJsepTest, DataChannelDoesNotRecycleMediaSection) {
680 auto caller = CreatePeerConnection();
681 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
682 auto callee = CreatePeerConnection();
683
684 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
685
686 transceiver->Stop();
687
688 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
689
690 caller->CreateDataChannel("dc");
691
692 auto offer = caller->CreateOffer();
693 auto offer_contents = offer->description()->contents();
694 ASSERT_EQ(2u, offer_contents.size());
695 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO,
696 offer_contents[0].media_description()->type());
697 EXPECT_EQ(cricket::MEDIA_TYPE_DATA,
698 offer_contents[1].media_description()->type());
699
700 ASSERT_TRUE(
701 caller->SetLocalDescription(CloneSessionDescription(offer.get())));
702 ASSERT_TRUE(callee->SetRemoteDescription(std::move(offer)));
703
704 auto answer = callee->CreateAnswer();
705 auto answer_contents = answer->description()->contents();
706 ASSERT_EQ(2u, answer_contents.size());
707 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO,
708 answer_contents[0].media_description()->type());
709 EXPECT_EQ(cricket::MEDIA_TYPE_DATA,
710 answer_contents[1].media_description()->type());
711}
712
713// Test that if a new track is added to an existing session that has a data,
714// the new section comes at the end of the new offer, after the existing data
715// section.
716TEST_F(PeerConnectionJsepTest, AudioTrackAddedAfterDataSectionInReoffer) {
717 auto caller = CreatePeerConnection();
718 caller->CreateDataChannel("dc");
719 auto callee = CreatePeerConnection();
720
721 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
722
723 caller->AddAudioTrack("a");
724
725 auto offer = caller->CreateOffer();
726 auto contents = offer->description()->contents();
727 ASSERT_EQ(2u, contents.size());
728 EXPECT_EQ(cricket::MEDIA_TYPE_DATA, contents[0].media_description()->type());
729 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO, contents[1].media_description()->type());
730}
731
Steve Antondcc3c022017-12-22 16:02:54 -0800732// Tests for MID properties.
733
734static void RenameSection(size_t mline_index,
735 const std::string& new_mid,
736 SessionDescriptionInterface* sdesc) {
737 cricket::SessionDescription* desc = sdesc->description();
738 std::string old_mid = desc->contents()[mline_index].name;
739 desc->contents()[mline_index].name = new_mid;
740 desc->transport_infos()[mline_index].content_name = new_mid;
741 const cricket::ContentGroup* bundle =
742 desc->GetGroupByName(cricket::GROUP_TYPE_BUNDLE);
743 if (bundle) {
744 cricket::ContentGroup new_bundle = *bundle;
745 if (new_bundle.RemoveContentName(old_mid)) {
746 new_bundle.AddContentName(new_mid);
747 }
748 desc->RemoveGroupByName(cricket::GROUP_TYPE_BUNDLE);
749 desc->AddGroup(new_bundle);
750 }
751}
752
753// Test that two PeerConnections can have a successful offer/answer exchange if
754// the MIDs are changed from the defaults.
755TEST_F(PeerConnectionJsepTest, OfferAnswerWithChangedMids) {
756 constexpr char kFirstMid[] = "nondefaultmid";
757 constexpr char kSecondMid[] = "randommid";
758
759 auto caller = CreatePeerConnection();
760 caller->AddAudioTrack("a");
761 caller->AddAudioTrack("b");
762 auto callee = CreatePeerConnection();
763
764 auto offer = caller->CreateOffer();
765 RenameSection(0, kFirstMid, offer.get());
766 RenameSection(1, kSecondMid, offer.get());
767
768 ASSERT_TRUE(
769 caller->SetLocalDescription(CloneSessionDescription(offer.get())));
770 auto caller_transceivers = caller->pc()->GetTransceivers();
771 EXPECT_EQ(kFirstMid, caller_transceivers[0]->mid());
772 EXPECT_EQ(kSecondMid, caller_transceivers[1]->mid());
773
774 ASSERT_TRUE(callee->SetRemoteDescription(std::move(offer)));
775 auto callee_transceivers = callee->pc()->GetTransceivers();
776 EXPECT_EQ(kFirstMid, callee_transceivers[0]->mid());
777 EXPECT_EQ(kSecondMid, callee_transceivers[1]->mid());
778
779 auto answer = callee->CreateAnswer();
780 auto answer_contents = answer->description()->contents();
781 EXPECT_EQ(kFirstMid, answer_contents[0].name);
782 EXPECT_EQ(kSecondMid, answer_contents[1].name);
783
784 ASSERT_TRUE(
785 callee->SetLocalDescription(CloneSessionDescription(answer.get())));
786 ASSERT_TRUE(caller->SetRemoteDescription(std::move(answer)));
787}
788
789// Test that CreateOffer will generate a MID that is not already used if the
790// default it would have picked is already taken. This is tested by using a
791// third PeerConnection to determine what the default would be for the second
792// media section then setting that as the first media section's MID.
793TEST_F(PeerConnectionJsepTest, CreateOfferGeneratesUniqueMidIfAlreadyTaken) {
794 // First, find what the default MID is for the second media section.
795 auto pc = CreatePeerConnection();
796 pc->AddAudioTrack("a");
797 pc->AddAudioTrack("b");
798 auto default_offer = pc->CreateOffer();
799 std::string default_second_mid =
800 default_offer->description()->contents()[1].name;
801
802 // Now, do an offer/answer with one track which has the MID set to the default
803 // second MID.
804 auto caller = CreatePeerConnection();
805 caller->AddAudioTrack("a");
806 auto callee = CreatePeerConnection();
807
808 auto offer = caller->CreateOffer();
809 RenameSection(0, default_second_mid, offer.get());
810
811 ASSERT_TRUE(
812 caller->SetLocalDescription(CloneSessionDescription(offer.get())));
813 ASSERT_TRUE(callee->SetRemoteDescription(std::move(offer)));
814 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
815
816 // Add a second track and ensure that the MID is different.
817 caller->AddAudioTrack("b");
818
819 auto reoffer = caller->CreateOffer();
820 auto reoffer_contents = reoffer->description()->contents();
821 EXPECT_EQ(default_second_mid, reoffer_contents[0].name);
822 EXPECT_NE(reoffer_contents[0].name, reoffer_contents[1].name);
823}
824
Steve Antonfa2260d2017-12-28 16:38:23 -0800825// Test that if an audio or video section has the default data section MID, then
826// CreateOffer will generate a unique MID for the newly added data section.
827TEST_F(PeerConnectionJsepTest,
828 CreateOfferGeneratesUniqueMidForDataSectionIfAlreadyTaken) {
829 // First, find what the default MID is for the data channel.
830 auto pc = CreatePeerConnection();
831 pc->CreateDataChannel("dc");
832 auto default_offer = pc->CreateOffer();
833 std::string default_data_mid =
834 default_offer->description()->contents()[0].name;
835
836 // Now do an offer/answer with one audio track which has a MID set to the
837 // default data MID.
838 auto caller = CreatePeerConnection();
839 caller->AddAudioTrack("a");
840 auto callee = CreatePeerConnection();
841
842 auto offer = caller->CreateOffer();
843 RenameSection(0, default_data_mid, offer.get());
844
845 ASSERT_TRUE(
846 caller->SetLocalDescription(CloneSessionDescription(offer.get())));
847 ASSERT_TRUE(callee->SetRemoteDescription(std::move(offer)));
848 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
849
850 // Add a data channel and ensure that the MID is different.
851 caller->CreateDataChannel("dc");
852
853 auto reoffer = caller->CreateOffer();
854 auto reoffer_contents = reoffer->description()->contents();
855 EXPECT_EQ(default_data_mid, reoffer_contents[0].name);
856 EXPECT_NE(reoffer_contents[0].name, reoffer_contents[1].name);
857}
858
Steve Antondcc3c022017-12-22 16:02:54 -0800859// Test that a reoffer initiated by the callee adds a new track to the caller.
860TEST_F(PeerConnectionJsepTest, CalleeDoesReoffer) {
861 auto caller = CreatePeerConnection();
862 caller->AddAudioTrack("a");
863 auto callee = CreatePeerConnection();
864 callee->AddAudioTrack("a");
865 callee->AddVideoTrack("v");
866
867 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
868
869 EXPECT_EQ(1u, caller->pc()->GetTransceivers().size());
870 EXPECT_EQ(2u, callee->pc()->GetTransceivers().size());
871
872 ASSERT_TRUE(callee->ExchangeOfferAnswerWith(caller.get()));
873
874 EXPECT_EQ(2u, caller->pc()->GetTransceivers().size());
875 EXPECT_EQ(2u, callee->pc()->GetTransceivers().size());
876}
877
Steve Anton02ee47c2018-01-10 16:26:06 -0800878// Tests for MSID properties.
879
880// Test that adding a track with AddTrack results in an offer that signals the
881// track's ID.
882TEST_F(PeerConnectionJsepTest, AddingTrackWithAddTrackSpecifiesTrackId) {
883 const std::string kTrackId = "audio_track";
884
885 auto caller = CreatePeerConnection();
886 caller->AddAudioTrack(kTrackId);
887
888 auto offer = caller->CreateOffer();
889 auto contents = offer->description()->contents();
890 ASSERT_EQ(1u, contents.size());
891 auto streams = contents[0].media_description()->streams();
892 ASSERT_EQ(1u, streams.size());
893 EXPECT_EQ(kTrackId, streams[0].id);
894}
895
896// Test that adding a track by calling AddTransceiver then SetTrack results in
897// an offer that does not signal the track's ID and signals a random ID.
898TEST_F(PeerConnectionJsepTest,
899 AddingTrackWithAddTransceiverSpecifiesRandomTrackId) {
900 const std::string kTrackId = "audio_track";
901
902 auto caller = CreatePeerConnection();
903 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
904 transceiver->sender()->SetTrack(caller->CreateAudioTrack(kTrackId));
905
906 auto offer = caller->CreateOffer();
907 auto contents = offer->description()->contents();
908 ASSERT_EQ(1u, contents.size());
909 auto streams = contents[0].media_description()->streams();
910 ASSERT_EQ(1u, streams.size());
911 EXPECT_NE(kTrackId, streams[0].id);
912}
913
Steve Anton5f94aa22018-02-01 10:58:30 -0800914// Test that if the transceiver is recvonly or inactive, then no MSID
915// information is included in the offer.
916TEST_F(PeerConnectionJsepTest, NoMsidInOfferIfTransceiverDirectionHasNoSend) {
917 auto caller = CreatePeerConnection();
918
919 RtpTransceiverInit init_recvonly;
920 init_recvonly.direction = RtpTransceiverDirection::kRecvOnly;
921 ASSERT_TRUE(caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO, init_recvonly));
922
923 RtpTransceiverInit init_inactive;
924 init_inactive.direction = RtpTransceiverDirection::kInactive;
925 ASSERT_TRUE(caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO, init_inactive));
926
927 auto offer = caller->CreateOffer();
928 auto contents = offer->description()->contents();
929 ASSERT_EQ(2u, contents.size());
930 // MSID is specified in the first stream, so no streams means no MSID.
931 EXPECT_EQ(0u, contents[0].media_description()->streams().size());
932 EXPECT_EQ(0u, contents[1].media_description()->streams().size());
933}
934
935// Test that if an answer negotiates transceiver directions of recvonly or
936// inactive, then no MSID information is included in the answer.
937TEST_F(PeerConnectionJsepTest, NoMsidInAnswerIfNoRespondingTracks) {
938 auto caller = CreatePeerConnection();
939 auto callee = CreatePeerConnection();
940
941 // recvonly transceiver will get negotiated to inactive since the callee has
942 // no tracks to send in response.
943 RtpTransceiverInit init_recvonly;
944 init_recvonly.direction = RtpTransceiverDirection::kRecvOnly;
945 caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO, init_recvonly);
946
947 // sendrecv transceiver will get negotiated to recvonly since the callee has
948 // no tracks to send in response.
949 RtpTransceiverInit init_sendrecv;
950 init_sendrecv.direction = RtpTransceiverDirection::kSendRecv;
951 caller->AddTransceiver(cricket::MEDIA_TYPE_VIDEO, init_sendrecv);
952
953 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
954
955 auto answer = callee->CreateAnswer();
956 auto contents = answer->description()->contents();
957 ASSERT_EQ(2u, contents.size());
958 // MSID is specified in the first stream, so no streams means no MSID.
959 EXPECT_EQ(0u, contents[0].media_description()->streams().size());
960 EXPECT_EQ(0u, contents[1].media_description()->streams().size());
961}
962
963// Test that the MSID is included even if the transceiver direction has changed
964// to inactive if the transceiver had previously sent media.
965TEST_F(PeerConnectionJsepTest, IncludeMsidEvenIfDirectionHasChanged) {
966 auto caller = CreatePeerConnection();
967 caller->AddAudioTrack("audio");
968 auto callee = CreatePeerConnection();
969 callee->AddAudioTrack("audio");
970
971 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
972
973 caller->pc()->GetTransceivers()[0]->SetDirection(
974 RtpTransceiverDirection::kInactive);
975
976 // The transceiver direction on both sides will turn to inactive.
977 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
978
979 auto* offer = callee->pc()->remote_description();
980 auto offer_contents = offer->description()->contents();
981 ASSERT_EQ(1u, offer_contents.size());
982 // MSID is specified in the first stream. If it is present, assume that MSID
983 // is there.
984 EXPECT_EQ(1u, offer_contents[0].media_description()->streams().size());
985
986 auto* answer = caller->pc()->remote_description();
987 auto answer_contents = answer->description()->contents();
988 ASSERT_EQ(1u, answer_contents.size());
989 EXPECT_EQ(1u, answer_contents[0].media_description()->streams().size());
990}
991
992// Test that stopping a RtpTransceiver will cause future offers to not include
993// any MSID information for that section.
994TEST_F(PeerConnectionJsepTest, RemoveMsidIfTransceiverStopped) {
995 auto caller = CreatePeerConnection();
996 auto transceiver = caller->AddTransceiver(cricket::MEDIA_TYPE_AUDIO);
997 auto callee = CreatePeerConnection();
998
999 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1000
1001 transceiver->Stop();
1002
1003 ASSERT_TRUE(caller->ExchangeOfferAnswerWith(callee.get()));
1004
1005 auto* offer = callee->pc()->remote_description();
1006 auto offer_contents = offer->description()->contents();
1007 ASSERT_EQ(1u, offer_contents.size());
1008 // MSID is specified in the first stream, so no streams means no MSID.
1009 EXPECT_EQ(0u, offer_contents[0].media_description()->streams().size());
1010}
1011
Steve Anton02ee47c2018-01-10 16:26:06 -08001012// Test that the callee RtpReceiver created by a call to SetRemoteDescription
1013// has its ID set to the signaled track ID.
1014TEST_F(PeerConnectionJsepTest,
1015 RtpReceiverCreatedBySetRemoteDescriptionHasSignaledTrackId) {
1016 const std::string kTrackId = "audio_track";
1017
1018 auto caller = CreatePeerConnection();
1019 auto callee = CreatePeerConnection();
1020 caller->AddAudioTrack(kTrackId);
1021
1022 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
1023
1024 ASSERT_EQ(1u, callee->pc()->GetReceivers().size());
1025 auto receiver = callee->pc()->GetReceivers()[0];
1026 EXPECT_EQ(kTrackId, receiver->id());
1027}
1028
1029// Test that if the callee RtpReceiver is reused by a call to
1030// SetRemoteDescription, its ID does not change.
1031TEST_F(PeerConnectionJsepTest,
1032 RtpReceiverCreatedBeforeSetRemoteDescriptionKeepsId) {
1033 const std::string kTrackId = "audio_track";
1034
1035 auto caller = CreatePeerConnection();
1036 auto callee = CreatePeerConnection();
1037 caller->AddAudioTrack(kTrackId);
1038 callee->AddAudioTrack("dummy_track");
1039
1040 ASSERT_EQ(1u, callee->pc()->GetReceivers().size());
1041 auto receiver = callee->pc()->GetReceivers()[0];
1042 std::string receiver_id = receiver->id();
1043
1044 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
1045
1046 EXPECT_EQ(receiver_id, receiver->id());
1047}
1048
Steve Antonef65ef12018-01-10 17:15:20 -08001049// Test that setting a remote offer with one track that has no streams fires off
1050// the correct OnAddTrack event.
1051TEST_F(PeerConnectionJsepTest,
1052 SetRemoteOfferWithOneTrackNoStreamFiresOnAddTrack) {
1053 const std::string kTrackLabel = "audio_track";
1054
1055 auto caller = CreatePeerConnection();
1056 auto callee = CreatePeerConnection();
1057 ASSERT_TRUE(caller->AddAudioTrack(kTrackLabel));
1058
1059 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
1060
1061 ASSERT_EQ(callee->observer()->add_track_events_.size(), 1u);
1062 EXPECT_EQ(kTrackLabel,
1063 callee->observer()->add_track_events_[0].receiver->track()->id());
1064 // TODO(bugs.webrtc.org/7933): Also verify that no stream was added to the
1065 // receiver.
1066}
1067
1068// Test that setting a remote offer with one track that has one stream fires off
1069// the correct OnAddTrack event.
1070TEST_F(PeerConnectionJsepTest,
1071 SetRemoteOfferWithOneTrackOneStreamFiresOnAddTrack) {
1072 const std::string kTrackLabel = "audio_track";
1073 const std::string kStreamLabel = "audio_stream";
1074
1075 auto caller = CreatePeerConnection();
1076 auto callee = CreatePeerConnection();
1077 ASSERT_TRUE(caller->AddAudioTrack(kTrackLabel, {kStreamLabel}));
1078
1079 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
1080
1081 const auto& track_events = callee->observer()->add_track_events_;
1082 ASSERT_EQ(1u, track_events.size());
1083 const auto& event = track_events[0];
1084 ASSERT_EQ(1u, event.streams.size());
1085 auto stream = event.streams[0];
1086 EXPECT_EQ(kStreamLabel, stream->label());
1087 EXPECT_THAT(track_events[0].snapshotted_stream_tracks.at(stream),
1088 ElementsAre(event.receiver->track()));
1089 EXPECT_EQ(event.receiver->streams(), track_events[0].streams);
1090}
1091
1092// Test that setting a remote offer with two tracks that share the same stream
1093// fires off two OnAddTrack events, both with the same stream that has both
1094// tracks present at the time of firing. This is to ensure that track events are
1095// not fired until SetRemoteDescription has finished processing all the media
1096// sections.
1097TEST_F(PeerConnectionJsepTest,
1098 SetRemoteOfferWithTwoTracksSameStreamFiresOnAddTrack) {
1099 const std::string kTrack1Label = "audio_track1";
1100 const std::string kTrack2Label = "audio_track2";
1101 const std::string kSharedStreamLabel = "stream";
1102
1103 auto caller = CreatePeerConnection();
1104 auto callee = CreatePeerConnection();
1105 ASSERT_TRUE(caller->AddAudioTrack(kTrack1Label, {kSharedStreamLabel}));
1106 ASSERT_TRUE(caller->AddAudioTrack(kTrack2Label, {kSharedStreamLabel}));
1107
1108 ASSERT_TRUE(callee->SetRemoteDescription(caller->CreateOfferAndSetAsLocal()));
1109
1110 const auto& track_events = callee->observer()->add_track_events_;
1111 ASSERT_EQ(2u, track_events.size());
1112 const auto& event1 = track_events[0];
1113 const auto& event2 = track_events[1];
1114 ASSERT_EQ(1u, event1.streams.size());
1115 auto stream = event1.streams[0];
1116 ASSERT_THAT(event2.streams, ElementsAre(stream));
1117 auto track1 = event1.receiver->track();
1118 auto track2 = event2.receiver->track();
1119 EXPECT_THAT(event1.snapshotted_stream_tracks.at(stream),
1120 UnorderedElementsAre(track1, track2));
1121 EXPECT_THAT(event2.snapshotted_stream_tracks.at(stream),
1122 UnorderedElementsAre(track1, track2));
1123}
1124
1125// TODO(bugs.webrtc.org/7932): Also test multi-stream case.
1126
Steve Antondcc3c022017-12-22 16:02:54 -08001127} // namespace webrtc