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

Unified Diff: talk/app/webrtc/webrtcsession_unittest.cc

Issue 1312643004: Replaces SSLIdentity* with scoped_refptr<RTCCertificate> in cricket::Transport layer. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 5 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 side-by-side diff with in-line comments
Download patch
Index: talk/app/webrtc/webrtcsession_unittest.cc
diff --git a/talk/app/webrtc/webrtcsession_unittest.cc b/talk/app/webrtc/webrtcsession_unittest.cc
index a23130db50e0311c5b334cf9e83de92ca1f71f09..ae12e6fa7b9e51d1ebbabd9f3864ff4344e07ed5 100644
--- a/talk/app/webrtc/webrtcsession_unittest.cc
+++ b/talk/app/webrtc/webrtcsession_unittest.cc
@@ -3575,20 +3575,17 @@ TEST_P(WebRtcSessionTest, TestSctpDataChannelSendPortParsing) {
EXPECT_EQ(new_recv_port, portnum);
}
-// TODO(hbos): Add the following test once RTCCertificate is passed around
-// outside of WebRtcSessionDescriptionFactory code and there exists a
-// WebRtcSession::certificate().
-//TEST_F(WebRtcSessionTest, TestUsesProvidedCertificate) {
-// rtc::scoped_refptr<rtc::RTCCertificate> certificate =
-// FakeDtlsIdentityStore::GenerateCertificate();
-//
-// PeerConnectionInterface::RTCConfiguration configuration;
-// configuration.certificates.push_back(certificate);
-// Init(nullptr, configuration);
-// EXPECT_TRUE_WAIT(!session_->waiting_for_identity_for_testing(), 1000);
-//
-// EXPECT_EQ(session_->certificate(), certificate);
-//}
+TEST_F(WebRtcSessionTest, TestUsesProvidedCertificate) {
+ rtc::scoped_refptr<rtc::RTCCertificate> certificate =
+ FakeDtlsIdentityStore::GenerateCertificate();
+
+ PeerConnectionInterface::RTCConfiguration configuration;
+ configuration.certificates.push_back(certificate);
+ Init(nullptr, configuration);
+ EXPECT_TRUE_WAIT(!session_->waiting_for_certificate_for_testing(), 1000);
+
+ EXPECT_EQ(session_->certificate(), certificate);
+}
// Verifies that CreateOffer succeeds when CreateOffer is called before async
// identity generation is finished (even if a certificate is provided this is
@@ -3597,7 +3594,7 @@ TEST_P(WebRtcSessionTest, TestCreateOfferBeforeIdentityRequestReturnSuccess) {
MAYBE_SKIP_TEST(rtc::SSLStreamAdapter::HaveDtlsSrtp);
InitWithDtls(GetParam());
- EXPECT_TRUE(session_->waiting_for_identity_for_testing());
+ EXPECT_TRUE(session_->waiting_for_certificate_for_testing());
mediastream_signaling_.SendAudioVideoStream1();
rtc::scoped_ptr<SessionDescriptionInterface> offer(CreateOffer());
@@ -3634,7 +3631,7 @@ TEST_P(WebRtcSessionTest, TestCreateOfferAfterIdentityRequestReturnSuccess) {
MAYBE_SKIP_TEST(rtc::SSLStreamAdapter::HaveDtlsSrtp);
InitWithDtls(GetParam());
- EXPECT_TRUE_WAIT(!session_->waiting_for_identity_for_testing(), 1000);
+ EXPECT_TRUE_WAIT(!session_->waiting_for_certificate_for_testing(), 1000);
rtc::scoped_ptr<SessionDescriptionInterface> offer(CreateOffer());
EXPECT_TRUE(offer != NULL);
@@ -3646,7 +3643,7 @@ TEST_F(WebRtcSessionTest, TestCreateOfferAfterIdentityRequestReturnFailure) {
MAYBE_SKIP_TEST(rtc::SSLStreamAdapter::HaveDtlsSrtp);
InitWithDtlsIdentityGenFail();
- EXPECT_TRUE_WAIT(!session_->waiting_for_identity_for_testing(), 1000);
+ EXPECT_TRUE_WAIT(!session_->waiting_for_certificate_for_testing(), 1000);
rtc::scoped_ptr<SessionDescriptionInterface> offer(CreateOffer());
EXPECT_TRUE(offer == NULL);

Powered by Google App Engine
This is Rietveld 408576698