From 6c30a940817fc3af08407cc5f19091f00a0137cb Mon Sep 17 00:00:00 2001 From: Tiger Oakes Date: Fri, 13 Sep 2019 21:29:11 -0700 Subject: [PATCH] Move settings classes to packages --- .../toolbar/BrowserToolbarController.kt | 2 +- .../fenix/settings/SettingsFragment.kt | 2 ++ .../AccountAuthErrorPreference.kt | 2 +- .../{ => account}/AccountPreference.kt | 2 +- .../{ => account}/AccountProblemFragment.kt | 2 +- .../settings/{ => account}/SignOutFragment.kt | 2 +- .../{ => account}/TurnOnSyncFragment.kt | 2 +- .../deletebrowsingdata}/DeleteAndQuit.kt | 5 +---- .../DeleteBrowsingDataController.kt | 2 +- .../DeleteBrowsingDataFragment.kt | 4 ++-- .../DeleteBrowsingDataItem.kt | 2 +- .../DeleteBrowsingDataOnQuitFragment.kt | 3 ++- .../DeleteBrowsingDataOnQuitType.kt | 2 +- .../RadioSearchEngineListPreference.kt | 2 +- .../{ => search}/SearchEngineFragment.kt | 3 ++- .../SearchEngineListPreference.kt | 2 +- ...itePermissionsDetailsExceptionsFragment.kt | 3 ++- .../SitePermissionsExceptionsFragment.kt | 2 +- .../SitePermissionsFragment.kt | 6 +++-- ...onsManageExceptionsPhoneFeatureFragment.kt | 5 ++++- ...tePermissionsManagePhoneFeatureFragment.kt | 5 ++++- .../java/org/mozilla/fenix/utils/Settings.kt | 2 +- .../layout/fragment_delete_browsing_data.xml | 12 +++++----- app/src/main/res/navigation/nav_graph.xml | 22 +++++++++---------- app/src/main/res/xml/preferences.xml | 6 ++--- .../res/xml/search_engine_preferences.xml | 2 +- .../DefaultBrowserToolbarControllerTest.kt | 4 ++-- .../AccountSettingsFragmentStoreTest.kt | 6 +---- .../AccountSettingsInteractorTest.kt | 6 +---- ...DefaultDeleteBrowsingDataControllerTest.kt | 2 +- .../deletebrowsingdata}/DeleteAndQuitTest.kt | 4 ++-- .../org/mozilla/fenix/utils/SettingsTest.kt | 2 +- 32 files changed, 65 insertions(+), 63 deletions(-) rename app/src/main/java/org/mozilla/fenix/settings/{ => account}/AccountAuthErrorPreference.kt (96%) rename app/src/main/java/org/mozilla/fenix/settings/{ => account}/AccountPreference.kt (97%) rename app/src/main/java/org/mozilla/fenix/settings/{ => account}/AccountProblemFragment.kt (98%) rename app/src/main/java/org/mozilla/fenix/settings/{ => account}/SignOutFragment.kt (98%) rename app/src/main/java/org/mozilla/fenix/settings/{ => account}/TurnOnSyncFragment.kt (98%) rename app/src/main/java/org/mozilla/fenix/{utils => settings/deletebrowsingdata}/DeleteAndQuit.kt (87%) rename app/src/main/java/org/mozilla/fenix/settings/{ => deletebrowsingdata}/DeleteBrowsingDataController.kt (98%) rename app/src/main/java/org/mozilla/fenix/settings/{ => deletebrowsingdata}/DeleteBrowsingDataFragment.kt (98%) rename app/src/main/java/org/mozilla/fenix/settings/{ => deletebrowsingdata}/DeleteBrowsingDataItem.kt (97%) rename app/src/main/java/org/mozilla/fenix/settings/{ => deletebrowsingdata}/DeleteBrowsingDataOnQuitFragment.kt (96%) rename app/src/main/java/org/mozilla/fenix/settings/{ => deletebrowsingdata}/DeleteBrowsingDataOnQuitType.kt (93%) rename app/src/main/java/org/mozilla/fenix/settings/{ => search}/RadioSearchEngineListPreference.kt (96%) rename app/src/main/java/org/mozilla/fenix/settings/{ => search}/SearchEngineFragment.kt (95%) rename app/src/main/java/org/mozilla/fenix/settings/{ => search}/SearchEngineListPreference.kt (99%) rename app/src/main/java/org/mozilla/fenix/settings/{ => sitepermissions}/SitePermissionsDetailsExceptionsFragment.kt (97%) rename app/src/main/java/org/mozilla/fenix/settings/{ => sitepermissions}/SitePermissionsExceptionsFragment.kt (99%) rename app/src/main/java/org/mozilla/fenix/settings/{ => sitepermissions}/SitePermissionsFragment.kt (92%) rename app/src/main/java/org/mozilla/fenix/settings/{ => sitepermissions}/SitePermissionsManageExceptionsPhoneFeatureFragment.kt (96%) rename app/src/main/java/org/mozilla/fenix/settings/{ => sitepermissions}/SitePermissionsManagePhoneFeatureFragment.kt (96%) rename app/src/test/java/org/mozilla/fenix/settings/{ => account}/AccountSettingsFragmentStoreTest.kt (84%) rename app/src/test/java/org/mozilla/fenix/settings/{ => account}/AccountSettingsInteractorTest.kt (84%) rename app/src/test/java/org/mozilla/fenix/settings/{ => deletebrowsingdata}/DefaultDeleteBrowsingDataControllerTest.kt (98%) rename app/src/test/java/org/mozilla/fenix/{utils => settings/deletebrowsingdata}/DeleteAndQuitTest.kt (97%) diff --git a/app/src/main/java/org/mozilla/fenix/components/toolbar/BrowserToolbarController.kt b/app/src/main/java/org/mozilla/fenix/components/toolbar/BrowserToolbarController.kt index 93c17c7fd..e4f56d684 100644 --- a/app/src/main/java/org/mozilla/fenix/components/toolbar/BrowserToolbarController.kt +++ b/app/src/main/java/org/mozilla/fenix/components/toolbar/BrowserToolbarController.kt @@ -30,7 +30,7 @@ import org.mozilla.fenix.ext.nav import org.mozilla.fenix.ext.toTab import org.mozilla.fenix.lib.Do import org.mozilla.fenix.quickactionsheet.QuickActionSheetBehavior -import org.mozilla.fenix.utils.deleteAndQuit +import org.mozilla.fenix.settings.deletebrowsingdata.deleteAndQuit /** * An interface that handles the view manipulation of the BrowserToolbar, triggered by the Interactor diff --git a/app/src/main/java/org/mozilla/fenix/settings/SettingsFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/SettingsFragment.kt index 2fe58371d..5a7737033 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/SettingsFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/SettingsFragment.kt @@ -64,6 +64,8 @@ import org.mozilla.fenix.ext.getPreferenceKey import org.mozilla.fenix.ext.metrics import org.mozilla.fenix.ext.requireComponents import org.mozilla.fenix.ext.settings +import org.mozilla.fenix.settings.account.AccountAuthErrorPreference +import org.mozilla.fenix.settings.account.AccountPreference import org.mozilla.fenix.utils.ItsNotBrokenSnack @SuppressWarnings("TooManyFunctions", "LargeClass") diff --git a/app/src/main/java/org/mozilla/fenix/settings/AccountAuthErrorPreference.kt b/app/src/main/java/org/mozilla/fenix/settings/account/AccountAuthErrorPreference.kt similarity index 96% rename from app/src/main/java/org/mozilla/fenix/settings/AccountAuthErrorPreference.kt rename to app/src/main/java/org/mozilla/fenix/settings/account/AccountAuthErrorPreference.kt index 33f298e10..d55fe2a8e 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/AccountAuthErrorPreference.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/account/AccountAuthErrorPreference.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.account import android.content.Context import android.util.AttributeSet diff --git a/app/src/main/java/org/mozilla/fenix/settings/AccountPreference.kt b/app/src/main/java/org/mozilla/fenix/settings/account/AccountPreference.kt similarity index 97% rename from app/src/main/java/org/mozilla/fenix/settings/AccountPreference.kt rename to app/src/main/java/org/mozilla/fenix/settings/account/AccountPreference.kt index 558a8e6c0..adc72c2db 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/AccountPreference.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/account/AccountPreference.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.account import android.content.Context import android.util.AttributeSet diff --git a/app/src/main/java/org/mozilla/fenix/settings/AccountProblemFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/account/AccountProblemFragment.kt similarity index 98% rename from app/src/main/java/org/mozilla/fenix/settings/AccountProblemFragment.kt rename to app/src/main/java/org/mozilla/fenix/settings/account/AccountProblemFragment.kt index b80b18433..22fdc081d 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/AccountProblemFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/account/AccountProblemFragment.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.account import android.os.Bundle import androidx.appcompat.app.AppCompatActivity diff --git a/app/src/main/java/org/mozilla/fenix/settings/SignOutFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/account/SignOutFragment.kt similarity index 98% rename from app/src/main/java/org/mozilla/fenix/settings/SignOutFragment.kt rename to app/src/main/java/org/mozilla/fenix/settings/account/SignOutFragment.kt index 02206ba3a..ae990de9a 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/SignOutFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/account/SignOutFragment.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.account import android.app.Dialog import android.os.Bundle diff --git a/app/src/main/java/org/mozilla/fenix/settings/TurnOnSyncFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/account/TurnOnSyncFragment.kt similarity index 98% rename from app/src/main/java/org/mozilla/fenix/settings/TurnOnSyncFragment.kt rename to app/src/main/java/org/mozilla/fenix/settings/account/TurnOnSyncFragment.kt index a22e552bc..99c74eb7a 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/TurnOnSyncFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/account/TurnOnSyncFragment.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.account import android.os.Bundle import android.view.LayoutInflater diff --git a/app/src/main/java/org/mozilla/fenix/utils/DeleteAndQuit.kt b/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteAndQuit.kt similarity index 87% rename from app/src/main/java/org/mozilla/fenix/utils/DeleteAndQuit.kt rename to app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteAndQuit.kt index b96021778..a6eab0d4b 100644 --- a/app/src/main/java/org/mozilla/fenix/utils/DeleteAndQuit.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteAndQuit.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.utils +package org.mozilla.fenix.settings.deletebrowsingdata import android.app.Activity import kotlinx.coroutines.CoroutineScope @@ -11,9 +11,6 @@ import kotlinx.coroutines.joinAll import kotlinx.coroutines.launch import kotlinx.coroutines.withContext import org.mozilla.fenix.ext.settings -import org.mozilla.fenix.settings.DefaultDeleteBrowsingDataController -import org.mozilla.fenix.settings.DeleteBrowsingDataController -import org.mozilla.fenix.settings.DeleteBrowsingDataOnQuitType /** * Deletes selected browsing data and finishes the activity. diff --git a/app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataController.kt b/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataController.kt similarity index 98% rename from app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataController.kt rename to app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataController.kt index b7bf2b8c7..790aa3116 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataController.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataController.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.deletebrowsingdata import android.content.Context import kotlinx.coroutines.Dispatchers diff --git a/app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataFragment.kt similarity index 98% rename from app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataFragment.kt rename to app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataFragment.kt index 2b15e6187..6f36b3d4c 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataFragment.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.deletebrowsingdata import android.content.DialogInterface import android.os.Bundle @@ -44,7 +44,7 @@ class DeleteBrowsingDataFragment : Fragment() { override fun onViewCreated(view: View, savedInstanceState: Bundle?) { super.onViewCreated(view, savedInstanceState) - controller = DefaultDeleteBrowsingDataController(context!!) + controller = DefaultDeleteBrowsingDataController(requireContext()) sessionObserver = object : SessionManager.Observer { override fun onSessionAdded(session: Session) = updateTabCount() diff --git a/app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataItem.kt b/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataItem.kt similarity index 97% rename from app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataItem.kt rename to app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataItem.kt index 716afe38f..47188ee0c 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataItem.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataItem.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.deletebrowsingdata import android.content.Context import android.util.AttributeSet diff --git a/app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataOnQuitFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataOnQuitFragment.kt similarity index 96% rename from app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataOnQuitFragment.kt rename to app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataOnQuitFragment.kt index 7d19ac190..d86159f0c 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataOnQuitFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataOnQuitFragment.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.deletebrowsingdata import android.os.Bundle import androidx.appcompat.app.AppCompatActivity @@ -13,6 +13,7 @@ import androidx.preference.SwitchPreference import org.mozilla.fenix.R import org.mozilla.fenix.ext.getPreferenceKey import org.mozilla.fenix.ext.settings +import org.mozilla.fenix.settings.SharedPreferenceUpdater class DeleteBrowsingDataOnQuitFragment : PreferenceFragmentCompat() { diff --git a/app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataOnQuitType.kt b/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataOnQuitType.kt similarity index 93% rename from app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataOnQuitType.kt rename to app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataOnQuitType.kt index a6d53fe9e..794d16a9d 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/DeleteBrowsingDataOnQuitType.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteBrowsingDataOnQuitType.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.deletebrowsingdata import android.content.Context import androidx.annotation.StringRes diff --git a/app/src/main/java/org/mozilla/fenix/settings/RadioSearchEngineListPreference.kt b/app/src/main/java/org/mozilla/fenix/settings/search/RadioSearchEngineListPreference.kt similarity index 96% rename from app/src/main/java/org/mozilla/fenix/settings/RadioSearchEngineListPreference.kt rename to app/src/main/java/org/mozilla/fenix/settings/search/RadioSearchEngineListPreference.kt index d9726cdce..9caa2f2f9 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/RadioSearchEngineListPreference.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/search/RadioSearchEngineListPreference.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.search import android.content.Context import android.util.AttributeSet diff --git a/app/src/main/java/org/mozilla/fenix/settings/SearchEngineFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/search/SearchEngineFragment.kt similarity index 95% rename from app/src/main/java/org/mozilla/fenix/settings/SearchEngineFragment.kt rename to app/src/main/java/org/mozilla/fenix/settings/search/SearchEngineFragment.kt index 7ca54b6b4..445c58272 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/SearchEngineFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/search/SearchEngineFragment.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.search import android.os.Bundle import androidx.appcompat.app.AppCompatActivity @@ -11,6 +11,7 @@ import androidx.preference.SwitchPreference import org.mozilla.fenix.R import org.mozilla.fenix.ext.getPreferenceKey import org.mozilla.fenix.ext.settings +import org.mozilla.fenix.settings.SharedPreferenceUpdater class SearchEngineFragment : PreferenceFragmentCompat() { diff --git a/app/src/main/java/org/mozilla/fenix/settings/SearchEngineListPreference.kt b/app/src/main/java/org/mozilla/fenix/settings/search/SearchEngineListPreference.kt similarity index 99% rename from app/src/main/java/org/mozilla/fenix/settings/SearchEngineListPreference.kt rename to app/src/main/java/org/mozilla/fenix/settings/search/SearchEngineListPreference.kt index 81ba56011..9d685f881 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/SearchEngineListPreference.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/search/SearchEngineListPreference.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.search import android.content.Context import android.content.res.Resources diff --git a/app/src/main/java/org/mozilla/fenix/settings/SitePermissionsDetailsExceptionsFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsDetailsExceptionsFragment.kt similarity index 97% rename from app/src/main/java/org/mozilla/fenix/settings/SitePermissionsDetailsExceptionsFragment.kt rename to app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsDetailsExceptionsFragment.kt index 3c8b5e302..e3bf5ab95 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/SitePermissionsDetailsExceptionsFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsDetailsExceptionsFragment.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.sitepermissions import android.os.Bundle import androidx.appcompat.app.AppCompatActivity @@ -20,6 +20,7 @@ import org.jetbrains.anko.yesButton import org.mozilla.fenix.R import org.mozilla.fenix.ext.components import org.mozilla.fenix.ext.getPreferenceKey +import org.mozilla.fenix.settings.PhoneFeature import org.mozilla.fenix.settings.PhoneFeature.CAMERA import org.mozilla.fenix.settings.PhoneFeature.LOCATION import org.mozilla.fenix.settings.PhoneFeature.MICROPHONE diff --git a/app/src/main/java/org/mozilla/fenix/settings/SitePermissionsExceptionsFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsExceptionsFragment.kt similarity index 99% rename from app/src/main/java/org/mozilla/fenix/settings/SitePermissionsExceptionsFragment.kt rename to app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsExceptionsFragment.kt index 0a54e8905..f00ccce7a 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/SitePermissionsExceptionsFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsExceptionsFragment.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.sitepermissions import android.os.Bundle import android.view.LayoutInflater diff --git a/app/src/main/java/org/mozilla/fenix/settings/SitePermissionsFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsFragment.kt similarity index 92% rename from app/src/main/java/org/mozilla/fenix/settings/SitePermissionsFragment.kt rename to app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsFragment.kt index f0e974a44..024c754ce 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/SitePermissionsFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsFragment.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.sitepermissions import android.os.Bundle import androidx.appcompat.app.AppCompatActivity @@ -13,6 +13,7 @@ import androidx.preference.PreferenceFragmentCompat import org.mozilla.fenix.R import org.mozilla.fenix.ext.getPreferenceKey import org.mozilla.fenix.ext.settings +import org.mozilla.fenix.settings.PhoneFeature @SuppressWarnings("TooManyFunctions") class SitePermissionsFragment : PreferenceFragmentCompat() { @@ -69,7 +70,8 @@ class SitePermissionsFragment : PreferenceFragmentCompat() { } private fun navigateToPhoneFeature(phoneFeature: PhoneFeature) { - val directions = SitePermissionsFragmentDirections.actionSitePermissionsToManagePhoneFeatures(phoneFeature.id) + val directions = SitePermissionsFragmentDirections + .actionSitePermissionsToManagePhoneFeatures(phoneFeature.id) Navigation.findNavController(view!!).navigate(directions) } } diff --git a/app/src/main/java/org/mozilla/fenix/settings/SitePermissionsManageExceptionsPhoneFeatureFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsManageExceptionsPhoneFeatureFragment.kt similarity index 96% rename from app/src/main/java/org/mozilla/fenix/settings/SitePermissionsManageExceptionsPhoneFeatureFragment.kt rename to app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsManageExceptionsPhoneFeatureFragment.kt index 8a56ab1b3..ab1810b39 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/SitePermissionsManageExceptionsPhoneFeatureFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsManageExceptionsPhoneFeatureFragment.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.sitepermissions import android.content.Intent import android.net.Uri @@ -27,6 +27,9 @@ import org.jetbrains.anko.yesButton import org.mozilla.fenix.R import org.mozilla.fenix.ext.requireComponents import org.mozilla.fenix.ext.settings +import org.mozilla.fenix.settings.PhoneFeature +import org.mozilla.fenix.settings.initBlockedByAndroidView +import org.mozilla.fenix.settings.setStartCheckedIndicator @SuppressWarnings("TooManyFunctions") class SitePermissionsManageExceptionsPhoneFeatureFragment : Fragment() { diff --git a/app/src/main/java/org/mozilla/fenix/settings/SitePermissionsManagePhoneFeatureFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsManagePhoneFeatureFragment.kt similarity index 96% rename from app/src/main/java/org/mozilla/fenix/settings/SitePermissionsManagePhoneFeatureFragment.kt rename to app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsManagePhoneFeatureFragment.kt index c25c250d8..314fe6db8 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/SitePermissionsManagePhoneFeatureFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/sitepermissions/SitePermissionsManagePhoneFeatureFragment.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.sitepermissions import android.content.Intent import android.graphics.Color @@ -27,6 +27,9 @@ import mozilla.components.feature.sitepermissions.SitePermissionsRules.Action.BL import org.mozilla.fenix.R import org.mozilla.fenix.ext.requireComponents import org.mozilla.fenix.ext.settings +import org.mozilla.fenix.settings.PhoneFeature +import org.mozilla.fenix.settings.initBlockedByAndroidView +import org.mozilla.fenix.settings.setStartCheckedIndicator import org.mozilla.fenix.utils.Settings @SuppressWarnings("TooManyFunctions") diff --git a/app/src/main/java/org/mozilla/fenix/utils/Settings.kt b/app/src/main/java/org/mozilla/fenix/utils/Settings.kt index d29430b41..026e82a09 100644 --- a/app/src/main/java/org/mozilla/fenix/utils/Settings.kt +++ b/app/src/main/java/org/mozilla/fenix/utils/Settings.kt @@ -22,7 +22,7 @@ import org.mozilla.fenix.Config import org.mozilla.fenix.R import org.mozilla.fenix.components.metrics.MozillaProductDetector import org.mozilla.fenix.ext.getPreferenceKey -import org.mozilla.fenix.settings.DeleteBrowsingDataOnQuitType +import org.mozilla.fenix.settings.deletebrowsingdata.DeleteBrowsingDataOnQuitType import org.mozilla.fenix.settings.PhoneFeature import java.security.InvalidParameterException diff --git a/app/src/main/res/layout/fragment_delete_browsing_data.xml b/app/src/main/res/layout/fragment_delete_browsing_data.xml index 780c4901d..441dbab7a 100644 --- a/app/src/main/res/layout/fragment_delete_browsing_data.xml +++ b/app/src/main/res/layout/fragment_delete_browsing_data.xml @@ -29,7 +29,7 @@ android:layout_width="match_parent" android:layout_height="wrap_content" android:orientation="vertical"> - - - - - - + android:name="org.mozilla.fenix.settings.account.AccountProblemFragment"> + android:name="org.mozilla.fenix.settings.account.SignOutFragment" /> @@ -582,7 +582,7 @@ - - @@ -120,4 +120,4 @@ android:key="@string/pref_key_about" android:title="@string/preferences_about" /> - \ No newline at end of file + diff --git a/app/src/main/res/xml/search_engine_preferences.xml b/app/src/main/res/xml/search_engine_preferences.xml index 4c3620010..c6a947ae0 100644 --- a/app/src/main/res/xml/search_engine_preferences.xml +++ b/app/src/main/res/xml/search_engine_preferences.xml @@ -9,7 +9,7 @@ android:title="@string/preferences_default_search_engine" android:selectable="false" app:iconSpaceReserved="false"> - ().toTab(any()) } returns currentSessionAsTab mockkStatic( - "org.mozilla.fenix.utils.DeleteAndQuitKt" + "org.mozilla.fenix.settings.deletebrowsingdata.DeleteAndQuitKt" ) every { deleteAndQuit(any(), any()) } just Runs diff --git a/app/src/test/java/org/mozilla/fenix/settings/AccountSettingsFragmentStoreTest.kt b/app/src/test/java/org/mozilla/fenix/settings/account/AccountSettingsFragmentStoreTest.kt similarity index 84% rename from app/src/test/java/org/mozilla/fenix/settings/AccountSettingsFragmentStoreTest.kt rename to app/src/test/java/org/mozilla/fenix/settings/account/AccountSettingsFragmentStoreTest.kt index 459230274..c751863e4 100644 --- a/app/src/test/java/org/mozilla/fenix/settings/AccountSettingsFragmentStoreTest.kt +++ b/app/src/test/java/org/mozilla/fenix/settings/account/AccountSettingsFragmentStoreTest.kt @@ -2,16 +2,12 @@ License, v. 2.0. If a copy of the MPL was not distributed with this file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.account import kotlinx.coroutines.runBlocking import org.junit.Assert.assertEquals import org.junit.Assert.assertNotSame import org.junit.Test -import org.mozilla.fenix.settings.account.AccountSettingsFragmentAction -import org.mozilla.fenix.settings.account.AccountSettingsFragmentState -import org.mozilla.fenix.settings.account.AccountSettingsFragmentStore -import org.mozilla.fenix.settings.account.LastSyncTime class AccountSettingsFragmentStoreTest { diff --git a/app/src/test/java/org/mozilla/fenix/settings/AccountSettingsInteractorTest.kt b/app/src/test/java/org/mozilla/fenix/settings/account/AccountSettingsInteractorTest.kt similarity index 84% rename from app/src/test/java/org/mozilla/fenix/settings/AccountSettingsInteractorTest.kt rename to app/src/test/java/org/mozilla/fenix/settings/account/AccountSettingsInteractorTest.kt index d41844787..f817975d9 100644 --- a/app/src/test/java/org/mozilla/fenix/settings/AccountSettingsInteractorTest.kt +++ b/app/src/test/java/org/mozilla/fenix/settings/account/AccountSettingsInteractorTest.kt @@ -2,7 +2,7 @@ License, v. 2.0. If a copy of the MPL was not distributed with this file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.account import androidx.navigation.NavController import androidx.navigation.NavDestination @@ -12,10 +12,6 @@ import io.mockk.verify import org.junit.Assert.assertEquals import org.junit.Test import org.mozilla.fenix.R -import org.mozilla.fenix.settings.account.AccountSettingsFragmentAction -import org.mozilla.fenix.settings.account.AccountSettingsFragmentDirections -import org.mozilla.fenix.settings.account.AccountSettingsInteractor -import org.mozilla.fenix.settings.account.AccountSettingsFragmentStore class AccountSettingsInteractorTest { diff --git a/app/src/test/java/org/mozilla/fenix/settings/DefaultDeleteBrowsingDataControllerTest.kt b/app/src/test/java/org/mozilla/fenix/settings/deletebrowsingdata/DefaultDeleteBrowsingDataControllerTest.kt similarity index 98% rename from app/src/test/java/org/mozilla/fenix/settings/DefaultDeleteBrowsingDataControllerTest.kt rename to app/src/test/java/org/mozilla/fenix/settings/deletebrowsingdata/DefaultDeleteBrowsingDataControllerTest.kt index 74494a221..b13a4d558 100644 --- a/app/src/test/java/org/mozilla/fenix/settings/DefaultDeleteBrowsingDataControllerTest.kt +++ b/app/src/test/java/org/mozilla/fenix/settings/deletebrowsingdata/DefaultDeleteBrowsingDataControllerTest.kt @@ -2,7 +2,7 @@ * License, v. 2.0. If a copy of the MPL was not distributed with this * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ -package org.mozilla.fenix.settings +package org.mozilla.fenix.settings.deletebrowsingdata import android.content.Context import androidx.test.ext.junit.runners.AndroidJUnit4 diff --git a/app/src/test/java/org/mozilla/fenix/utils/DeleteAndQuitTest.kt b/app/src/test/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteAndQuitTest.kt similarity index 97% rename from app/src/test/java/org/mozilla/fenix/utils/DeleteAndQuitTest.kt rename to app/src/test/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteAndQuitTest.kt index b55b8b3b8..d2e31280c 100644 --- a/app/src/test/java/org/mozilla/fenix/utils/DeleteAndQuitTest.kt +++ b/app/src/test/java/org/mozilla/fenix/settings/deletebrowsingdata/DeleteAndQuitTest.kt @@ -4,7 +4,7 @@ @file:Suppress("DEPRECATION") -package org.mozilla.fenix.utils +package org.mozilla.fenix.settings.deletebrowsingdata import androidx.test.ext.junit.runners.AndroidJUnit4 import io.mockk.every @@ -30,7 +30,7 @@ import org.mozilla.fenix.TestApplication import org.mozilla.fenix.components.PermissionStorage import org.mozilla.fenix.ext.clearAndCommit import org.mozilla.fenix.ext.components -import org.mozilla.fenix.settings.DeleteBrowsingDataOnQuitType +import org.mozilla.fenix.utils.Settings import org.robolectric.annotation.Config @ObsoleteCoroutinesApi diff --git a/app/src/test/java/org/mozilla/fenix/utils/SettingsTest.kt b/app/src/test/java/org/mozilla/fenix/utils/SettingsTest.kt index 9128542e8..cd67c1766 100644 --- a/app/src/test/java/org/mozilla/fenix/utils/SettingsTest.kt +++ b/app/src/test/java/org/mozilla/fenix/utils/SettingsTest.kt @@ -19,7 +19,7 @@ import org.junit.runner.RunWith import org.mozilla.fenix.TestApplication import org.mozilla.fenix.ext.clearAndCommit import org.mozilla.fenix.ext.settings -import org.mozilla.fenix.settings.DeleteBrowsingDataOnQuitType +import org.mozilla.fenix.settings.deletebrowsingdata.DeleteBrowsingDataOnQuitType import org.mozilla.fenix.settings.PhoneFeature import org.robolectric.annotation.Config