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

Side by Side Diff: webrtc/ortc/ortcrtpreceiver_unittest.cc

Issue 2981513002: Wire up RTP keep-alive in ortc api. (Closed)
Patch Set: deps, again Created 3 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « webrtc/ortc/ortcfactory_unittest.cc ('k') | webrtc/ortc/ortcrtpsender_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2017 The WebRTC project authors. All Rights Reserved. 2 * Copyright 2017 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 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 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 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 18 matching lines...) Expand all
29 class OrtcRtpReceiverTest : public testing::Test { 29 class OrtcRtpReceiverTest : public testing::Test {
30 public: 30 public:
31 OrtcRtpReceiverTest() : fake_packet_transport_("fake") { 31 OrtcRtpReceiverTest() : fake_packet_transport_("fake") {
32 fake_media_engine_ = new cricket::FakeMediaEngine(); 32 fake_media_engine_ = new cricket::FakeMediaEngine();
33 // Note: This doesn't need to use fake network classes, since we already 33 // Note: This doesn't need to use fake network classes, since we already
34 // use FakePacketTransport. 34 // use FakePacketTransport.
35 auto ortc_factory_result = OrtcFactory::Create( 35 auto ortc_factory_result = OrtcFactory::Create(
36 nullptr, nullptr, nullptr, nullptr, nullptr, 36 nullptr, nullptr, nullptr, nullptr, nullptr,
37 std::unique_ptr<cricket::MediaEngineInterface>(fake_media_engine_)); 37 std::unique_ptr<cricket::MediaEngineInterface>(fake_media_engine_));
38 ortc_factory_ = ortc_factory_result.MoveValue(); 38 ortc_factory_ = ortc_factory_result.MoveValue();
39 RtcpParameters rtcp_parameters; 39 RtpTransportParameters parameters;
40 rtcp_parameters.mux = true; 40 parameters.rtcp.mux = true;
41 auto rtp_transport_result = ortc_factory_->CreateRtpTransport( 41 auto rtp_transport_result = ortc_factory_->CreateRtpTransport(
42 rtcp_parameters, &fake_packet_transport_, nullptr, nullptr); 42 parameters, &fake_packet_transport_, nullptr, nullptr);
43 rtp_transport_ = rtp_transport_result.MoveValue(); 43 rtp_transport_ = rtp_transport_result.MoveValue();
44 } 44 }
45 45
46 protected: 46 protected:
47 // Owned by |ortc_factory_|. 47 // Owned by |ortc_factory_|.
48 cricket::FakeMediaEngine* fake_media_engine_; 48 cricket::FakeMediaEngine* fake_media_engine_;
49 rtc::FakePacketTransport fake_packet_transport_; 49 rtc::FakePacketTransport fake_packet_transport_;
50 std::unique_ptr<OrtcFactoryInterface> ortc_factory_; 50 std::unique_ptr<OrtcFactoryInterface> ortc_factory_;
51 std::unique_ptr<RtpTransportInterface> rtp_transport_; 51 std::unique_ptr<RtpTransportInterface> rtp_transport_;
52 }; 52 };
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
90 RtpParameters no_encodings = MakeMinimalVp9ParametersWithSsrc(0xdeadbeef); 90 RtpParameters no_encodings = MakeMinimalVp9ParametersWithSsrc(0xdeadbeef);
91 no_encodings.encodings.clear(); 91 no_encodings.encodings.clear();
92 EXPECT_TRUE(receiver->Receive(no_encodings).ok()); 92 EXPECT_TRUE(receiver->Receive(no_encodings).ok());
93 EXPECT_EQ(nullptr, receiver->GetTrack()); 93 EXPECT_EQ(nullptr, receiver->GetTrack());
94 } 94 }
95 95
96 // Currently SetTransport isn't supported. When it is, replace this test with a 96 // Currently SetTransport isn't supported. When it is, replace this test with a
97 // test/tests for it. 97 // test/tests for it.
98 TEST_F(OrtcRtpReceiverTest, SetTransportFails) { 98 TEST_F(OrtcRtpReceiverTest, SetTransportFails) {
99 rtc::FakePacketTransport fake_packet_transport("another_transport"); 99 rtc::FakePacketTransport fake_packet_transport("another_transport");
100 RtcpParameters rtcp_parameters; 100 RtpTransportParameters parameters;
101 rtcp_parameters.mux = true; 101 parameters.rtcp.mux = true;
102 auto rtp_transport_result = ortc_factory_->CreateRtpTransport( 102 auto rtp_transport_result = ortc_factory_->CreateRtpTransport(
103 rtcp_parameters, &fake_packet_transport, nullptr, nullptr); 103 parameters, &fake_packet_transport, nullptr, nullptr);
104 auto rtp_transport = rtp_transport_result.MoveValue(); 104 auto rtp_transport = rtp_transport_result.MoveValue();
105 105
106 auto receiver_result = ortc_factory_->CreateRtpReceiver( 106 auto receiver_result = ortc_factory_->CreateRtpReceiver(
107 cricket::MEDIA_TYPE_AUDIO, rtp_transport_.get()); 107 cricket::MEDIA_TYPE_AUDIO, rtp_transport_.get());
108 auto receiver = receiver_result.MoveValue(); 108 auto receiver = receiver_result.MoveValue();
109 EXPECT_EQ(RTCErrorType::UNSUPPORTED_OPERATION, 109 EXPECT_EQ(RTCErrorType::UNSUPPORTED_OPERATION,
110 receiver->SetTransport(rtp_transport.get()).type()); 110 receiver->SetTransport(rtp_transport.get()).type());
111 } 111 }
112 112
113 TEST_F(OrtcRtpReceiverTest, GetTransport) { 113 TEST_F(OrtcRtpReceiverTest, GetTransport) {
(...skipping 424 matching lines...) Expand 10 before | Expand all | Expand 10 after
538 cricket::MEDIA_TYPE_AUDIO, rtp_transport_.get()); 538 cricket::MEDIA_TYPE_AUDIO, rtp_transport_.get());
539 auto video_receiver_result = ortc_factory_->CreateRtpReceiver( 539 auto video_receiver_result = ortc_factory_->CreateRtpReceiver(
540 cricket::MEDIA_TYPE_VIDEO, rtp_transport_.get()); 540 cricket::MEDIA_TYPE_VIDEO, rtp_transport_.get());
541 auto audio_receiver = audio_receiver_result.MoveValue(); 541 auto audio_receiver = audio_receiver_result.MoveValue();
542 auto video_receiver = video_receiver_result.MoveValue(); 542 auto video_receiver = video_receiver_result.MoveValue();
543 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO, audio_receiver->GetKind()); 543 EXPECT_EQ(cricket::MEDIA_TYPE_AUDIO, audio_receiver->GetKind());
544 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, video_receiver->GetKind()); 544 EXPECT_EQ(cricket::MEDIA_TYPE_VIDEO, video_receiver->GetKind());
545 } 545 }
546 546
547 } // namespace webrtc 547 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/ortc/ortcfactory_unittest.cc ('k') | webrtc/ortc/ortcrtpsender_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698