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

Commit d9dd674e authored by Matt Pietal's avatar Matt Pietal Committed by Automerger Merge Worker
Browse files

[DO NOT MERGE] Smartspace - Don't create session ahead of init am: 1c127572 

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

Change-Id: Iee48bfc717df5fdfedf0b0f9327bb1ad274da95e
parents acbf655d 1c127572

packages/SystemUI/src/com/android/systemui/statusbar/lockscreen/LockscreenSmartspaceController.kt

+1 −2
Original line number Diff line number Diff line
@@ -24,7 +24,6 @@ import android.app.smartspace.SmartspaceTarget 
import android.content.ContentResolver

import android.content.Context

import android.content.Intent

import android.content.pm.UserInfo

import android.database.ContentObserver

import android.net.Uri

import android.os.Handler
@@ -193,7 +192,7 @@ class LockscreenSmartspaceController @Inject constructor( 
    }



    private fun connectSession() {

        if (plugin == null || session != null) {

        if (plugin == null || session != null || smartspaceViews.isEmpty()) {

            return

        }

packages/SystemUI/tests/src/com/android/systemui/statusbar/lockscreen/LockscreenSmartspaceControllerTest.kt

+14 −0
Original line number Diff line number Diff line
@@ -445,6 +445,20 @@ class LockscreenSmartspaceControllerTest : SysuiTestCase() { 
        verify(smartspaceView2).registerDataProvider(plugin)

    }



    @Test

    fun testConnectAttemptBeforeInitializationShouldNotCreateSession() {

        // GIVEN an uninitalized smartspaceView

        // WHEN the device is provisioned

        `when`(deviceProvisionedController.isDeviceProvisioned()).thenReturn(true)

        `when`(deviceProvisionedController.isCurrentUserSetup()).thenReturn(true)

        deviceProvisionedListener.onDeviceProvisionedChanged()



        // THEN no calls to createSmartspaceSession should occur

        verify(smartspaceManager, never()).createSmartspaceSession(any())

        // THEN no listeners should be registered

        verify(configurationController, never()).addCallback(any())

    }



    private fun connectSession() {

        val view = controller.buildAndConnectView(fakeParent)

        smartspaceView = view as SmartspaceView