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

Unified Diff: chrome/browser/chromeos/settings/install_attributes_unittest.cc

Issue 2727713003: Update FWMP in TPM (Closed)
Patch Set: Test fixed Created 3 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/settings/install_attributes_unittest.cc
diff --git a/chrome/browser/chromeos/settings/install_attributes_unittest.cc b/chrome/browser/chromeos/settings/install_attributes_unittest.cc
index 2b5e4526662660cf0c7da0d33d232f28f1925f3b..88db8b6194a28375e51433b2c944df1acdb3f1d8 100644
--- a/chrome/browser/chromeos/settings/install_attributes_unittest.cc
+++ b/chrome/browser/chromeos/settings/install_attributes_unittest.cc
@@ -32,6 +32,15 @@ void CopyLockResult(base::RunLoop* loop,
loop->Quit();
}
+void OnSetBlockDevmode(base::RunLoop* loop,
+ chromeos::DBusMethodCallStatus* out_status,
+ chromeos::DBusMethodCallStatus call_status,
+ bool result,
+ const cryptohome::BaseReply& reply) {
+ *out_status = call_status;
+ loop->Quit();
+}
+
} // namespace
static const char kTestDomain[] = "example.com";
@@ -295,4 +304,14 @@ TEST_F(InstallAttributesTest, VerifyFakeInstallAttributesCache) {
EXPECT_EQ(std::string(), install_attributes_->GetDeviceId());
}
+TEST_F(InstallAttributesTest, CheckSetBlockDevmodeInTpm) {
+ chromeos::DBusMethodCallStatus status;
+ base::RunLoop loop;
+ install_attributes_->SetBlockDevmodeInTpm(
+ true, base::Bind(&OnSetBlockDevmode, &loop, &status));
+ loop.Run();
Daniel Erat 2017/03/06 21:18:26 nit: just do: base::RunLoop().Run(); ?
igorcov 2017/03/09 12:22:57 Done.
+
+ EXPECT_EQ(chromeos::DBusMethodCallStatus::DBUS_METHOD_CALL_SUCCESS, status);
+}
+
} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698