WKWebView: Implemented recoverable SSL interstitials.

ios/web/net/cert_host_pair_unittest.cc

Index: ios/web/net/cert_host_pair_unittest.cc
diff --git a/ios/web/net/cert_host_pair_unittest.cc b/ios/web/net/cert_host_pair_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..b22ef77142a0913b257c0d24fb3b01452f43cb77
--- /dev/null
+++ b/ios/web/net/cert_host_pair_unittest.cc
@@ -0,0 +1,84 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "ios/web/net/cert_host_pair.h"
+
+#include "net/base/test_data_directory.h"
+#include "net/test/cert_test_util.h"
+#include "testing/platform_test.h"
+
+namespace web {
+
+namespace {
+
+// Test cert filenames.
+const char kCertFileName1[] = "ok_cert.pem";
+const char kCertFileName2[] = "expired_cert.pem";
+
+// Test hostnames.
+const char kHostName1[] = "www.example.com";
+const char kHostName2[] = "www.chromium.test";
+
+// Loads cert with the given |file_name|.
+scoped_refptr<net::X509Certificate> GetCert(const std::string& file_name) {
+  return net::ImportCertFromFile(net::GetTestCertsDirectory(), file_name);
+}
+
+}  // namespace
+
+// Test fixture to test CertHostPair struct.
+typedef PlatformTest CertHostPairTest;
+
+// Tests constructions.
+TEST_F(CertHostPairTest, Construction) {
+  scoped_refptr<net::X509Certificate> cert = GetCert(kCertFileName1);
+  ASSERT_TRUE(cert);
+  CertHostPair pair(cert, kHostName1);
+  EXPECT_EQ(cert, pair.cert);
+  EXPECT_EQ(std::string(kHostName1), pair.host);
+}
+
+// Tests comparision with different certs and hosts.
+TEST_F(CertHostPairTest, ComparisonWithDifferentCertsAndHosts) {
+  ASSERT_TRUE(GetCert(kCertFileName1));
+  ASSERT_TRUE(GetCert(kCertFileName2));

[Ryan Sleevi, 2015/10/29 22:37:27]

I was more trying to suggest you should follow line 35 and assign to a local
variable - and assert that local variable is fine.

This pattern is dangerous because there is zero relationship with the subsequent
lines - that is, if you changed line 47 to kCertFilename3, it would have no
effect on these.

Assigning to a local variable, asserting that variable is valid, and then
pushing it to the object gives you the desired guarantee.

[Eugene But (OOO till 7-30), 2015/10/30 01:40:55]

Done.

+  CertHostPair pair1(GetCert(kCertFileName1), kHostName1);
+  CertHostPair pair2(GetCert(kCertFileName2), kHostName2);
+
+  EXPECT_TRUE(pair2 < pair1);
+  EXPECT_FALSE(pair1 < pair2);
+}
+
+// Tests comparision with same cert.
+TEST_F(CertHostPairTest, ComparisonWithSameCert) {
+  ASSERT_TRUE(GetCert(kCertFileName1));
+  CertHostPair pair1(GetCert(kCertFileName1), kHostName1);
+  CertHostPair pair2(GetCert(kCertFileName1), kHostName2);
+
+  EXPECT_TRUE(pair2 < pair1);
+  EXPECT_FALSE(pair1 < pair2);
+}
+
+// Tests comparision with same host.
+TEST_F(CertHostPairTest, ComparisonWithSameHost) {
+  ASSERT_TRUE(GetCert(kCertFileName1));
+  ASSERT_TRUE(GetCert(kCertFileName2));
+  CertHostPair pair1(GetCert(kCertFileName1), kHostName1);
+  CertHostPair pair2(GetCert(kCertFileName2), kHostName1);
+
+  EXPECT_TRUE(pair1 < pair2);
+  EXPECT_FALSE(pair2 < pair1);
+}
+
+// Tests comparision with same cert and host.
+TEST_F(CertHostPairTest, ComparisonWithSameCertAndHost) {
+  ASSERT_TRUE(GetCert(kCertFileName1));
+  CertHostPair pair1(GetCert(kCertFileName1), kHostName1);
+  CertHostPair pair2(GetCert(kCertFileName1), kHostName1);
+
+  EXPECT_FALSE(pair1 < pair2);
+  EXPECT_FALSE(pair2 < pair1);
+}
+
+}  // namespace web