diff options
author | Amin Hassani <ahassani@chromium.org> | 2019-01-14 16:29:47 -0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2019-01-16 01:46:42 -0800 |
commit | 7cc8bb085b6f04cbeca7cb77ddb6798fa9725e2f (patch) | |
tree | 55b88a1bafb293b4210bf2de59c8892c0985f8b5 /certificate_checker_unittest.cc | |
parent | 4b71743b20d0ca155df3414a863a48cb898622c9 (diff) |
update_engine: Run clang-format on ./ (root directory)
BUG=none
TEST=unittest
Change-Id: Ibd075dc7ea9a18e798f612e35725f1c83c112809
Reviewed-on: https://chromium-review.googlesource.com/1409708
Commit-Ready: Amin Hassani <ahassani@chromium.org>
Tested-by: Amin Hassani <ahassani@chromium.org>
Reviewed-by: Sen Jiang <senj@chromium.org>
Diffstat (limited to 'certificate_checker_unittest.cc')
-rw-r--r-- | certificate_checker_unittest.cc | 52 |
1 files changed, 24 insertions, 28 deletions
diff --git a/certificate_checker_unittest.cc b/certificate_checker_unittest.cc index 66b92d68..15d65552 100644 --- a/certificate_checker_unittest.cc +++ b/certificate_checker_unittest.cc @@ -27,12 +27,12 @@ #include "update_engine/common/mock_prefs.h" #include "update_engine/mock_certificate_checker.h" +using std::string; +using ::testing::_; using ::testing::DoAll; using ::testing::Return; using ::testing::SetArgPointee; using ::testing::SetArrayArgument; -using ::testing::_; -using std::string; namespace chromeos_update_engine { @@ -54,9 +54,7 @@ class CertificateCheckerTest : public testing::Test { cert_checker.SetObserver(&observer_); } - void TearDown() override { - cert_checker.SetObserver(nullptr); - } + void TearDown() override { cert_checker.SetObserver(nullptr); } MockPrefs prefs_; MockOpenSSLWrapper openssl_wrapper_; @@ -77,16 +75,15 @@ class CertificateCheckerTest : public testing::Test { // check certificate change, new TEST_F(CertificateCheckerTest, NewCertificate) { EXPECT_CALL(openssl_wrapper_, GetCertificateDigest(nullptr, _, _, _)) - .WillOnce(DoAll( - SetArgPointee<1>(depth_), - SetArgPointee<2>(length_), - SetArrayArgument<3>(digest_, digest_ + 4), - Return(true))); + .WillOnce(DoAll(SetArgPointee<1>(depth_), + SetArgPointee<2>(length_), + SetArrayArgument<3>(digest_, digest_ + 4), + Return(true))); EXPECT_CALL(prefs_, GetString(cert_key_, _)).WillOnce(Return(false)); EXPECT_CALL(prefs_, SetString(cert_key_, digest_hex_)).WillOnce(Return(true)); - EXPECT_CALL(observer_, - CertificateChecked(server_to_check_, - CertificateCheckResult::kValid)); + EXPECT_CALL( + observer_, + CertificateChecked(server_to_check_, CertificateCheckResult::kValid)); ASSERT_TRUE( cert_checker.CheckCertificateChange(1, nullptr, server_to_check_)); } @@ -94,17 +91,16 @@ TEST_F(CertificateCheckerTest, NewCertificate) { // check certificate change, unchanged TEST_F(CertificateCheckerTest, SameCertificate) { EXPECT_CALL(openssl_wrapper_, GetCertificateDigest(nullptr, _, _, _)) - .WillOnce(DoAll( - SetArgPointee<1>(depth_), - SetArgPointee<2>(length_), - SetArrayArgument<3>(digest_, digest_ + 4), - Return(true))); + .WillOnce(DoAll(SetArgPointee<1>(depth_), + SetArgPointee<2>(length_), + SetArrayArgument<3>(digest_, digest_ + 4), + Return(true))); EXPECT_CALL(prefs_, GetString(cert_key_, _)) .WillOnce(DoAll(SetArgPointee<1>(digest_hex_), Return(true))); EXPECT_CALL(prefs_, SetString(_, _)).Times(0); - EXPECT_CALL(observer_, - CertificateChecked(server_to_check_, - CertificateCheckResult::kValid)); + EXPECT_CALL( + observer_, + CertificateChecked(server_to_check_, CertificateCheckResult::kValid)); ASSERT_TRUE( cert_checker.CheckCertificateChange(1, nullptr, server_to_check_)); } @@ -112,11 +108,10 @@ TEST_F(CertificateCheckerTest, SameCertificate) { // check certificate change, changed TEST_F(CertificateCheckerTest, ChangedCertificate) { EXPECT_CALL(openssl_wrapper_, GetCertificateDigest(nullptr, _, _, _)) - .WillOnce(DoAll( - SetArgPointee<1>(depth_), - SetArgPointee<2>(length_), - SetArrayArgument<3>(digest_, digest_ + 4), - Return(true))); + .WillOnce(DoAll(SetArgPointee<1>(depth_), + SetArgPointee<2>(length_), + SetArrayArgument<3>(digest_, digest_ + 4), + Return(true))); EXPECT_CALL(prefs_, GetString(cert_key_, _)) .WillOnce(DoAll(SetArgPointee<1>(diff_digest_hex_), Return(true))); EXPECT_CALL(observer_, @@ -129,8 +124,9 @@ TEST_F(CertificateCheckerTest, ChangedCertificate) { // check certificate change, failed TEST_F(CertificateCheckerTest, FailedCertificate) { - EXPECT_CALL(observer_, CertificateChecked(server_to_check_, - CertificateCheckResult::kFailed)); + EXPECT_CALL( + observer_, + CertificateChecked(server_to_check_, CertificateCheckResult::kFailed)); EXPECT_CALL(prefs_, GetString(_, _)).Times(0); EXPECT_CALL(openssl_wrapper_, GetCertificateDigest(_, _, _, _)).Times(0); ASSERT_FALSE( |