For #12025: Set Search Shortcuts Off By Default

app/src/androidTest/java/org/mozilla/fenix/ui/SearchTest.kt

@@ -50,6 +50,12 @@ class SearchTest {
     @Test
     fun shortcutButtonTest() {
         homeScreen {
+        }.openThreeDotMenu {
+        }.openSettings {
+        }.openSearchSubMenu {
+            enableShowSearchShortcuts()
+        }.goBack {
+        }.goBack {
         }.openSearch {
             verifySearchWithText()
             clickSearchEngineButton("DuckDuckGo")
@@ -63,6 +69,12 @@ class SearchTest {
     @Test
     fun shortcutSearchEngineSettingsTest() {
         homeScreen {
+        }.openThreeDotMenu {
+        }.openSettings {
+        }.openSearchSubMenu {
+            enableShowSearchShortcuts()
+        }.goBack {
+        }.goBack {
         }.openSearch {
             scrollToSearchEngineSettings()
             clickSearchEngineSettings()

app/src/androidTest/java/org/mozilla/fenix/ui/robots/SettingsSubMenuSearchRobot.kt

@@ -36,6 +36,7 @@ class SettingsSubMenuSearchRobot {
         selectDefaultSearchEngine(searchEngineName)
 
     fun disableShowSearchSuggestions() = toggleShowSearchSuggestions()
+    fun enableShowSearchShortcuts() = toggleShowSearchShortcuts()
 
     class Transition {
         val mDevice = UiDevice.getInstance(InstrumentationRegistry.getInstrumentation())
@@ -142,5 +143,16 @@ private fun toggleShowSearchSuggestions() {
         .perform(click())
 }
 
+private fun toggleShowSearchShortcuts() {
+    onView(withId(androidx.preference.R.id.recycler_view)).perform(
+        RecyclerViewActions.scrollTo<RecyclerView.ViewHolder>(
+            hasDescendant(withText("Show search shortcuts"))
+        )
+    )
+
+    onView(withText("Show search shortcuts"))
+        .perform(click())
+}
+
 private fun goBackButton() =
     onView(CoreMatchers.allOf(withContentDescription("Navigate up")))

app/src/main/java/org/mozilla/fenix/utils/Settings.kt

@@ -309,7 +309,7 @@ class Settings private constructor(
 
     val shouldShowSearchShortcuts by booleanPreference(
         appContext.getPreferenceKey(R.string.pref_key_show_search_shortcuts),
-        default = true
+        default = false
     )
 
     val shouldUseDarkTheme by booleanPreference(

app/src/test/java/org/mozilla/fenix/search/DefaultSearchControllerTest.kt

@@ -150,6 +150,10 @@ class DefaultSearchControllerTest {
     @Test
     fun `show search shortcuts when setting enabled AND query empty`() {
         val text = ""
+        testContext.settings().preferences
+                .edit()
+                .putBoolean(testContext.getString(R.string.pref_key_show_search_shortcuts), true)
+                .apply()
 
         controller.handleTextChanged(text)
 
@@ -160,6 +164,10 @@ class DefaultSearchControllerTest {
     fun `show search shortcuts when setting enabled AND query equals url`() {
         val text = "mozilla.org"
         every { session?.url } returns "mozilla.org"
+        testContext.settings().preferences
+                .edit()
+                .putBoolean(testContext.getString(R.string.pref_key_show_search_shortcuts), true)
+                .apply()
 
         assertEquals(text, session?.url)
 

app/src/test/java/org/mozilla/fenix/utils/SettingsTest.kt

@@ -254,7 +254,7 @@ class SettingsTest {
     fun shouldShowSearchShortcuts() {
         // When just created
         // Then
-        assertTrue(settings.shouldShowSearchShortcuts)
+        assertFalse(settings.shouldShowSearchShortcuts)
     }
 
     @Test