Merge "Stop enabling the Boot Aware Launcher prefs code during Tests after running LauncherPref unit tests." into tm-qpr-dev

This commit is contained in:
Stefan Andonian 2023-03-22 17:17:57 +00:00 committed by Android (Google) Code Review
commit acaecb6511
2 changed files with 17 additions and 3 deletions

View File

@ -36,6 +36,8 @@ import com.android.launcher3.util.Themes
* Use same context for shared preferences, so that we use a single cached instance
*
* TODO(b/262721340): Replace all direct SharedPreference refs with LauncherPrefs / Item methods.
* TODO(b/274501660): Fix ReorderWidgets#simpleReorder test before enabling
* isBootAwareStartupDataEnabled
*/
class LauncherPrefs(private val encryptedContext: Context) {
private val deviceProtectedStorageContext =
@ -228,7 +230,7 @@ class LauncherPrefs(private val encryptedContext: Context) {
* files.
*
* @return a list of editors with all transactions added so that the caller can determine to use
* .apply() or .commit()
* .apply() or .commit()
*/
private fun prepareToRemove(items: Array<out Item>): List<SharedPreferences.Editor> {
val itemsPerFile = items.groupBy { it.encryptedPrefs }.toMutableMap()

View File

@ -25,6 +25,8 @@ import com.android.launcher3.LauncherPrefs.Companion.BOOT_AWARE_PREFS_KEY
import com.google.common.truth.Truth.assertThat
import java.util.concurrent.CountDownLatch
import java.util.concurrent.TimeUnit
import org.junit.AfterClass
import org.junit.BeforeClass
import org.junit.Test
import org.junit.runner.RunWith
@ -45,8 +47,18 @@ class LauncherPrefsTest {
private val context by lazy { InstrumentationRegistry.getInstrumentation().targetContext }
private val launcherPrefs by lazy { LauncherPrefs.get(context) }
init {
isBootAwareStartupDataEnabled = true
companion object {
@BeforeClass
@JvmStatic
fun setup() {
isBootAwareStartupDataEnabled = true
}
@AfterClass
@JvmStatic
fun teardown() {
isBootAwareStartupDataEnabled = false
}
}
@Test