Hotword Audio Verification app: control the hotword settings

chrome/browser/extensions/api/hotword_private/hotword_private_apitest.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/command_line.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/extensions/api/hotword_private/hotword_private_api.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_test_message_listener.h"
@@ skipping 138 matching lines @@
 
   ExtensionTestMessageListener listenerFalse("ready", false);
   ASSERT_TRUE(RunComponentExtensionTest("setAudioLoggingEnableFalse"))
       << message_;
   EXPECT_TRUE(listenerFalse.WaitUntilSatisfied());
   EXPECT_FALSE(profile()->GetPrefs()->GetBoolean(
       prefs::kHotwordAudioLoggingEnabled));
   EXPECT_FALSE(service()->IsOptedIntoAudioLogging());
 }
 
+IN_PROC_BROWSER_TEST_F(HotwordPrivateApiTest, SetHotwordAlwaysOnSearchEnabled) {
+  EXPECT_FALSE(profile()->GetPrefs()->GetBoolean(
+      prefs::kHotwordAlwaysOnSearchEnabled));
+
+  ExtensionTestMessageListener listenerTrue("ready", false);

[rpetterson, 2014/09/11 20:14:17]

Do you mean true here instead of false?

[kcarattini, 2014/09/12 01:02:07]

I don't think I need to wait for a reply here because I'm using a different
listener in the second case. The AudioLogging test does the same thing. Let me
know if that's incorrect.

[benwells, 2014/09/12 02:58:13]

This is correct. The boolean argument is to say whether this listener sends a
reply. You only need true if you do something like listenerTrue.reply('blah') at
some point.

Maybe it's a bit confusing as you have true in the name and false in the name
later. If you want you can reuse the same listener (as it listens for the same
thing) and call reset() on it after the first test.

[rpetterson, 2014/09/12 03:03:32]

Got it. Thanks for the clarification!

[kcarattini, 2014/09/12 05:17:56]

Done.

+  ASSERT_TRUE(RunComponentExtensionTest("setHotwordAlwaysOnSearchEnableTrue"))
+      << message_;
+  EXPECT_TRUE(listenerTrue.WaitUntilSatisfied());
+  EXPECT_TRUE(profile()->GetPrefs()->GetBoolean(
+      prefs::kHotwordAlwaysOnSearchEnabled));
+
+  ExtensionTestMessageListener listenerFalse("ready", false);
+  ASSERT_TRUE(RunComponentExtensionTest("setHotwordAlwaysOnSearchEnableFalse"))
+      << message_;
+  EXPECT_TRUE(listenerFalse.WaitUntilSatisfied());
+  EXPECT_FALSE(profile()->GetPrefs()->GetBoolean(
+      prefs::kHotwordAlwaysOnSearchEnabled));
+}
+
 IN_PROC_BROWSER_TEST_F(HotwordPrivateApiTest, GetStatus) {
   ASSERT_TRUE(RunComponentExtensionTest("getEnabled")) << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(HotwordPrivateApiTest, IsAvailableTrue) {
   service()->setServiceAvailable(true);
   ExtensionTestMessageListener listener("available: true", false);
   ASSERT_TRUE(RunComponentExtensionTest("isAvailable")) << message_;
   EXPECT_TRUE(listener.WaitUntilSatisfied());
 }
@@ skipping 67 matching lines @@
 }
 
 IN_PROC_BROWSER_TEST_F(HotwordPrivateApiTest,
     GetLaunchStateHotwordAudioHistory) {
   service()->SetHotwordAudioVerificationLaunchMode(
       HotwordService::HOTWORD_AND_AUDIO_HISTORY);
   ExtensionTestMessageListener listener("launchMode: 2", false);
   ASSERT_TRUE(RunComponentExtensionTest("getLaunchState")) << message_;
   EXPECT_TRUE(listener.WaitUntilSatisfied());
 }