Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 1 | // Copyright 2019 The Chromium OS Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
| 5 | #include "kerberos/account_manager.h" |
| 6 | |
| 7 | #include <map> |
| 8 | #include <memory> |
| 9 | #include <utility> |
| 10 | |
| 11 | #include <base/bind.h> |
Qijiang Fan | 713061e | 2021-03-08 15:45:12 +0900 | [diff] [blame] | 12 | #include <base/check.h> |
Lutz Justen | e39cbd4 | 2019-05-14 14:52:24 +0200 | [diff] [blame] | 13 | #include <base/files/file_path.h> |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 14 | #include <base/files/file_util.h> |
| 15 | #include <base/files/scoped_temp_dir.h> |
Lutz Justen | d925d71 | 2019-06-27 17:34:00 +0200 | [diff] [blame] | 16 | #include <base/memory/ref_counted.h> |
| 17 | #include <base/test/test_mock_time_task_runner.h> |
Lutz Justen | 2d857e1 | 2019-07-16 15:58:09 +0200 | [diff] [blame] | 18 | #include <gmock/gmock.h> |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 19 | #include <gtest/gtest.h> |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 20 | #include <libpasswordprovider/fake_password_provider.h> |
| 21 | #include <libpasswordprovider/password_provider_test_utils.h> |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 22 | |
Lutz Justen | e39cbd4 | 2019-05-14 14:52:24 +0200 | [diff] [blame] | 23 | #include "kerberos/fake_krb5_interface.h" |
Lutz Justen | 2d857e1 | 2019-07-16 15:58:09 +0200 | [diff] [blame] | 24 | #include "kerberos/kerberos_metrics.h" |
Lutz Justen | 3360180 | 2019-07-03 22:40:57 +0200 | [diff] [blame] | 25 | #include "kerberos/krb5_jail_wrapper.h" |
Lutz Justen | e39cbd4 | 2019-05-14 14:52:24 +0200 | [diff] [blame] | 26 | |
Lutz Justen | 2d857e1 | 2019-07-16 15:58:09 +0200 | [diff] [blame] | 27 | using testing::_; |
| 28 | using testing::Mock; |
| 29 | using testing::NiceMock; |
| 30 | using testing::Return; |
| 31 | |
Lutz Justen | d925d71 | 2019-06-27 17:34:00 +0200 | [diff] [blame] | 32 | namespace kerberos { |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 33 | namespace { |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 34 | |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 35 | constexpr char kUser[] = "user@REALM.COM"; |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 36 | constexpr char kUser2[] = "user2@REALM2.COM"; |
Lutz Justen | 781825e | 2019-07-13 07:49:44 +0200 | [diff] [blame] | 37 | constexpr char kUser3[] = "user3@REALM3.COM"; |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 38 | constexpr char kPassword[] = "i<3k3R8e5Oz"; |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 39 | constexpr char kPassword2[] = "ih4zf00d"; |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 40 | constexpr char kKrb5Conf[] = R"( |
| 41 | [libdefaults] |
Felipe Andrade | 66aaf6b | 2020-03-24 13:05:57 +0100 | [diff] [blame] | 42 | forwardable = true)"; |
| 43 | constexpr char kStrongKrb5Conf[] = R"( |
| 44 | [libdefaults] |
| 45 | forwardable = true |
| 46 | default_tkt_enctypes = aes256-cts-hmac-sha1-96 |
| 47 | default_tgs_enctypes = aes256-cts-hmac-sha1-96 |
| 48 | permitted_enctypes = aes256-cts-hmac-sha1-96)"; |
| 49 | constexpr char kLegacyKrb5Conf[] = R"( |
| 50 | [libdefaults] |
| 51 | forwardable = true |
| 52 | default_tkt_enctypes = arcfour-hmac |
| 53 | default_tgs_enctypes = arcfour-hmac |
| 54 | permitted_enctypes = arcfour-hmac)"; |
Felipe Andrade | 90cb84e | 2020-04-07 20:28:33 +0200 | [diff] [blame] | 55 | constexpr char kInvalidKrb5Conf[] = R"( |
| 56 | [libdefaults] |
| 57 | stonkskew = 123)"; |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 58 | |
Lutz Justen | d925d71 | 2019-06-27 17:34:00 +0200 | [diff] [blame] | 59 | constexpr Krb5Interface::TgtStatus kValidTgt(3600, 3600); |
| 60 | constexpr Krb5Interface::TgtStatus kExpiredTgt(0, 0); |
| 61 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 62 | // Convenience defines to make code more readable |
| 63 | constexpr bool kManaged = true; |
| 64 | constexpr bool kUnmanaged = false; |
| 65 | |
| 66 | constexpr bool kRememberPassword = true; |
| 67 | constexpr bool kDontRememberPassword = false; |
| 68 | |
| 69 | constexpr bool kUseLoginPassword = true; |
| 70 | constexpr bool kDontUseLoginPassword = false; |
| 71 | |
| 72 | constexpr char kEmptyPassword[] = ""; |
| 73 | |
Lutz Justen | 2d857e1 | 2019-07-16 15:58:09 +0200 | [diff] [blame] | 74 | class MockMetrics : public KerberosMetrics { |
| 75 | public: |
| 76 | explicit MockMetrics(const base::FilePath& storage_dir) |
| 77 | : KerberosMetrics(storage_dir) {} |
Qijiang Fan | 6bc59e1 | 2020-11-11 02:51:06 +0900 | [diff] [blame] | 78 | MockMetrics(const MockMetrics&) = delete; |
| 79 | MockMetrics& operator=(const MockMetrics&) = delete; |
| 80 | |
Lutz Justen | 2d857e1 | 2019-07-16 15:58:09 +0200 | [diff] [blame] | 81 | ~MockMetrics() override = default; |
| 82 | |
Ben Chan | 6047001 | 2019-09-24 14:01:54 -0700 | [diff] [blame] | 83 | MOCK_METHOD(bool, ShouldReportDailyUsageStats, (), (override)); |
| 84 | MOCK_METHOD(void, |
| 85 | ReportDailyUsageStats, |
| 86 | (int, int, int, int, int), |
| 87 | (override)); |
Felipe Andrade | 66aaf6b | 2020-03-24 13:05:57 +0100 | [diff] [blame] | 88 | MOCK_METHOD(void, |
| 89 | ReportKerberosEncryptionTypes, |
| 90 | (KerberosEncryptionTypes), |
| 91 | (override)); |
Lutz Justen | 2d857e1 | 2019-07-16 15:58:09 +0200 | [diff] [blame] | 92 | }; |
| 93 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 94 | } // namespace |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 95 | |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 96 | class AccountManagerTest : public ::testing::Test { |
| 97 | public: |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 98 | AccountManagerTest() |
| 99 | : kerberos_files_changed_( |
| 100 | base::BindRepeating(&AccountManagerTest::OnKerberosFilesChanged, |
Lutz Justen | 8eeb4a2 | 2019-06-19 17:00:32 +0200 | [diff] [blame] | 101 | base::Unretained(this))), |
| 102 | kerberos_ticket_expiring_( |
| 103 | base::BindRepeating(&AccountManagerTest::OnKerberosTicketExpiring, |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 104 | base::Unretained(this))) {} |
Qijiang Fan | 6bc59e1 | 2020-11-11 02:51:06 +0900 | [diff] [blame] | 105 | AccountManagerTest(const AccountManagerTest&) = delete; |
| 106 | AccountManagerTest& operator=(const AccountManagerTest&) = delete; |
| 107 | |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 108 | ~AccountManagerTest() override = default; |
| 109 | |
| 110 | void SetUp() override { |
| 111 | ::testing::Test::SetUp(); |
| 112 | |
| 113 | // Create temp directory for files written during tests. |
| 114 | CHECK(storage_dir_.CreateUniqueTempDir()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 115 | accounts_path_ = storage_dir_.GetPath().Append("accounts"); |
Lutz Justen | 3360180 | 2019-07-03 22:40:57 +0200 | [diff] [blame] | 116 | account_dir_ = storage_dir_.GetPath().Append( |
Lutz Justen | c569050 | 2019-05-24 17:00:54 +0200 | [diff] [blame] | 117 | AccountManager::GetSafeFilenameForTesting(kUser)); |
Lutz Justen | 3360180 | 2019-07-03 22:40:57 +0200 | [diff] [blame] | 118 | krb5cc_path_ = account_dir_.Append("krb5cc"); |
| 119 | krb5conf_path_ = account_dir_.Append("krb5.conf"); |
| 120 | password_path_ = account_dir_.Append("password"); |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 121 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 122 | // Create the manager with a fake krb5 interface. |
| 123 | auto krb5 = std::make_unique<FakeKrb5Interface>(); |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 124 | auto password_provider = |
| 125 | std::make_unique<password_provider::FakePasswordProvider>(); |
Lutz Justen | 2d857e1 | 2019-07-16 15:58:09 +0200 | [diff] [blame] | 126 | metrics_ = std::make_unique<NiceMock<MockMetrics>>(storage_dir_.GetPath()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 127 | krb5_ = krb5.get(); |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 128 | password_provider_ = password_provider.get(); |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 129 | manager_ = std::make_unique<AccountManager>( |
Lutz Justen | 8eeb4a2 | 2019-06-19 17:00:32 +0200 | [diff] [blame] | 130 | storage_dir_.GetPath(), kerberos_files_changed_, |
| 131 | kerberos_ticket_expiring_, std::move(krb5), |
Lutz Justen | 2d857e1 | 2019-07-16 15:58:09 +0200 | [diff] [blame] | 132 | std::move(password_provider), metrics_.get()); |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 133 | } |
| 134 | |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 135 | void TearDown() override { |
| 136 | // Make sure the file stored on disk contains the same accounts as the |
| 137 | // manager instance. This catches cases where AccountManager forgets to save |
| 138 | // accounts on some change. |
| 139 | if (base::PathExists(accounts_path_)) { |
Felipe Andrade | 89399f0 | 2019-11-11 15:52:14 +0100 | [diff] [blame] | 140 | std::vector<Account> accounts = manager_->ListAccounts(); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 141 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 142 | AccountManager other_manager( |
| 143 | storage_dir_.GetPath(), kerberos_files_changed_, |
Lutz Justen | 8eeb4a2 | 2019-06-19 17:00:32 +0200 | [diff] [blame] | 144 | kerberos_ticket_expiring_, std::make_unique<FakeKrb5Interface>(), |
Lutz Justen | 2d857e1 | 2019-07-16 15:58:09 +0200 | [diff] [blame] | 145 | std::make_unique<password_provider::FakePasswordProvider>(), |
| 146 | metrics_.get()); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 147 | other_manager.LoadAccounts(); |
Felipe Andrade | 89399f0 | 2019-11-11 15:52:14 +0100 | [diff] [blame] | 148 | std::vector<Account> other_accounts = other_manager.ListAccounts(); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 149 | |
| 150 | ASSERT_NO_FATAL_FAILURE(ExpectAccountsEqual(accounts, other_accounts)); |
| 151 | } |
| 152 | |
| 153 | ::testing::Test::TearDown(); |
| 154 | } |
| 155 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 156 | // Add account with default settings. |
| 157 | ErrorType AddAccount() { return manager_->AddAccount(kUser, kUnmanaged); } |
| 158 | |
| 159 | // Sets some default Kerberos configuration. |
Felipe Andrade | 66aaf6b | 2020-03-24 13:05:57 +0100 | [diff] [blame] | 160 | ErrorType SetConfig() { return SetConfig(kKrb5Conf); } |
| 161 | |
| 162 | // Sets Kerberos configuration. |
| 163 | ErrorType SetConfig(const std::string& config) { |
| 164 | return manager_->SetConfig(kUser, config); |
| 165 | } |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 166 | |
| 167 | // Acquire Kerberos ticket with default credentials and settings. |
| 168 | ErrorType AcquireTgt() { |
| 169 | return manager_->AcquireTgt(kUser, kPassword, kDontRememberPassword, |
| 170 | kDontUseLoginPassword); |
| 171 | } |
| 172 | |
| 173 | void SaveLoginPassword(const char* password) { |
| 174 | auto password_ptr = password_provider::test::CreatePassword(password); |
| 175 | password_provider_->SavePassword(*password_ptr); |
| 176 | } |
| 177 | |
Lutz Justen | d925d71 | 2019-06-27 17:34:00 +0200 | [diff] [blame] | 178 | // Fast forwards to the next scheduled task (assumed to be the renewal task) |
| 179 | // and verifies expectation that |krb5_->RenewTgt() was called|. |
| 180 | void RunScheduledRenewalTask() { |
| 181 | int initial_count = krb5_->renew_tgt_call_count(); |
| 182 | EXPECT_EQ(1, task_runner_->GetPendingTaskCount()); |
| 183 | task_runner_->FastForwardBy(task_runner_->NextPendingTaskDelay()); |
| 184 | EXPECT_EQ(initial_count + 1, krb5_->renew_tgt_call_count()); |
| 185 | } |
| 186 | |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 187 | protected: |
| 188 | void OnKerberosFilesChanged(const std::string& principal_name) { |
| 189 | kerberos_files_changed_count_[principal_name]++; |
| 190 | } |
| 191 | |
Lutz Justen | 8eeb4a2 | 2019-06-19 17:00:32 +0200 | [diff] [blame] | 192 | void OnKerberosTicketExpiring(const std::string& principal_name) { |
| 193 | kerberos_ticket_expiring_count_[principal_name]++; |
| 194 | } |
| 195 | |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 196 | void ExpectAccountsEqual(const std::vector<Account>& account_list_1, |
| 197 | const std::vector<Account>& account_list_2) { |
| 198 | ASSERT_EQ(account_list_1.size(), account_list_2.size()); |
| 199 | for (size_t n = 0; n < account_list_1.size(); ++n) { |
| 200 | const Account& account1 = account_list_1[n]; |
| 201 | const Account& account2 = account_list_2[n]; |
| 202 | |
| 203 | EXPECT_EQ(account1.principal_name(), account2.principal_name()); |
| 204 | EXPECT_EQ(account1.is_managed(), account2.is_managed()); |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 205 | EXPECT_EQ(account1.use_login_password(), account2.use_login_password()); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 206 | // TODO(https://crbug.com/952239): Check additional properties. |
| 207 | } |
| 208 | } |
| 209 | |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 210 | std::unique_ptr<AccountManager> manager_; |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 211 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 212 | // Fake Kerberos interface used by |manager_|. Not owned. |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 213 | FakeKrb5Interface* krb5_; |
| 214 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 215 | // Fake password provider to get the login password. Not owned. |
| 216 | password_provider::FakePasswordProvider* password_provider_; |
| 217 | |
Lutz Justen | 2d857e1 | 2019-07-16 15:58:09 +0200 | [diff] [blame] | 218 | // Mock metrics for testing UMA stat recording. |
| 219 | std::unique_ptr<NiceMock<MockMetrics>> metrics_; |
| 220 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 221 | // Paths of files stored by |manager_|. |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 222 | base::ScopedTempDir storage_dir_; |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 223 | base::FilePath accounts_path_; |
Lutz Justen | 3360180 | 2019-07-03 22:40:57 +0200 | [diff] [blame] | 224 | base::FilePath account_dir_; |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 225 | base::FilePath krb5conf_path_; |
| 226 | base::FilePath krb5cc_path_; |
Lutz Justen | c569050 | 2019-05-24 17:00:54 +0200 | [diff] [blame] | 227 | base::FilePath password_path_; |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 228 | |
| 229 | AccountManager::KerberosFilesChangedCallback kerberos_files_changed_; |
Lutz Justen | 8eeb4a2 | 2019-06-19 17:00:32 +0200 | [diff] [blame] | 230 | AccountManager::KerberosTicketExpiringCallback kerberos_ticket_expiring_; |
| 231 | |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 232 | std::map<std::string, int> kerberos_files_changed_count_; |
Lutz Justen | 8eeb4a2 | 2019-06-19 17:00:32 +0200 | [diff] [blame] | 233 | std::map<std::string, int> kerberos_ticket_expiring_count_; |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 234 | |
Lutz Justen | d925d71 | 2019-06-27 17:34:00 +0200 | [diff] [blame] | 235 | scoped_refptr<base::TestMockTimeTaskRunner> task_runner_{ |
| 236 | new base::TestMockTimeTaskRunner()}; |
| 237 | base::TestMockTimeTaskRunner::ScopedContext scoped_context_{task_runner_}; |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 238 | }; |
| 239 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 240 | // Adding an account succeeds and serializes the file on disk. |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 241 | TEST_F(AccountManagerTest, AddAccountSuccess) { |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 242 | EXPECT_FALSE(base::PathExists(accounts_path_)); |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 243 | EXPECT_EQ(ERROR_NONE, AddAccount()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 244 | EXPECT_TRUE(base::PathExists(accounts_path_)); |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 245 | } |
| 246 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 247 | // AddAccount() fails if the same account is added twice. |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 248 | TEST_F(AccountManagerTest, AddDuplicateAccountFail) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 249 | ignore_result(AddAccount()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 250 | |
hscham | 53cf73a | 2020-11-30 15:58:42 +0900 | [diff] [blame] | 251 | EXPECT_TRUE(base::DeleteFile(accounts_path_)); |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 252 | EXPECT_EQ(ERROR_DUPLICATE_PRINCIPAL_NAME, AddAccount()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 253 | EXPECT_FALSE(base::PathExists(accounts_path_)); |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 254 | } |
| 255 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 256 | // Adding a managed account overwrites an existing unmanaged account. |
| 257 | TEST_F(AccountManagerTest, ManagedOverridesUnmanaged) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 258 | ignore_result(manager_->AddAccount(kUser, kUnmanaged)); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 259 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 260 | EXPECT_EQ(ERROR_NONE, AcquireTgt()); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 261 | EXPECT_TRUE(base::PathExists(krb5cc_path_)); |
| 262 | |
| 263 | // Overwriting with a managed account should wipe existing files and make the |
| 264 | // account managed. |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 265 | EXPECT_EQ(ERROR_DUPLICATE_PRINCIPAL_NAME, |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 266 | manager_->AddAccount(kUser, kManaged)); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 267 | EXPECT_FALSE(base::PathExists(krb5cc_path_)); |
| 268 | |
Felipe Andrade | 89399f0 | 2019-11-11 15:52:14 +0100 | [diff] [blame] | 269 | std::vector<Account> accounts = manager_->ListAccounts(); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 270 | ASSERT_EQ(1u, accounts.size()); |
| 271 | EXPECT_TRUE(accounts[0].is_managed()); |
| 272 | } |
| 273 | |
| 274 | // Adding an unmanaged account does not overwrite an existing managed account. |
| 275 | TEST_F(AccountManagerTest, UnmanagedDoesNotOverrideManaged) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 276 | ignore_result(manager_->AddAccount(kUser, kManaged)); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 277 | |
| 278 | EXPECT_EQ(ERROR_DUPLICATE_PRINCIPAL_NAME, |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 279 | manager_->AddAccount(kUser, kUnmanaged)); |
Felipe Andrade | 89399f0 | 2019-11-11 15:52:14 +0100 | [diff] [blame] | 280 | std::vector<Account> accounts = manager_->ListAccounts(); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 281 | ASSERT_EQ(1u, accounts.size()); |
| 282 | EXPECT_TRUE(accounts[0].is_managed()); |
| 283 | } |
| 284 | |
| 285 | // RemoveAccount() succeeds if the account exists and serializes the file on |
| 286 | // disk. |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 287 | TEST_F(AccountManagerTest, RemoveAccountSuccess) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 288 | ignore_result(AddAccount()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 289 | |
hscham | 53cf73a | 2020-11-30 15:58:42 +0900 | [diff] [blame] | 290 | EXPECT_TRUE(base::DeleteFile(accounts_path_)); |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 291 | EXPECT_EQ(ERROR_NONE, manager_->RemoveAccount(kUser)); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 292 | EXPECT_TRUE(base::PathExists(accounts_path_)); |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 293 | } |
| 294 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 295 | // RemoveAccount() fails if the account does not exist. |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 296 | TEST_F(AccountManagerTest, RemoveUnknownAccountFail) { |
| 297 | EXPECT_EQ(ERROR_UNKNOWN_PRINCIPAL_NAME, manager_->RemoveAccount(kUser)); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 298 | EXPECT_FALSE(base::PathExists(accounts_path_)); |
| 299 | } |
| 300 | |
| 301 | // RemoveAccount() does not trigger KerberosFilesChanged if the credential cache |
| 302 | // does not exists. |
| 303 | TEST_F(AccountManagerTest, RemoveAccountTriggersKFCIfCCExists) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 304 | ignore_result(AddAccount()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 305 | |
| 306 | EXPECT_EQ(ERROR_NONE, manager_->RemoveAccount(kUser)); |
| 307 | EXPECT_EQ(0, kerberos_files_changed_count_[kUser]); |
| 308 | } |
| 309 | |
| 310 | // RemoveAccount() triggers KerberosFilesChanged if the credential cache exists. |
| 311 | TEST_F(AccountManagerTest, RemoveAccountDoesNotTriggerKFCIfCCDoesNotExist) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 312 | ignore_result(AddAccount()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 313 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 314 | EXPECT_EQ(ERROR_NONE, AcquireTgt()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 315 | EXPECT_EQ(1, kerberos_files_changed_count_[kUser]); |
| 316 | EXPECT_EQ(ERROR_NONE, manager_->RemoveAccount(kUser)); |
| 317 | EXPECT_EQ(2, kerberos_files_changed_count_[kUser]); |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 318 | } |
| 319 | |
| 320 | // Repeatedly calling AddAccount() and RemoveAccount() succeeds. |
| 321 | TEST_F(AccountManagerTest, RepeatedAddRemoveSuccess) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 322 | ignore_result(AddAccount()); |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 323 | ignore_result(manager_->RemoveAccount(kUser)); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 324 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 325 | EXPECT_EQ(ERROR_NONE, AddAccount()); |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 326 | EXPECT_EQ(ERROR_NONE, manager_->RemoveAccount(kUser)); |
| 327 | } |
| 328 | |
Lutz Justen | 2d42087 | 2019-06-25 14:17:21 +0200 | [diff] [blame] | 329 | // ClearAccounts(CLEAR_ALL) clears all accounts. |
| 330 | TEST_F(AccountManagerTest, ClearAccountsSuccess) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 331 | ignore_result(manager_->AddAccount(kUser, kUnmanaged)); |
| 332 | ignore_result(manager_->AddAccount(kUser2, kManaged)); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 333 | |
Lutz Justen | 781825e | 2019-07-13 07:49:44 +0200 | [diff] [blame] | 334 | EXPECT_EQ(ERROR_NONE, manager_->ClearAccounts(CLEAR_ALL, {})); |
Felipe Andrade | 89399f0 | 2019-11-11 15:52:14 +0100 | [diff] [blame] | 335 | std::vector<Account> accounts = manager_->ListAccounts(); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 336 | EXPECT_EQ(0u, accounts.size()); |
| 337 | } |
| 338 | |
Lutz Justen | 2d42087 | 2019-06-25 14:17:21 +0200 | [diff] [blame] | 339 | // ClearAccounts(CLEAR_ALL) wipes Kerberos configuration and credential cache. |
| 340 | TEST_F(AccountManagerTest, ClearAccountsRemovesKerberosFiles) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 341 | ignore_result(AddAccount()); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 342 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 343 | EXPECT_EQ(ERROR_NONE, SetConfig()); |
| 344 | EXPECT_EQ(ERROR_NONE, AcquireTgt()); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 345 | EXPECT_TRUE(base::PathExists(krb5conf_path_)); |
| 346 | EXPECT_TRUE(base::PathExists(krb5cc_path_)); |
Lutz Justen | 781825e | 2019-07-13 07:49:44 +0200 | [diff] [blame] | 347 | EXPECT_EQ(ERROR_NONE, manager_->ClearAccounts(CLEAR_ALL, {})); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 348 | EXPECT_FALSE(base::PathExists(krb5conf_path_)); |
| 349 | EXPECT_FALSE(base::PathExists(krb5cc_path_)); |
| 350 | } |
| 351 | |
Lutz Justen | 2d42087 | 2019-06-25 14:17:21 +0200 | [diff] [blame] | 352 | // ClearAccounts(CLEAR_ALL) triggers KerberosFilesChanged if the credential |
| 353 | // cache exists. |
| 354 | TEST_F(AccountManagerTest, ClearAccountsTriggersKFCIfCCExists) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 355 | ignore_result(AddAccount()); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 356 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 357 | EXPECT_EQ(ERROR_NONE, AcquireTgt()); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 358 | EXPECT_EQ(1, kerberos_files_changed_count_[kUser]); |
Lutz Justen | 781825e | 2019-07-13 07:49:44 +0200 | [diff] [blame] | 359 | EXPECT_EQ(ERROR_NONE, manager_->ClearAccounts(CLEAR_ALL, {})); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 360 | EXPECT_EQ(2, kerberos_files_changed_count_[kUser]); |
| 361 | } |
| 362 | |
Lutz Justen | 2d42087 | 2019-06-25 14:17:21 +0200 | [diff] [blame] | 363 | // ClearAccounts(CLEAR_ALL) does not trigger KerberosFilesChanged if the |
| 364 | // credential cache does not exist. |
| 365 | TEST_F(AccountManagerTest, ClearAccountsDoesNotTriggerKFCIfDoesNotCCExist) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 366 | ignore_result(AddAccount()); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 367 | |
Lutz Justen | 781825e | 2019-07-13 07:49:44 +0200 | [diff] [blame] | 368 | EXPECT_EQ(ERROR_NONE, manager_->ClearAccounts(CLEAR_ALL, {})); |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 369 | EXPECT_EQ(0, kerberos_files_changed_count_[kUser]); |
| 370 | } |
| 371 | |
Lutz Justen | 2d42087 | 2019-06-25 14:17:21 +0200 | [diff] [blame] | 372 | // ClearAccounts(CLEAR_ONLY_UNMANAGED_ACCOUNTS) clears only unmanaged accounts. |
| 373 | TEST_F(AccountManagerTest, ClearUnmanagedAccountsSuccess) { |
| 374 | ignore_result(manager_->AddAccount(kUser, kUnmanaged)); |
| 375 | ignore_result(manager_->AddAccount(kUser2, kManaged)); |
| 376 | |
Lutz Justen | 781825e | 2019-07-13 07:49:44 +0200 | [diff] [blame] | 377 | EXPECT_EQ(ERROR_NONE, |
| 378 | manager_->ClearAccounts(CLEAR_ONLY_UNMANAGED_ACCOUNTS, {})); |
Felipe Andrade | 89399f0 | 2019-11-11 15:52:14 +0100 | [diff] [blame] | 379 | std::vector<Account> accounts = manager_->ListAccounts(); |
Lutz Justen | 2d42087 | 2019-06-25 14:17:21 +0200 | [diff] [blame] | 380 | ASSERT_EQ(1u, accounts.size()); |
| 381 | EXPECT_EQ(kUser2, accounts[0].principal_name()); |
| 382 | } |
| 383 | |
| 384 | // ClearAccounts(CLEAR_ONLY_UNMANAGED_REMEMBERED_PASSWORDS) clears only |
| 385 | // passwords of unmanaged accounts. |
| 386 | TEST_F(AccountManagerTest, ClearUnmanagedPasswordsSuccess) { |
| 387 | // kUser is unmanaged, kUser2 is managed. |
| 388 | ignore_result(manager_->AddAccount(kUser, kUnmanaged)); |
| 389 | ignore_result(manager_->AddAccount(kUser2, kManaged)); |
| 390 | ignore_result(manager_->AcquireTgt(kUser, kPassword, kRememberPassword, |
| 391 | kDontUseLoginPassword)); |
| 392 | ignore_result(manager_->AcquireTgt(kUser2, kPassword, kRememberPassword, |
| 393 | kDontUseLoginPassword)); |
| 394 | |
| 395 | base::FilePath password_path_2 = |
| 396 | storage_dir_.GetPath() |
| 397 | .Append(AccountManager::GetSafeFilenameForTesting(kUser2)) |
| 398 | .Append("password"); |
| 399 | EXPECT_TRUE(base::PathExists(password_path_)); |
| 400 | EXPECT_TRUE(base::PathExists(password_path_2)); |
| 401 | |
Lutz Justen | 781825e | 2019-07-13 07:49:44 +0200 | [diff] [blame] | 402 | EXPECT_EQ(ERROR_NONE, manager_->ClearAccounts( |
| 403 | CLEAR_ONLY_UNMANAGED_REMEMBERED_PASSWORDS, {})); |
Felipe Andrade | 89399f0 | 2019-11-11 15:52:14 +0100 | [diff] [blame] | 404 | std::vector<Account> accounts = manager_->ListAccounts(); |
Lutz Justen | 2d42087 | 2019-06-25 14:17:21 +0200 | [diff] [blame] | 405 | ASSERT_EQ(2u, accounts.size()); |
| 406 | EXPECT_FALSE(base::PathExists(password_path_)); |
| 407 | EXPECT_TRUE(base::PathExists(password_path_2)); |
| 408 | } |
| 409 | |
Lutz Justen | 781825e | 2019-07-13 07:49:44 +0200 | [diff] [blame] | 410 | // ClearAccounts(CLEAR_ONLY_MANAGED_ACCOUNTS) clears only managed accounts that |
| 411 | // are not on the keep list. |
| 412 | TEST_F(AccountManagerTest, ClearManagedPasswordsWithKeepListSuccess) { |
| 413 | ignore_result(manager_->AddAccount(kUser, kManaged)); |
| 414 | ignore_result(manager_->AddAccount(kUser2, kManaged)); |
| 415 | ignore_result(manager_->AddAccount(kUser3, kUnmanaged)); |
| 416 | |
| 417 | // Keep the managed kUser-account. |
| 418 | EXPECT_EQ(ERROR_NONE, |
| 419 | manager_->ClearAccounts(CLEAR_ONLY_MANAGED_ACCOUNTS, {kUser})); |
Felipe Andrade | 89399f0 | 2019-11-11 15:52:14 +0100 | [diff] [blame] | 420 | std::vector<Account> accounts = manager_->ListAccounts(); |
Lutz Justen | 781825e | 2019-07-13 07:49:44 +0200 | [diff] [blame] | 421 | ASSERT_EQ(2u, accounts.size()); |
| 422 | EXPECT_EQ(kUser, accounts[0].principal_name()); |
| 423 | EXPECT_EQ(kUser3, accounts[1].principal_name()); |
| 424 | } |
| 425 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 426 | // SetConfig() succeeds and writes the config to |krb5conf_path_|. |
| 427 | TEST_F(AccountManagerTest, SetConfigSuccess) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 428 | ignore_result(AddAccount()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 429 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 430 | EXPECT_EQ(ERROR_NONE, SetConfig()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 431 | std::string krb5_conf; |
| 432 | EXPECT_TRUE(base::ReadFileToString(krb5conf_path_, &krb5_conf)); |
| 433 | EXPECT_EQ(krb5_conf, kKrb5Conf); |
| 434 | } |
| 435 | |
Lutz Justen | 9028140 | 2019-07-05 15:14:37 +0200 | [diff] [blame] | 436 | // SetConfig() calls ValidateConfig on the Kerberos interface. |
| 437 | TEST_F(AccountManagerTest, SetConfigValidatesConfig) { |
| 438 | ignore_result(AddAccount()); |
| 439 | |
| 440 | krb5_->set_validate_config_error(ERROR_BAD_CONFIG); |
| 441 | EXPECT_EQ(ERROR_BAD_CONFIG, SetConfig()); |
| 442 | } |
| 443 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 444 | // SetConfig() triggers KerberosFilesChanged if the credential cache exists. |
| 445 | TEST_F(AccountManagerTest, SetConfigTriggersKFCIfCCExists) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 446 | ignore_result(AddAccount()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 447 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 448 | EXPECT_EQ(ERROR_NONE, AcquireTgt()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 449 | EXPECT_EQ(1, kerberos_files_changed_count_[kUser]); |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 450 | EXPECT_EQ(ERROR_NONE, SetConfig()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 451 | EXPECT_EQ(2, kerberos_files_changed_count_[kUser]); |
| 452 | } |
| 453 | |
Lutz Justen | 82d07ce | 2019-05-16 10:46:14 +0200 | [diff] [blame] | 454 | // SetConfig() does not trigger KerberosFilesChanged if the credential cache |
| 455 | // does not exist. |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 456 | TEST_F(AccountManagerTest, SetConfigDoesNotTriggerKFCIfDoesNotCCExist) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 457 | ignore_result(AddAccount()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 458 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 459 | EXPECT_EQ(ERROR_NONE, SetConfig()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 460 | EXPECT_EQ(0, kerberos_files_changed_count_[kUser]); |
| 461 | } |
| 462 | |
| 463 | // RemoveAccount() removes the config file. |
| 464 | TEST_F(AccountManagerTest, RemoveAccountRemovesConfig) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 465 | ignore_result(AddAccount()); |
| 466 | ignore_result(SetConfig()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 467 | |
| 468 | EXPECT_TRUE(base::PathExists(krb5conf_path_)); |
| 469 | EXPECT_EQ(ERROR_NONE, manager_->RemoveAccount(kUser)); |
| 470 | EXPECT_FALSE(base::PathExists(krb5conf_path_)); |
| 471 | } |
| 472 | |
Lutz Justen | 9028140 | 2019-07-05 15:14:37 +0200 | [diff] [blame] | 473 | // ValidateConfig() validates a good config successfully. |
| 474 | TEST_F(AccountManagerTest, ValidateConfigSuccess) { |
| 475 | constexpr char kValidKrb5Conf[] = ""; |
| 476 | ConfigErrorInfo error_info; |
| 477 | EXPECT_EQ(ERROR_NONE, manager_->ValidateConfig(kValidKrb5Conf, &error_info)); |
| 478 | EXPECT_EQ(CONFIG_ERROR_NONE, error_info.code()); |
| 479 | } |
| 480 | |
| 481 | // ValidateConfig() returns the correct error for a bad config. |
| 482 | TEST_F(AccountManagerTest, ValidateConfigFailure) { |
| 483 | ConfigErrorInfo expected_error_info; |
| 484 | expected_error_info.set_code(CONFIG_ERROR_SECTION_SYNTAX); |
| 485 | krb5_->set_config_error_info(expected_error_info); |
| 486 | krb5_->set_validate_config_error(ERROR_BAD_CONFIG); |
| 487 | |
| 488 | constexpr char kBadKrb5Conf[] = |
| 489 | "[libdefaults]'); DROP TABLE KerberosTickets;--"; |
| 490 | ConfigErrorInfo error_info; |
| 491 | EXPECT_EQ(ERROR_BAD_CONFIG, |
| 492 | manager_->ValidateConfig(kBadKrb5Conf, &error_info)); |
| 493 | EXPECT_EQ(expected_error_info.SerializeAsString(), |
| 494 | error_info.SerializeAsString()); |
| 495 | } |
| 496 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 497 | // AcquireTgt() succeeds and writes a credential cache file. |
| 498 | TEST_F(AccountManagerTest, AcquireTgtSuccess) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 499 | ignore_result(AddAccount()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 500 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 501 | EXPECT_EQ(ERROR_NONE, AcquireTgt()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 502 | EXPECT_TRUE(base::PathExists(krb5cc_path_)); |
| 503 | } |
| 504 | |
| 505 | // AcquireTgt() triggers KerberosFilesChanged on success. |
| 506 | TEST_F(AccountManagerTest, AcquireTgtTriggersKFCOnSuccess) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 507 | ignore_result(AddAccount()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 508 | |
| 509 | EXPECT_EQ(0, kerberos_files_changed_count_[kUser]); |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 510 | EXPECT_EQ(ERROR_NONE, AcquireTgt()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 511 | EXPECT_EQ(1, kerberos_files_changed_count_[kUser]); |
| 512 | } |
| 513 | |
| 514 | // AcquireTgt() does not trigger KerberosFilesChanged on failure. |
| 515 | TEST_F(AccountManagerTest, AcquireTgtDoesNotTriggerKFCOnFailure) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 516 | ignore_result(AddAccount()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 517 | |
| 518 | krb5_->set_acquire_tgt_error(ERROR_UNKNOWN); |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 519 | EXPECT_EQ(ERROR_UNKNOWN, AcquireTgt()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 520 | EXPECT_EQ(0, kerberos_files_changed_count_[kUser]); |
| 521 | } |
| 522 | |
Lutz Justen | c569050 | 2019-05-24 17:00:54 +0200 | [diff] [blame] | 523 | // AcquireTgt() saves password to disk if |remember_password| is true and |
| 524 | // removes the file again if |remember_password| is false. |
| 525 | TEST_F(AccountManagerTest, AcquireTgtRemembersPasswordsIfWanted) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 526 | ignore_result(AddAccount()); |
Lutz Justen | c569050 | 2019-05-24 17:00:54 +0200 | [diff] [blame] | 527 | |
| 528 | EXPECT_FALSE(base::PathExists(password_path_)); |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 529 | EXPECT_EQ(ERROR_NONE, |
| 530 | manager_->AcquireTgt(kUser, kPassword, kRememberPassword, |
| 531 | kDontUseLoginPassword)); |
Lutz Justen | c569050 | 2019-05-24 17:00:54 +0200 | [diff] [blame] | 532 | EXPECT_TRUE(base::PathExists(password_path_)); |
| 533 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 534 | EXPECT_EQ(ERROR_NONE, |
| 535 | manager_->AcquireTgt(kUser, kPassword, kDontRememberPassword, |
| 536 | kDontUseLoginPassword)); |
Lutz Justen | c569050 | 2019-05-24 17:00:54 +0200 | [diff] [blame] | 537 | EXPECT_FALSE(base::PathExists(password_path_)); |
| 538 | } |
| 539 | |
| 540 | // AcquireTgt() uses saved password if none is given, no matter if it should be |
| 541 | // remembered again or not. |
| 542 | TEST_F(AccountManagerTest, AcquireTgtLoadsRememberedPassword) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 543 | ignore_result(AddAccount()); |
| 544 | ignore_result(manager_->AcquireTgt(kUser, kPassword, kRememberPassword, |
| 545 | kDontUseLoginPassword)); |
Lutz Justen | c569050 | 2019-05-24 17:00:54 +0200 | [diff] [blame] | 546 | |
| 547 | // This should load stored password and keep it. |
| 548 | EXPECT_EQ(ERROR_NONE, |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 549 | manager_->AcquireTgt(kUser, kEmptyPassword, kRememberPassword, |
| 550 | kDontUseLoginPassword)); |
Lutz Justen | c569050 | 2019-05-24 17:00:54 +0200 | [diff] [blame] | 551 | EXPECT_TRUE(base::PathExists(password_path_)); |
| 552 | |
| 553 | // This should load stored password, but erase it afterwards. |
| 554 | EXPECT_EQ(ERROR_NONE, |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 555 | manager_->AcquireTgt(kUser, kEmptyPassword, kDontRememberPassword, |
| 556 | kDontUseLoginPassword)); |
Lutz Justen | c569050 | 2019-05-24 17:00:54 +0200 | [diff] [blame] | 557 | EXPECT_FALSE(base::PathExists(password_path_)); |
| 558 | |
| 559 | // Check that the fake krb5 interface returns an error for a missing password. |
| 560 | // This verifies that the above AcquireTgt() call actually loaded the |
| 561 | // password from disk. |
| 562 | EXPECT_EQ(ERROR_BAD_PASSWORD, |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 563 | manager_->AcquireTgt(kUser, kEmptyPassword, kDontRememberPassword, |
| 564 | kDontUseLoginPassword)); |
| 565 | } |
| 566 | |
| 567 | // AcquireTgt() uses the login password if saved. |
| 568 | TEST_F(AccountManagerTest, AcquireTgtUsesLoginPassword) { |
| 569 | ignore_result(AddAccount()); |
| 570 | |
| 571 | // Shouldn't explode if the login password not set yet. |
| 572 | EXPECT_EQ(ERROR_BAD_PASSWORD, |
| 573 | manager_->AcquireTgt(kUser, kEmptyPassword, kDontRememberPassword, |
| 574 | kUseLoginPassword)); |
| 575 | |
| 576 | SaveLoginPassword(kPassword); |
| 577 | krb5_->set_expected_password(kPassword); |
| 578 | |
| 579 | // Uses the login password. |
| 580 | EXPECT_EQ(ERROR_NONE, |
| 581 | manager_->AcquireTgt(kUser, kEmptyPassword, kDontRememberPassword, |
| 582 | kUseLoginPassword)); |
| 583 | |
| 584 | // Check if auth fails without kUseLoginPassword. |
| 585 | EXPECT_EQ(ERROR_BAD_PASSWORD, |
| 586 | manager_->AcquireTgt(kUser, kEmptyPassword, kDontRememberPassword, |
| 587 | kDontUseLoginPassword)); |
| 588 | } |
| 589 | |
| 590 | // AcquireTgt() wipes a saved password if the login password is used. |
| 591 | TEST_F(AccountManagerTest, AcquireTgtWipesStoredPasswordOnUsesLoginPassword) { |
| 592 | ignore_result(AddAccount()); |
| 593 | ignore_result(manager_->AcquireTgt(kUser, kPassword, kRememberPassword, |
| 594 | kDontUseLoginPassword)); |
| 595 | EXPECT_TRUE(base::PathExists(password_path_)); |
| 596 | |
| 597 | SaveLoginPassword(kPassword); |
| 598 | |
| 599 | // Note: kRememberPassword gets ignored if kUseLoginPassword is passed. |
| 600 | EXPECT_EQ(ERROR_NONE, |
| 601 | manager_->AcquireTgt(kUser, kEmptyPassword, kRememberPassword, |
| 602 | kUseLoginPassword)); |
| 603 | EXPECT_FALSE(base::PathExists(password_path_)); |
| 604 | } |
| 605 | |
| 606 | // AcquireTgt() ignores the passed password if the login password is used. |
| 607 | TEST_F(AccountManagerTest, AcquireTgtIgnoresPassedPasswordOnUsesLoginPassword) { |
| 608 | ignore_result(AddAccount()); |
| 609 | |
| 610 | SaveLoginPassword(kPassword); |
| 611 | krb5_->set_expected_password(kPassword); |
| 612 | |
| 613 | // Auth works despite passed kPassword2 != expected kPassword because the |
| 614 | // login kPassword is used. |
| 615 | EXPECT_EQ(ERROR_NONE, |
| 616 | manager_->AcquireTgt(kUser, kPassword2, kDontRememberPassword, |
| 617 | kUseLoginPassword)); |
Lutz Justen | c569050 | 2019-05-24 17:00:54 +0200 | [diff] [blame] | 618 | } |
| 619 | |
Felipe Andrade | 66aaf6b | 2020-03-24 13:05:57 +0100 | [diff] [blame] | 620 | // AcquireTgt() records all encryption types UMA stats on success. |
| 621 | TEST_F(AccountManagerTest, AcquireTgtEnctypesMetricsAll) { |
| 622 | ignore_result(AddAccount()); |
| 623 | ignore_result(SetConfig()); |
| 624 | |
| 625 | // The expected encryption type should be reported through |metric_|. |
| 626 | EXPECT_CALL(*metrics_, |
| 627 | ReportKerberosEncryptionTypes(KerberosEncryptionTypes::kAll)); |
| 628 | |
| 629 | EXPECT_EQ(ERROR_NONE, AcquireTgt()); |
| 630 | } |
| 631 | |
| 632 | // AcquireTgt() records strong encryption types UMA stats on success. |
| 633 | TEST_F(AccountManagerTest, AcquireTgtEnctypesMetricsStrong) { |
| 634 | ignore_result(AddAccount()); |
| 635 | ignore_result(SetConfig(kStrongKrb5Conf)); |
| 636 | |
| 637 | // The expected encryption type should be reported through |metric_|. |
| 638 | EXPECT_CALL(*metrics_, |
| 639 | ReportKerberosEncryptionTypes(KerberosEncryptionTypes::kStrong)); |
| 640 | |
| 641 | EXPECT_EQ(ERROR_NONE, AcquireTgt()); |
| 642 | } |
| 643 | |
| 644 | // AcquireTgt() records legacy encryption types UMA stats on success. |
| 645 | TEST_F(AccountManagerTest, AcquireTgtEnctypesMetricsLegacy) { |
| 646 | ignore_result(AddAccount()); |
| 647 | ignore_result(SetConfig(kLegacyKrb5Conf)); |
| 648 | |
| 649 | // The expected encryption type should be reported through |metric_|. |
| 650 | EXPECT_CALL(*metrics_, |
| 651 | ReportKerberosEncryptionTypes(KerberosEncryptionTypes::kLegacy)); |
| 652 | |
| 653 | EXPECT_EQ(ERROR_NONE, AcquireTgt()); |
| 654 | } |
| 655 | |
| 656 | // AcquireTgt() doesn't record encryption types UMA stats on failure. |
| 657 | TEST_F(AccountManagerTest, AcquireTgtEnctypesMetricsFailure) { |
| 658 | ignore_result(AddAccount()); |
| 659 | ignore_result(SetConfig()); |
| 660 | |
| 661 | // No encryption type should be reported through |metric_|. |
| 662 | EXPECT_CALL(*metrics_, ReportKerberosEncryptionTypes(_)).Times(0); |
| 663 | |
| 664 | krb5_->set_acquire_tgt_error(ERROR_UNKNOWN); |
| 665 | EXPECT_EQ(ERROR_UNKNOWN, AcquireTgt()); |
| 666 | } |
| 667 | |
| 668 | // AcquireTgt() doesn't record encryption types UMA stats if no config is |
| 669 | // available. |
| 670 | TEST_F(AccountManagerTest, AcquireTgtEnctypesMetricsNoConfig) { |
| 671 | ignore_result(AddAccount()); |
| 672 | |
| 673 | // No encryption type should be reported through |metric_|. |
| 674 | EXPECT_CALL(*metrics_, ReportKerberosEncryptionTypes(_)).Times(0); |
| 675 | |
| 676 | EXPECT_EQ(ERROR_NONE, AcquireTgt()); |
| 677 | } |
| 678 | |
Felipe Andrade | 90cb84e | 2020-04-07 20:28:33 +0200 | [diff] [blame] | 679 | // AcquireTgt() doesn't record encryption types UMA stats if config is invalid. |
| 680 | TEST_F(AccountManagerTest, AcquireTgtEnctypesMetricsInvalidConfig) { |
| 681 | ignore_result(AddAccount()); |
| 682 | ignore_result(SetConfig(kInvalidKrb5Conf)); |
| 683 | |
| 684 | // No encryption type should be reported through |metric_|. |
| 685 | EXPECT_CALL(*metrics_, ReportKerberosEncryptionTypes(_)).Times(0); |
| 686 | |
| 687 | EXPECT_EQ(ERROR_NONE, AcquireTgt()); |
| 688 | } |
| 689 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 690 | // RemoveAccount() removes the credential cache file. |
| 691 | TEST_F(AccountManagerTest, RemoveAccountRemovesCC) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 692 | ignore_result(AddAccount()); |
| 693 | ignore_result(AcquireTgt()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 694 | |
| 695 | EXPECT_TRUE(base::PathExists(krb5cc_path_)); |
| 696 | EXPECT_EQ(ERROR_NONE, manager_->RemoveAccount(kUser)); |
| 697 | EXPECT_FALSE(base::PathExists(krb5cc_path_)); |
| 698 | } |
| 699 | |
Lutz Justen | c569050 | 2019-05-24 17:00:54 +0200 | [diff] [blame] | 700 | // RemoveAccount() removes saved passwords. |
| 701 | TEST_F(AccountManagerTest, RemoveAccountRemovesPassword) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 702 | ignore_result(AddAccount()); |
| 703 | ignore_result(manager_->AcquireTgt(kUser, kPassword, kRememberPassword, |
| 704 | kDontUseLoginPassword)); |
Lutz Justen | c569050 | 2019-05-24 17:00:54 +0200 | [diff] [blame] | 705 | |
| 706 | EXPECT_TRUE(base::PathExists(password_path_)); |
| 707 | EXPECT_EQ(ERROR_NONE, manager_->RemoveAccount(kUser)); |
| 708 | EXPECT_FALSE(base::PathExists(password_path_)); |
| 709 | } |
| 710 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 711 | // ListAccounts() succeeds and contains the expected data. |
| 712 | TEST_F(AccountManagerTest, ListAccountsSuccess) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 713 | ignore_result(manager_->AddAccount(kUser, kManaged)); |
| 714 | ignore_result(SetConfig()); |
| 715 | ignore_result(manager_->AcquireTgt(kUser, kPassword, kRememberPassword, |
| 716 | kDontUseLoginPassword)); |
| 717 | SaveLoginPassword(kPassword); |
| 718 | ignore_result(manager_->AddAccount(kUser2, kUnmanaged)); |
| 719 | // Note: kRememberPassword should be ignored here, see below. |
| 720 | ignore_result(manager_->AcquireTgt(kUser2, kPassword, kRememberPassword, |
| 721 | kUseLoginPassword)); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 722 | EXPECT_TRUE(base::PathExists(krb5cc_path_)); |
| 723 | |
| 724 | // Set a fake tgt status. |
| 725 | constexpr int kRenewalSeconds = 10; |
| 726 | constexpr int kValiditySeconds = 90; |
Lutz Justen | d925d71 | 2019-06-27 17:34:00 +0200 | [diff] [blame] | 727 | krb5_->set_tgt_status( |
| 728 | Krb5Interface::TgtStatus(kValiditySeconds, kRenewalSeconds)); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 729 | |
| 730 | // Verify that ListAccounts returns the expected account. |
Felipe Andrade | 89399f0 | 2019-11-11 15:52:14 +0100 | [diff] [blame] | 731 | std::vector<Account> accounts = manager_->ListAccounts(); |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 732 | ASSERT_EQ(2u, accounts.size()); |
| 733 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 734 | EXPECT_EQ(kUser, accounts[0].principal_name()); |
| 735 | EXPECT_EQ(kKrb5Conf, accounts[0].krb5conf()); |
| 736 | EXPECT_EQ(kRenewalSeconds, accounts[0].tgt_renewal_seconds()); |
| 737 | EXPECT_EQ(kValiditySeconds, accounts[0].tgt_validity_seconds()); |
| 738 | EXPECT_TRUE(accounts[0].is_managed()); |
Lutz Justen | c569050 | 2019-05-24 17:00:54 +0200 | [diff] [blame] | 739 | EXPECT_TRUE(accounts[0].password_was_remembered()); |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 740 | |
| 741 | EXPECT_EQ(kUser2, accounts[1].principal_name()); |
| 742 | EXPECT_FALSE(accounts[1].password_was_remembered()); |
| 743 | EXPECT_TRUE(accounts[1].use_login_password()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 744 | } |
| 745 | |
| 746 | // ListAccounts() ignores failures in GetTgtStatus() and loading the config. |
| 747 | TEST_F(AccountManagerTest, ListAccountsIgnoresFailures) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 748 | ignore_result(AddAccount()); |
| 749 | ignore_result(SetConfig()); |
| 750 | ignore_result(AcquireTgt()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 751 | EXPECT_TRUE(base::PathExists(krb5cc_path_)); |
| 752 | |
| 753 | // Make reading the config fail. |
| 754 | EXPECT_TRUE(base::SetPosixFilePermissions(krb5conf_path_, 0)); |
| 755 | |
| 756 | // Make GetTgtStatus() fail. |
| 757 | krb5_->set_get_tgt_status_error(ERROR_UNKNOWN); |
| 758 | |
| 759 | // ListAccounts() should still work, despite the errors. |
Felipe Andrade | 89399f0 | 2019-11-11 15:52:14 +0100 | [diff] [blame] | 760 | std::vector<Account> accounts = manager_->ListAccounts(); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 761 | ASSERT_EQ(1u, accounts.size()); |
| 762 | EXPECT_EQ(kUser, accounts[0].principal_name()); |
| 763 | |
| 764 | // The config should not be set since we made reading the file fail. |
| 765 | EXPECT_FALSE(accounts[0].has_krb5conf()); |
| 766 | |
| 767 | // tgt_*_seconds should not be set since we made GetTgtStatus() fail. |
| 768 | EXPECT_FALSE(accounts[0].has_tgt_renewal_seconds()); |
| 769 | EXPECT_FALSE(accounts[0].has_tgt_validity_seconds()); |
| 770 | } |
| 771 | |
| 772 | // GetKerberosFiles returns empty KerberosFiles if there is no credential cache, |
| 773 | // even if there is a config. |
Felipe Andrade | 66aaf6b | 2020-03-24 13:05:57 +0100 | [diff] [blame] | 774 | TEST_F(AccountManagerTest, GetKerberosFilesSucceedsWithoutCC) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 775 | ignore_result(AddAccount()); |
| 776 | ignore_result(SetConfig()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 777 | |
| 778 | KerberosFiles files; |
| 779 | EXPECT_EQ(ERROR_NONE, manager_->GetKerberosFiles(kUser, &files)); |
| 780 | EXPECT_FALSE(files.has_krb5cc()); |
| 781 | EXPECT_FALSE(files.has_krb5conf()); |
| 782 | } |
| 783 | |
| 784 | // GetKerberosFiles returns the expected KerberosFiles if there is a credential |
| 785 | // cache. |
Felipe Andrade | 66aaf6b | 2020-03-24 13:05:57 +0100 | [diff] [blame] | 786 | TEST_F(AccountManagerTest, GetKerberosFilesSucceedsWithCC) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 787 | ignore_result(AddAccount()); |
| 788 | ignore_result(SetConfig()); |
| 789 | ignore_result(AcquireTgt()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 790 | |
| 791 | KerberosFiles files; |
| 792 | EXPECT_EQ(ERROR_NONE, manager_->GetKerberosFiles(kUser, &files)); |
| 793 | EXPECT_FALSE(files.krb5cc().empty()); |
| 794 | EXPECT_EQ(kKrb5Conf, files.krb5conf()); |
| 795 | } |
| 796 | |
| 797 | // Most methods return ERROR_UNKNOWN_PRINCIPAL if called with such a principal. |
| 798 | TEST_F(AccountManagerTest, MethodsReturnUnknownPrincipal) { |
| 799 | KerberosFiles files; |
| 800 | EXPECT_EQ(ERROR_UNKNOWN_PRINCIPAL_NAME, manager_->RemoveAccount(kUser)); |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 801 | EXPECT_EQ(ERROR_UNKNOWN_PRINCIPAL_NAME, SetConfig()); |
| 802 | EXPECT_EQ(ERROR_UNKNOWN_PRINCIPAL_NAME, AcquireTgt()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 803 | EXPECT_EQ(ERROR_UNKNOWN_PRINCIPAL_NAME, |
| 804 | manager_->GetKerberosFiles(kUser, &files)); |
| 805 | } |
| 806 | |
| 807 | // Accounts can be saved to disk and loaded from disk. |
| 808 | TEST_F(AccountManagerTest, SerializationSuccess) { |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 809 | ignore_result(manager_->AddAccount(kUser, kManaged)); |
| 810 | ignore_result(manager_->AcquireTgt(kUser, kPassword, kDontRememberPassword, |
| 811 | kUseLoginPassword)); |
| 812 | |
| 813 | ignore_result(manager_->AddAccount(kUser2, kUnmanaged)); |
| 814 | ignore_result(manager_->AcquireTgt(kUser2, kPassword, kDontRememberPassword, |
| 815 | kDontUseLoginPassword)); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 816 | |
| 817 | EXPECT_EQ(ERROR_NONE, manager_->SaveAccounts()); |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 818 | AccountManager other_manager( |
| 819 | storage_dir_.GetPath(), kerberos_files_changed_, |
Lutz Justen | 8eeb4a2 | 2019-06-19 17:00:32 +0200 | [diff] [blame] | 820 | kerberos_ticket_expiring_, std::make_unique<FakeKrb5Interface>(), |
Lutz Justen | 2d857e1 | 2019-07-16 15:58:09 +0200 | [diff] [blame] | 821 | std::make_unique<password_provider::FakePasswordProvider>(), |
| 822 | metrics_.get()); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 823 | other_manager.LoadAccounts(); |
Felipe Andrade | 89399f0 | 2019-11-11 15:52:14 +0100 | [diff] [blame] | 824 | std::vector<Account> accounts = other_manager.ListAccounts(); |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 825 | ASSERT_EQ(2u, accounts.size()); |
| 826 | |
| 827 | EXPECT_EQ(kUser, accounts[0].principal_name()); |
| 828 | EXPECT_EQ(kUser2, accounts[1].principal_name()); |
| 829 | |
| 830 | EXPECT_TRUE(accounts[0].is_managed()); |
| 831 | EXPECT_FALSE(accounts[1].is_managed()); |
| 832 | |
Lutz Justen | 04afd27 | 2019-06-14 16:31:26 +0200 | [diff] [blame] | 833 | EXPECT_TRUE(accounts[0].use_login_password()); |
| 834 | EXPECT_FALSE(accounts[1].use_login_password()); |
| 835 | |
Lutz Justen | 6f58298 | 2019-05-15 10:24:22 +0200 | [diff] [blame] | 836 | // TODO(https://crbug.com/952239): Check additional Account properties. |
| 837 | } |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 838 | |
Lutz Justen | d925d71 | 2019-06-27 17:34:00 +0200 | [diff] [blame] | 839 | // The StartObservingTickets() method triggers KerberosTicketExpiring for |
| 840 | // expired signals and starts observing valid tickets. |
| 841 | TEST_F(AccountManagerTest, StartObservingTickets) { |
| 842 | krb5_->set_tgt_status(kValidTgt); |
Lutz Justen | 8eeb4a2 | 2019-06-19 17:00:32 +0200 | [diff] [blame] | 843 | ignore_result(AddAccount()); |
| 844 | ignore_result(SetConfig()); |
| 845 | ignore_result(AcquireTgt()); |
| 846 | EXPECT_EQ(0, kerberos_ticket_expiring_count_[kUser]); |
Lutz Justen | d925d71 | 2019-06-27 17:34:00 +0200 | [diff] [blame] | 847 | task_runner_->ClearPendingTasks(); |
Lutz Justen | 8eeb4a2 | 2019-06-19 17:00:32 +0200 | [diff] [blame] | 848 | |
Lutz Justen | d925d71 | 2019-06-27 17:34:00 +0200 | [diff] [blame] | 849 | // Fake an expired ticket. Check that KerberosTicketExpiring is triggered, but |
| 850 | // no renewal task is scheduled. |
| 851 | krb5_->set_tgt_status(kExpiredTgt); |
| 852 | manager_->StartObservingTickets(); |
Lutz Justen | 8eeb4a2 | 2019-06-19 17:00:32 +0200 | [diff] [blame] | 853 | EXPECT_EQ(1, kerberos_ticket_expiring_count_[kUser]); |
Lutz Justen | d925d71 | 2019-06-27 17:34:00 +0200 | [diff] [blame] | 854 | EXPECT_EQ(0, task_runner_->GetPendingTaskCount()); |
Lutz Justen | 8eeb4a2 | 2019-06-19 17:00:32 +0200 | [diff] [blame] | 855 | |
Lutz Justen | d925d71 | 2019-06-27 17:34:00 +0200 | [diff] [blame] | 856 | // Fake a valid ticket. Check that KerberosTicketExpiring is NOT triggered, |
| 857 | // but a renewal task is scheduled. |
| 858 | krb5_->set_tgt_status(kValidTgt); |
| 859 | EXPECT_EQ(0, task_runner_->GetPendingTaskCount()); |
| 860 | manager_->StartObservingTickets(); |
| 861 | EXPECT_EQ(1, task_runner_->GetPendingTaskCount()); |
Lutz Justen | 8eeb4a2 | 2019-06-19 17:00:32 +0200 | [diff] [blame] | 862 | EXPECT_EQ(1, kerberos_ticket_expiring_count_[kUser]); |
Lutz Justen | d925d71 | 2019-06-27 17:34:00 +0200 | [diff] [blame] | 863 | EXPECT_EQ(0, krb5_->renew_tgt_call_count()); |
| 864 | task_runner_->FastForwardBy(task_runner_->NextPendingTaskDelay()); |
| 865 | EXPECT_EQ(1, krb5_->renew_tgt_call_count()); |
| 866 | } |
| 867 | |
| 868 | // When a TGT is acquired successfully, automatic renewal is scheduled. |
| 869 | TEST_F(AccountManagerTest, AcquireTgtSchedulesRenewalOnSuccess) { |
| 870 | ignore_result(AddAccount()); |
| 871 | |
| 872 | krb5_->set_tgt_status(kValidTgt); |
| 873 | EXPECT_EQ(0, task_runner_->GetPendingTaskCount()); |
| 874 | EXPECT_EQ(ERROR_NONE, AcquireTgt()); |
| 875 | EXPECT_EQ(1, task_runner_->GetPendingTaskCount()); |
| 876 | } |
| 877 | |
| 878 | // When a TGT fails to be acquired, no automatic renewal is scheduled. |
| 879 | TEST_F(AccountManagerTest, AcquireTgtDoesNotScheduleRenewalOnFailure) { |
| 880 | ignore_result(AddAccount()); |
| 881 | |
| 882 | krb5_->set_tgt_status(kValidTgt); |
| 883 | krb5_->set_acquire_tgt_error(ERROR_UNKNOWN); |
| 884 | EXPECT_EQ(0, task_runner_->GetPendingTaskCount()); |
| 885 | EXPECT_EQ(ERROR_UNKNOWN, AcquireTgt()); |
| 886 | EXPECT_EQ(0, task_runner_->GetPendingTaskCount()); |
| 887 | } |
| 888 | |
| 889 | // A scheduled TGT renewal task calls |krb5_->RenewTgt()|. |
| 890 | TEST_F(AccountManagerTest, AutoRenewalCallsRenewTgt) { |
| 891 | krb5_->set_tgt_status(kValidTgt); |
| 892 | ignore_result(AddAccount()); |
| 893 | ignore_result(AcquireTgt()); |
| 894 | int initial_acquire_tgt_call_count = krb5_->acquire_tgt_call_count(); |
| 895 | |
| 896 | // Set some return value for the RenewTgt() call and fast forward to scheduled |
| 897 | // renewal task. |
| 898 | const ErrorType expected_error = ERROR_UNKNOWN; |
| 899 | krb5_->set_renew_tgt_error(expected_error); |
| 900 | RunScheduledRenewalTask(); |
| 901 | |
| 902 | EXPECT_EQ(initial_acquire_tgt_call_count, krb5_->acquire_tgt_call_count()); |
| 903 | EXPECT_EQ(expected_error, manager_->last_renew_tgt_error_for_testing()); |
| 904 | } |
| 905 | |
| 906 | // A scheduled TGT renewal task calls |krb5_->AcquireTgt()| using the login |
| 907 | // password if the call to |krb5_->RenewTgt()| fails and the login password was |
| 908 | // used for the initial AcquireTgt() call. |
| 909 | TEST_F(AccountManagerTest, AutoRenewalUsesLoginPasswordIfRenewalFails) { |
| 910 | krb5_->set_tgt_status(kValidTgt); |
| 911 | ignore_result(AddAccount()); |
| 912 | |
| 913 | // Acquire TGT with login password. |
| 914 | SaveLoginPassword(kPassword); |
| 915 | krb5_->set_expected_password(kPassword); |
| 916 | EXPECT_EQ(ERROR_NONE, |
| 917 | manager_->AcquireTgt(kUser, std::string(), kDontRememberPassword, |
| 918 | kUseLoginPassword)); |
| 919 | int initial_acquire_tgt_call_count = krb5_->acquire_tgt_call_count(); |
| 920 | |
| 921 | krb5_->set_renew_tgt_error(ERROR_UNKNOWN); |
| 922 | RunScheduledRenewalTask(); |
| 923 | |
| 924 | // The scheduled renewal task should have called AcquireTgt() with the login |
| 925 | // password and succeeded. |
| 926 | EXPECT_EQ(initial_acquire_tgt_call_count + 1, |
| 927 | krb5_->acquire_tgt_call_count()); |
| 928 | EXPECT_EQ(ERROR_NONE, manager_->last_renew_tgt_error_for_testing()); |
| 929 | } |
| 930 | |
| 931 | // A scheduled TGT renewal task calls |krb5_->AcquireTgt()| using the remembered |
| 932 | // password if the call to |krb5_->RenewTgt()| fails and the password was |
| 933 | // remembered for the initial AcquireTgt() call. |
| 934 | TEST_F(AccountManagerTest, AutoRenewalUsesRememberedPasswordIfRenewalFails) { |
| 935 | krb5_->set_tgt_status(kValidTgt); |
| 936 | ignore_result(AddAccount()); |
| 937 | |
| 938 | // Acquire TGT and remember password. |
| 939 | krb5_->set_expected_password(kPassword); |
| 940 | EXPECT_EQ(ERROR_NONE, |
| 941 | manager_->AcquireTgt(kUser, kPassword, kRememberPassword, |
| 942 | kDontUseLoginPassword)); |
| 943 | int initial_acquire_tgt_call_count = krb5_->acquire_tgt_call_count(); |
| 944 | |
| 945 | krb5_->set_renew_tgt_error(ERROR_UNKNOWN); |
| 946 | RunScheduledRenewalTask(); |
| 947 | |
| 948 | // The scheduled renewal task should have called AcquireTgt() with the |
| 949 | // remembered password and succeeded. |
| 950 | EXPECT_EQ(initial_acquire_tgt_call_count + 1, |
| 951 | krb5_->acquire_tgt_call_count()); |
| 952 | EXPECT_EQ(ERROR_NONE, manager_->last_renew_tgt_error_for_testing()); |
| 953 | } |
| 954 | |
| 955 | // A scheduled TGT renewal task does not call |krb5_->AcquireTgt()| using the |
| 956 | // remembered password if the call to |krb5_->RenewTgt()| succeeds and the |
| 957 | // password was remembered for the initial AcquireTgt() call (similar for login |
| 958 | // password, but we don't test that). |
| 959 | TEST_F(AccountManagerTest, AutoRenewalDoesNotCallAcquireTgtIfRenewalSucceeds) { |
| 960 | krb5_->set_tgt_status(kValidTgt); |
| 961 | ignore_result(AddAccount()); |
| 962 | |
| 963 | // Acquire TGT and remember password. |
| 964 | krb5_->set_expected_password(kPassword); |
| 965 | EXPECT_EQ(ERROR_NONE, |
| 966 | manager_->AcquireTgt(kUser, kPassword, kRememberPassword, |
| 967 | kDontUseLoginPassword)); |
| 968 | int initial_acquire_tgt_call_count = krb5_->acquire_tgt_call_count(); |
| 969 | |
| 970 | krb5_->set_renew_tgt_error(ERROR_NONE); |
| 971 | RunScheduledRenewalTask(); |
| 972 | |
| 973 | // The scheduled renewal task should NOT have called AcquireTgt() again since |
| 974 | // |krb5_->RenewTgt()|. |
| 975 | EXPECT_EQ(initial_acquire_tgt_call_count, krb5_->acquire_tgt_call_count()); |
| 976 | EXPECT_EQ(ERROR_NONE, manager_->last_renew_tgt_error_for_testing()); |
Lutz Justen | 8eeb4a2 | 2019-06-19 17:00:32 +0200 | [diff] [blame] | 977 | } |
| 978 | |
Lutz Justen | 3360180 | 2019-07-03 22:40:57 +0200 | [diff] [blame] | 979 | // Verifies that all files written have the expected access permissions. |
| 980 | // Unfortunately, file ownership can't be tested as the test won't run as |
| 981 | // kerberosd user nor can it switch to it. |
| 982 | TEST_F(AccountManagerTest, FilePermissions) { |
| 983 | constexpr int kFileMode_rw = |
| 984 | base::FILE_PERMISSION_READ_BY_USER | base::FILE_PERMISSION_WRITE_BY_USER; |
| 985 | constexpr int kFileMode_rw_r = |
| 986 | kFileMode_rw | base::FILE_PERMISSION_READ_BY_GROUP; |
| 987 | constexpr int kFileMode_rw_r__r = |
| 988 | kFileMode_rw_r | base::FILE_PERMISSION_READ_BY_OTHERS; |
| 989 | constexpr int kFileMode_rwxrwx = |
| 990 | base::FILE_PERMISSION_USER_MASK | base::FILE_PERMISSION_GROUP_MASK; |
| 991 | |
| 992 | // Wrap the fake krb5 in a jail wrapper to get the file permissions of krb5cc |
| 993 | // right. Note that we can't use a Krb5JailWrapper for the whole test since |
| 994 | // that would break the counters in FakeKrb5Interface (they would be inc'ed in |
| 995 | // another process!). |
| 996 | manager_->WrapKrb5ForTesting(); |
| 997 | |
| 998 | // Can't set user in this test. |
| 999 | Krb5JailWrapper::DisableChangeUserForTesting(true); |
| 1000 | |
| 1001 | EXPECT_EQ(ERROR_NONE, AddAccount()); |
| 1002 | EXPECT_EQ(ERROR_NONE, SetConfig()); |
| 1003 | EXPECT_EQ(ERROR_NONE, |
| 1004 | manager_->AcquireTgt(kUser, kPassword, kRememberPassword, |
| 1005 | kDontUseLoginPassword)); |
| 1006 | |
| 1007 | int mode; |
| 1008 | |
| 1009 | EXPECT_TRUE(GetPosixFilePermissions(accounts_path_, &mode)); |
| 1010 | EXPECT_EQ(kFileMode_rw, mode); |
| 1011 | |
| 1012 | EXPECT_TRUE(GetPosixFilePermissions(account_dir_, &mode)); |
| 1013 | EXPECT_EQ(kFileMode_rwxrwx, mode); |
| 1014 | |
| 1015 | EXPECT_TRUE(GetPosixFilePermissions(krb5cc_path_, &mode)); |
| 1016 | EXPECT_EQ(kFileMode_rw_r, mode); |
| 1017 | |
| 1018 | EXPECT_TRUE(GetPosixFilePermissions(krb5conf_path_, &mode)); |
| 1019 | EXPECT_EQ(kFileMode_rw_r__r, mode); |
| 1020 | |
| 1021 | EXPECT_TRUE(GetPosixFilePermissions(password_path_, &mode)); |
| 1022 | EXPECT_EQ(kFileMode_rw, mode); |
| 1023 | } |
| 1024 | |
Lutz Justen | 2d857e1 | 2019-07-16 15:58:09 +0200 | [diff] [blame] | 1025 | // Tests that [Should]ReportDailyUsageStats is called as advertised. |
| 1026 | TEST_F(AccountManagerTest, ReportDailyUsageStats) { |
| 1027 | // ShouldReportDailyUsageStats() should be called by GetKerberosFiles() even |
| 1028 | // if there is no account, and if that returns true, ReportDailyUsageStats() |
| 1029 | // should be called as well. |
| 1030 | EXPECT_CALL(*metrics_, ShouldReportDailyUsageStats()).WillOnce(Return(true)); |
| 1031 | EXPECT_CALL(*metrics_, ReportDailyUsageStats(0, 0, 0, 0, 0)); |
| 1032 | KerberosFiles files; |
| 1033 | EXPECT_EQ(ERROR_UNKNOWN_PRINCIPAL_NAME, |
| 1034 | manager_->GetKerberosFiles(kUser, &files)); |
| 1035 | Mock::VerifyAndClearExpectations(metrics_.get()); |
| 1036 | |
| 1037 | AddAccount(); |
| 1038 | |
| 1039 | // ShouldReportDailyUsageStats() should be called by AcquireTgt(), but if that |
| 1040 | // returns false, ReportDailyUsageStats() should NOT be called. |
| 1041 | EXPECT_CALL(*metrics_, ShouldReportDailyUsageStats()).WillOnce(Return(false)); |
| 1042 | EXPECT_CALL(*metrics_, ReportDailyUsageStats(_, _, _, _, _)).Times(0); |
| 1043 | AcquireTgt(); |
| 1044 | Mock::VerifyAndClearExpectations(metrics_.get()); |
| 1045 | |
| 1046 | // ShouldReportDailyUsageStats() should be called by AcquireTgt(), and if that |
| 1047 | // returns true, ReportDailyUsageStats() should be called as well. |
| 1048 | EXPECT_CALL(*metrics_, ShouldReportDailyUsageStats()).WillOnce(Return(true)); |
| 1049 | EXPECT_CALL(*metrics_, ReportDailyUsageStats(_, _, _, _, _)); |
| 1050 | AcquireTgt(); |
| 1051 | Mock::VerifyAndClearExpectations(metrics_.get()); |
| 1052 | } |
| 1053 | |
| 1054 | TEST_F(AccountManagerTest, AccountStats) { |
| 1055 | SaveLoginPassword(kPassword); |
| 1056 | |
| 1057 | EXPECT_EQ(ERROR_NONE, manager_->AddAccount(kUser, kManaged)); |
| 1058 | EXPECT_EQ(ERROR_NONE, manager_->AddAccount(kUser2, kManaged)); |
| 1059 | EXPECT_EQ(ERROR_NONE, manager_->AddAccount(kUser3, kUnmanaged)); |
| 1060 | |
| 1061 | // Set metrics up so that the stats are reported on the last call. |
| 1062 | EXPECT_CALL(*metrics_, ShouldReportDailyUsageStats()) |
| 1063 | .WillOnce(Return(false)) |
| 1064 | .WillOnce(Return(false)) |
| 1065 | .WillOnce(Return(true)); |
| 1066 | |
| 1067 | // 3 accounts total, 2 managed, 1 unmanaged, 1 remembering the password, 1 |
| 1068 | // using the login password. |
| 1069 | EXPECT_CALL(*metrics_, ReportDailyUsageStats(3, 2, 1, 1, 1)); |
| 1070 | |
| 1071 | EXPECT_EQ(ERROR_NONE, |
| 1072 | manager_->AcquireTgt(kUser, kPassword, kDontRememberPassword, |
| 1073 | kDontUseLoginPassword)); |
| 1074 | EXPECT_EQ(ERROR_NONE, |
| 1075 | manager_->AcquireTgt(kUser2, kPassword, kRememberPassword, |
| 1076 | kDontUseLoginPassword)); |
| 1077 | EXPECT_EQ(ERROR_NONE, |
| 1078 | manager_->AcquireTgt(kUser3, kEmptyPassword, kDontRememberPassword, |
| 1079 | kUseLoginPassword)); |
| 1080 | } |
| 1081 | |
Lutz Justen | b7ef580 | 2019-05-14 14:03:20 +0200 | [diff] [blame] | 1082 | } // namespace kerberos |