Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a36e8077 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "fix FileNotFound exception when attempting to read bubble xml" into...

Merge "fix FileNotFound exception when attempting to read bubble xml" into rvc-dev am: 750b4c93 am: 64d88bc3

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/11758287

Change-Id: I7604b301d49e4fb75d8bc22baf73ab86852f7849
parents f39ddc4a 64d88bc3
Loading
Loading
Loading
Loading
+1 −0
Original line number Diff line number Diff line
@@ -54,6 +54,7 @@ class BubblePersistentRepository @Inject constructor(

    fun readFromDisk(): List<BubbleEntity> {
        synchronized(bubbleFile) {
            if (!bubbleFile.exists()) return emptyList()
            try { return bubbleFile.openRead().use(::readXml) } catch (e: Throwable) {
                Log.e(TAG, "Failed to open bubble file", e)
            }
+7 −0
Original line number Diff line number Diff line
@@ -20,6 +20,8 @@ import android.testing.AndroidTestingRunner
import androidx.test.filters.SmallTest
import com.android.systemui.SysuiTestCase
import junit.framework.Assert.assertEquals
import junit.framework.Assert.assertNotNull
import junit.framework.Assert.assertTrue
import org.junit.Before
import org.junit.Test
import org.junit.runner.RunWith
@@ -42,6 +44,11 @@ class BubblePersistentRepositoryTest : SysuiTestCase() {

    @Test
    fun testReadWriteOperation() {
        // Verify read before write doesn't cause FileNotFoundException
        val actual = repository.readFromDisk()
        assertNotNull(actual)
        assertTrue(actual.isEmpty())

        repository.persistsToDisk(bubbles)
        assertEquals(bubbles, repository.readFromDisk())
    }