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

Side by Side Diff: chrome/browser/chromeos/cros/onc_network_parser_unittest.cc

Issue 9826010: Made a ScopedMockUserManagerEnabler to simplify UserManager mocking life-cycle. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "chrome/browser/chromeos/cros/onc_network_parser.h" 5 #include "chrome/browser/chromeos/cros/onc_network_parser.h"
6 6
7 #include <cert.h> 7 #include <cert.h>
8 #include <keyhi.h> 8 #include <keyhi.h>
9 #include <pk11pub.h> 9 #include <pk11pub.h>
10 10
(...skipping 867 matching lines...) Expand 10 before | Expand all | Expand 10 after
878 net::ProxyServer(net::ProxyServer::SCHEME_SOCKS5, 878 net::ProxyServer(net::ProxyServer::SCHEME_SOCKS5,
879 net::HostPortPair(socks_host, kSocksPort))); 879 net::HostPortPair(socks_host, kSocksPort)));
880 // Verify bypass rules. 880 // Verify bypass rules.
881 net::ProxyBypassRules expected_bypass_rules; 881 net::ProxyBypassRules expected_bypass_rules;
882 expected_bypass_rules.AddRuleFromString("google.com"); 882 expected_bypass_rules.AddRuleFromString("google.com");
883 expected_bypass_rules.AddRuleToBypassLocal(); 883 expected_bypass_rules.AddRuleToBypassLocal();
884 EXPECT_TRUE(expected_bypass_rules.Equals(rules.bypass_rules)); 884 EXPECT_TRUE(expected_bypass_rules.Equals(rules.bypass_rules));
885 } 885 }
886 886
887 TEST(OncNetworkParserUserExpansionTest, GetUserExpandedValue) { 887 TEST(OncNetworkParserUserExpansionTest, GetUserExpandedValue) {
888 scoped_ptr<MockUserManager> mock_user_manager(new MockUserManager()); 888 ScopedMockUserManagerEnabler mock_user_manager;
889 UserManager* old_user_manager = UserManager::Set(mock_user_manager.get()); 889 mock_user_manager.user_manager()->SetLoggedInUser("onc@example.com", false);
890 mock_user_manager->SetLoggedInUser("onc@example.com", false);
891 890
892 EXPECT_CALL(*mock_user_manager, IsUserLoggedIn()) 891 EXPECT_CALL(*mock_user_manager.user_manager(), IsUserLoggedIn())
893 .Times(2) 892 .Times(2)
894 .WillRepeatedly(Return(false)); 893 .WillRepeatedly(Return(false));
895 894
896 NetworkUIData::ONCSource source = NetworkUIData::ONC_SOURCE_USER_IMPORT; 895 NetworkUIData::ONCSource source = NetworkUIData::ONC_SOURCE_USER_IMPORT;
897 896
898 // Setup environment needed by UserManager. 897 // Setup environment needed by UserManager.
899 MessageLoop loop; 898 MessageLoop loop;
900 content::TestBrowserThread ui_thread(content::BrowserThread::UI, &loop); 899 content::TestBrowserThread ui_thread(content::BrowserThread::UI, &loop);
901 base::ShadowingAtExitManager at_exit_manager; 900 base::ShadowingAtExitManager at_exit_manager;
902 ScopedTestingLocalState local_state( 901 ScopedTestingLocalState local_state(
903 static_cast<TestingBrowserProcess*>(g_browser_process)); 902 static_cast<TestingBrowserProcess*>(g_browser_process));
904 903
905 base::StringValue login_id_pattern("a ${LOGIN_ID} b"); 904 base::StringValue login_id_pattern("a ${LOGIN_ID} b");
906 base::StringValue login_email_pattern("a ${LOGIN_EMAIL} b"); 905 base::StringValue login_email_pattern("a ${LOGIN_EMAIL} b");
907 906
908 // No expansion if there is no user logged in. 907 // No expansion if there is no user logged in.
909 EXPECT_EQ("a ${LOGIN_ID} b", 908 EXPECT_EQ("a ${LOGIN_ID} b",
910 chromeos::OncNetworkParser::GetUserExpandedValue( 909 chromeos::OncNetworkParser::GetUserExpandedValue(
911 login_id_pattern, source)); 910 login_id_pattern, source));
912 EXPECT_EQ("a ${LOGIN_EMAIL} b", 911 EXPECT_EQ("a ${LOGIN_EMAIL} b",
913 chromeos::OncNetworkParser::GetUserExpandedValue( 912 chromeos::OncNetworkParser::GetUserExpandedValue(
914 login_email_pattern, source)); 913 login_email_pattern, source));
915 914
916 // Log in a user and check that the expansions work as expected. 915 // Log in a user and check that the expansions work as expected.
917 EXPECT_CALL(*mock_user_manager, IsUserLoggedIn()) 916 EXPECT_CALL(*mock_user_manager.user_manager(), IsUserLoggedIn())
918 .Times(2) 917 .Times(2)
919 .WillRepeatedly(Return(true)); 918 .WillRepeatedly(Return(true));
920 919
921 EXPECT_EQ("a onc b", 920 EXPECT_EQ("a onc b",
922 chromeos::OncNetworkParser::GetUserExpandedValue( 921 chromeos::OncNetworkParser::GetUserExpandedValue(
923 login_id_pattern, source)); 922 login_id_pattern, source));
924 EXPECT_EQ("a onc@example.com b", 923 EXPECT_EQ("a onc@example.com b",
925 chromeos::OncNetworkParser::GetUserExpandedValue( 924 chromeos::OncNetworkParser::GetUserExpandedValue(
926 login_email_pattern, source)); 925 login_email_pattern, source));
927
928 UserManager::Set(old_user_manager);
929 } 926 }
930 927
931 } // namespace chromeos 928 } // namespace chromeos
OLDNEW
« no previous file with comments | « chrome/browser/browser_commands_unittest.cc ('k') | chrome/browser/chromeos/cros_settings_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698