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

Side by Side Diff: net/base/ssl_client_auth_cache_unittest.cc

Issue 179028: Revert "Fix a ton of compiler warnings." (Closed)
Patch Set: Created 11 years, 3 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 | « net/base/host_cache_unittest.cc ('k') | net/ftp/ftp_auth_cache_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 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/base/ssl_client_auth_cache.h" 5 #include "net/base/ssl_client_auth_cache.h"
6 6
7 #include "base/time.h" 7 #include "base/time.h"
8 #include "testing/gtest/include/gtest/gtest.h" 8 #include "testing/gtest/include/gtest/gtest.h"
9 9
10 namespace net { 10 namespace net {
(...skipping 10 matching lines...) Expand all
21 21
22 std::string server2("foo2:443"); 22 std::string server2("foo2:443");
23 scoped_refptr<X509Certificate> cert2( 23 scoped_refptr<X509Certificate> cert2(
24 new X509Certificate("foo2", "CA", start_date, expiration_date)); 24 new X509Certificate("foo2", "CA", start_date, expiration_date));
25 25
26 std::string server3("foo3:443"); 26 std::string server3("foo3:443");
27 scoped_refptr<X509Certificate> cert3( 27 scoped_refptr<X509Certificate> cert3(
28 new X509Certificate("foo3", "CA", start_date, expiration_date)); 28 new X509Certificate("foo3", "CA", start_date, expiration_date));
29 29
30 // Lookup non-existent client certificate. 30 // Lookup non-existent client certificate.
31 EXPECT_TRUE(NULL == cache.Lookup(server1)); 31 EXPECT_EQ(NULL, cache.Lookup(server1));
32 32
33 // Add client certificate for server1. 33 // Add client certificate for server1.
34 cache.Add(server1, cert1.get()); 34 cache.Add(server1, cert1.get());
35 EXPECT_EQ(cert1.get(), cache.Lookup(server1)); 35 EXPECT_EQ(cert1.get(), cache.Lookup(server1));
36 36
37 // Add client certificate for server2. 37 // Add client certificate for server2.
38 cache.Add(server2, cert2.get()); 38 cache.Add(server2, cert2.get());
39 EXPECT_EQ(cert1.get(), cache.Lookup(server1)); 39 EXPECT_EQ(cert1.get(), cache.Lookup(server1));
40 EXPECT_EQ(cert2.get(), cache.Lookup(server2)); 40 EXPECT_EQ(cert2.get(), cache.Lookup(server2));
41 41
42 // Overwrite the client certificate for server1. 42 // Overwrite the client certificate for server1.
43 cache.Add(server1, cert3.get()); 43 cache.Add(server1, cert3.get());
44 EXPECT_EQ(cert3.get(), cache.Lookup(server1)); 44 EXPECT_EQ(cert3.get(), cache.Lookup(server1));
45 EXPECT_EQ(cert2.get(), cache.Lookup(server2)); 45 EXPECT_EQ(cert2.get(), cache.Lookup(server2));
46 46
47 // Remove client certificate of server1. 47 // Remove client certificate of server1.
48 cache.Remove(server1); 48 cache.Remove(server1);
49 EXPECT_TRUE(NULL == cache.Lookup(server1)); 49 EXPECT_EQ(NULL, cache.Lookup(server1));
50 EXPECT_EQ(cert2.get(), cache.Lookup(server2)); 50 EXPECT_EQ(cert2.get(), cache.Lookup(server2));
51 51
52 // Remove non-existent client certificate. 52 // Remove non-existent client certificate.
53 cache.Remove(server1); 53 cache.Remove(server1);
54 EXPECT_TRUE(NULL == cache.Lookup(server1)); 54 EXPECT_EQ(NULL, cache.Lookup(server1));
55 EXPECT_EQ(cert2.get(), cache.Lookup(server2)); 55 EXPECT_EQ(cert2.get(), cache.Lookup(server2));
56 } 56 }
57 57
58 // Check that if the server differs only by port number, it is considered 58 // Check that if the server differs only by port number, it is considered
59 // a separate server. 59 // a separate server.
60 TEST(SSLClientAuthCacheTest, LookupWithPort) { 60 TEST(SSLClientAuthCacheTest, LookupWithPort) {
61 SSLClientAuthCache cache; 61 SSLClientAuthCache cache;
62 62
63 base::Time start_date = base::Time::Now(); 63 base::Time start_date = base::Time::Now();
64 base::Time expiration_date = start_date + base::TimeDelta::FromDays(1); 64 base::Time expiration_date = start_date + base::TimeDelta::FromDays(1);
65 65
66 std::string server1("foo:443"); 66 std::string server1("foo:443");
67 scoped_refptr<X509Certificate> cert1( 67 scoped_refptr<X509Certificate> cert1(
68 new X509Certificate("foo", "CA", start_date, expiration_date)); 68 new X509Certificate("foo", "CA", start_date, expiration_date));
69 69
70 std::string server2("foo:8443"); 70 std::string server2("foo:8443");
71 scoped_refptr<X509Certificate> cert2( 71 scoped_refptr<X509Certificate> cert2(
72 new X509Certificate("foo", "CA", start_date, expiration_date)); 72 new X509Certificate("foo", "CA", start_date, expiration_date));
73 73
74 cache.Add(server1, cert1.get()); 74 cache.Add(server1, cert1.get());
75 cache.Add(server2, cert2.get()); 75 cache.Add(server2, cert2.get());
76 76
77 EXPECT_EQ(cert1.get(), cache.Lookup(server1)); 77 EXPECT_EQ(cert1.get(), cache.Lookup(server1));
78 EXPECT_EQ(cert2.get(), cache.Lookup(server2)); 78 EXPECT_EQ(cert2.get(), cache.Lookup(server2));
79 } 79 }
80 80
81 } // namespace net 81 } // namespace net
OLDNEW
« no previous file with comments | « net/base/host_cache_unittest.cc ('k') | net/ftp/ftp_auth_cache_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698