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

Unified Diff: webrtc/base/rtccertificate_unittests.cc

Issue 1506883005: Revert of RTCCertificate::Expires() and ::HasExpired() implemented (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 5 years 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
« no previous file with comments | « webrtc/base/rtccertificate.cc ('k') | webrtc/base/sslidentity.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/base/rtccertificate_unittests.cc
diff --git a/webrtc/base/rtccertificate_unittests.cc b/webrtc/base/rtccertificate_unittests.cc
deleted file mode 100644
index 3e9439fedb9e2325b6c64bcaa1cb79fd61754823..0000000000000000000000000000000000000000
--- a/webrtc/base/rtccertificate_unittests.cc
+++ /dev/null
@@ -1,116 +0,0 @@
-/*
- * Copyright 2015 The WebRTC Project Authors. All rights reserved.
- *
- * Use of this source code is governed by a BSD-style license
- * that can be found in the LICENSE file in the root of the source
- * tree. An additional intellectual property rights grant can be found
- * in the file PATENTS. All contributing project authors may
- * be found in the AUTHORS file in the root of the source tree.
- */
-
-#include "webrtc/base/checks.h"
-#include "webrtc/base/fakesslidentity.h"
-#include "webrtc/base/gunit.h"
-#include "webrtc/base/logging.h"
-#include "webrtc/base/rtccertificate.h"
-#include "webrtc/base/safe_conversions.h"
-#include "webrtc/base/scoped_ptr.h"
-#include "webrtc/base/sslidentity.h"
-#include "webrtc/base/thread.h"
-#include "webrtc/base/timeutils.h"
-
-namespace rtc {
-
-namespace {
-
-static const char* kTestCertCommonName = "RTCCertificateTest's certificate";
-
-} // namespace
-
-class RTCCertificateTest : public testing::Test {
- public:
- RTCCertificateTest() {}
- ~RTCCertificateTest() {}
-
- protected:
- // Timestamp note:
- // All timestamps in this unittest are expressed in number of seconds since
- // epoch, 1970-01-01T00:00:00Z (UTC). The RTCCertificate interface uses ms,
- // but only seconds-precision is supported by SSLCertificate. To make the
- // tests clearer we convert everything to seconds since the precision matters
- // when generating certificates or comparing timestamps.
- // As a result, ExpiresSeconds and HasExpiredSeconds are used instead of
- // RTCCertificate::Expires and ::HasExpired for ms -> s conversion.
-
- uint64_t NowSeconds() const {
- return TimeNanos() / kNumNanosecsPerSec;
- }
-
- uint64_t ExpiresSeconds(const scoped_refptr<RTCCertificate>& cert) const {
- uint64_t exp_ms = cert->Expires();
- uint64_t exp_s = exp_ms / kNumMillisecsPerSec;
- // Make sure this did not result in loss of precision.
- RTC_CHECK_EQ(exp_s * kNumMillisecsPerSec, exp_ms);
- return exp_s;
- }
-
- bool HasExpiredSeconds(const scoped_refptr<RTCCertificate>& cert,
- uint64_t now_s) const {
- return cert->HasExpired(now_s * kNumMillisecsPerSec);
- }
-
- // An RTC_CHECK ensures that |expires_s| this is in valid range of time_t as
- // is required by SSLIdentityParams. On some 32-bit systems time_t is limited
- // to < 2^31. On such systems this will fail for expiration times of year 2038
- // or later.
- scoped_refptr<RTCCertificate> GenerateCertificateWithExpires(
- uint64_t expires_s) const {
- RTC_CHECK(IsValueInRangeForNumericType<time_t>(expires_s));
-
- SSLIdentityParams params;
- params.common_name = kTestCertCommonName;
- params.not_before = 0;
- params.not_after = static_cast<time_t>(expires_s);
- // Certificate type does not matter for our purposes, using ECDSA because it
- // is fast to generate.
- params.key_params = KeyParams::ECDSA();
-
- scoped_ptr<SSLIdentity> identity(SSLIdentity::GenerateForTest(params));
- return RTCCertificate::Create(identity.Pass());
- }
-};
-
-TEST_F(RTCCertificateTest, NewCertificateNotExpired) {
- // Generate a real certificate without specifying the expiration time.
- // Certificate type doesn't matter, using ECDSA because it's fast to generate.
- scoped_ptr<SSLIdentity> identity(
- SSLIdentity::Generate(kTestCertCommonName, KeyParams::ECDSA()));
- scoped_refptr<RTCCertificate> certificate =
- RTCCertificate::Create(identity.Pass());
-
- uint64_t now = NowSeconds();
- EXPECT_FALSE(HasExpiredSeconds(certificate, now));
- // Even without specifying the expiration time we would expect it to be valid
- // for at least half an hour.
- EXPECT_FALSE(HasExpiredSeconds(certificate, now + 30*60));
-}
-
-TEST_F(RTCCertificateTest, UsesExpiresAskedFor) {
- uint64_t now = NowSeconds();
- scoped_refptr<RTCCertificate> certificate =
- GenerateCertificateWithExpires(now);
- EXPECT_EQ(now, ExpiresSeconds(certificate));
-}
-
-TEST_F(RTCCertificateTest, ExpiresInOneSecond) {
- // Generate a certificate that expires in 1s.
- uint64_t now = NowSeconds();
- scoped_refptr<RTCCertificate> certificate =
- GenerateCertificateWithExpires(now + 1);
- // Now it should not have expired.
- EXPECT_FALSE(HasExpiredSeconds(certificate, now));
- // In 2s it should have expired.
- EXPECT_TRUE(HasExpiredSeconds(certificate, now + 2));
-}
-
-} // namespace rtc
« no previous file with comments | « webrtc/base/rtccertificate.cc ('k') | webrtc/base/sslidentity.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698