Merge "Unit tests for GenericMultiProfilePagerAdapter" into main

1 files changed, 276 insertions, 0 deletions

diff --git a/java/tests/src/com/android/intentresolver/GenericMultiProfilePagerAdapterTest.kt b/java/tests/src/com/android/intentresolver/GenericMultiProfilePagerAdapterTest.kt
new file mode 100644
index 00000000..d8e1e4b9
--- /dev/null
+++ b/java/tests/src/com/android/intentresolver/GenericMultiProfilePagerAdapterTest.kt
@@ -0,0 +1,276 @@
+/*
+ * Copyright (C) 2023 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.android.intentresolver
+
+import android.os.UserHandle
+import android.view.View
+import android.widget.ListView
+import androidx.test.platform.app.InstrumentationRegistry
+import com.android.intentresolver.AbstractMultiProfilePagerAdapter.PROFILE_PERSONAL
+import com.android.intentresolver.AbstractMultiProfilePagerAdapter.PROFILE_WORK
+import com.google.common.collect.ImmutableList
+import com.google.common.truth.Truth.assertThat
+import java.util.Optional
+import org.junit.Test
+import org.mockito.Mockito.never
+import org.mockito.Mockito.verify
+
+class GenericMultiProfilePagerAdapterTest {
+    private val PERSONAL_USER_HANDLE = UserHandle.of(10)
+    private val WORK_USER_HANDLE = UserHandle.of(20)
+
+    private val context = InstrumentationRegistry.getInstrumentation().getContext()
+
+    @Test
+    fun testSinglePageProfileAdapter() {
+        val personalListAdapter =
+            mock<ResolverListAdapter> { whenever(getUserHandle()).thenReturn(PERSONAL_USER_HANDLE) }
+        val pagerAdapter =
+            GenericMultiProfilePagerAdapter(
+                context,
+                { listAdapter: ResolverListAdapter -> listAdapter },
+                { listView: ListView, bindAdapter: ResolverListAdapter ->
+                    listView.setAdapter(bindAdapter)
+                },
+                ImmutableList.of(personalListAdapter),
+                object : AbstractMultiProfilePagerAdapter.EmptyStateProvider {},
+                { false },
+                PROFILE_PERSONAL,
+                null,
+                null,
+                { ListView(context) },
+                { Optional.empty() }
+            )
+        assertThat(pagerAdapter.count).isEqualTo(1)
+        assertThat(pagerAdapter.currentPage).isEqualTo(PROFILE_PERSONAL)
+        assertThat(pagerAdapter.currentUserHandle).isEqualTo(PERSONAL_USER_HANDLE)
+        assertThat(pagerAdapter.getAdapterForIndex(0)).isSameInstanceAs(personalListAdapter)
+        assertThat(pagerAdapter.activeListAdapter).isSameInstanceAs(personalListAdapter)
+        assertThat(pagerAdapter.inactiveListAdapter).isNull()
+        assertThat(pagerAdapter.personalListAdapter).isSameInstanceAs(personalListAdapter)
+        assertThat(pagerAdapter.workListAdapter).isNull()
+        assertThat(pagerAdapter.itemCount).isEqualTo(1)
+        // TODO: consider covering some of the package-private methods (and making them public?).
+        // TODO: consider exercising responsibilities as an implementation of a ViewPager adapter.
+    }
+
+    @Test
+    fun testTwoProfilePagerAdapter() {
+        val personalListAdapter =
+            mock<ResolverListAdapter> { whenever(getUserHandle()).thenReturn(PERSONAL_USER_HANDLE) }
+        val workListAdapter =
+            mock<ResolverListAdapter> { whenever(getUserHandle()).thenReturn(WORK_USER_HANDLE) }
+        val pagerAdapter =
+            GenericMultiProfilePagerAdapter(
+                context,
+                { listAdapter: ResolverListAdapter -> listAdapter },
+                { listView: ListView, bindAdapter: ResolverListAdapter ->
+                    listView.setAdapter(bindAdapter)
+                },
+                ImmutableList.of(personalListAdapter, workListAdapter),
+                object : AbstractMultiProfilePagerAdapter.EmptyStateProvider {},
+                { false },
+                PROFILE_PERSONAL,
+                WORK_USER_HANDLE, // TODO: why does this test pass even if this is null?
+                null,
+                { ListView(context) },
+                { Optional.empty() }
+            )
+        assertThat(pagerAdapter.count).isEqualTo(2)
+        assertThat(pagerAdapter.currentPage).isEqualTo(PROFILE_PERSONAL)
+        assertThat(pagerAdapter.currentUserHandle).isEqualTo(PERSONAL_USER_HANDLE)
+        assertThat(pagerAdapter.getAdapterForIndex(0)).isSameInstanceAs(personalListAdapter)
+        assertThat(pagerAdapter.getAdapterForIndex(1)).isSameInstanceAs(workListAdapter)
+        assertThat(pagerAdapter.activeListAdapter).isSameInstanceAs(personalListAdapter)
+        assertThat(pagerAdapter.inactiveListAdapter).isSameInstanceAs(workListAdapter)
+        assertThat(pagerAdapter.personalListAdapter).isSameInstanceAs(personalListAdapter)
+        assertThat(pagerAdapter.workListAdapter).isSameInstanceAs(workListAdapter)
+        assertThat(pagerAdapter.itemCount).isEqualTo(2)
+        // TODO: consider covering some of the package-private methods (and making them public?).
+        // TODO: consider exercising responsibilities as an implementation of a ViewPager adapter;
+        // especially matching profiles to ListViews?
+        // TODO: test ProfileSelectedListener (and getters for "current" state) as the selected
+        // page changes. Currently there's no API to change the selected page directly; that's
+        // only possible through manipulation of the bound ViewPager.
+    }
+
+    @Test
+    fun testTwoProfilePagerAdapter_workIsDefault() {
+        val personalListAdapter =
+            mock<ResolverListAdapter> { whenever(getUserHandle()).thenReturn(PERSONAL_USER_HANDLE) }
+        val workListAdapter =
+            mock<ResolverListAdapter> { whenever(getUserHandle()).thenReturn(WORK_USER_HANDLE) }
+        val pagerAdapter =
+            GenericMultiProfilePagerAdapter(
+                context,
+                { listAdapter: ResolverListAdapter -> listAdapter },
+                { listView: ListView, bindAdapter: ResolverListAdapter ->
+                    listView.setAdapter(bindAdapter)
+                },
+                ImmutableList.of(personalListAdapter, workListAdapter),
+                object : AbstractMultiProfilePagerAdapter.EmptyStateProvider {},
+                { false },
+                PROFILE_WORK, // <-- This test specifically requests we start on work profile.
+                WORK_USER_HANDLE, // TODO: why does this test pass even if this is null?
+                null,
+                { ListView(context) },
+                { Optional.empty() }
+            )
+        assertThat(pagerAdapter.count).isEqualTo(2)
+        assertThat(pagerAdapter.currentPage).isEqualTo(PROFILE_WORK)
+        assertThat(pagerAdapter.currentUserHandle).isEqualTo(WORK_USER_HANDLE)
+        assertThat(pagerAdapter.getAdapterForIndex(0)).isSameInstanceAs(personalListAdapter)
+        assertThat(pagerAdapter.getAdapterForIndex(1)).isSameInstanceAs(workListAdapter)
+        assertThat(pagerAdapter.activeListAdapter).isSameInstanceAs(workListAdapter)
+        assertThat(pagerAdapter.inactiveListAdapter).isSameInstanceAs(personalListAdapter)
+        assertThat(pagerAdapter.personalListAdapter).isSameInstanceAs(personalListAdapter)
+        assertThat(pagerAdapter.workListAdapter).isSameInstanceAs(workListAdapter)
+        assertThat(pagerAdapter.itemCount).isEqualTo(2)
+        // TODO: consider covering some of the package-private methods (and making them public?).
+        // TODO: test ProfileSelectedListener (and getters for "current" state) as the selected
+        // page changes. Currently there's no API to change the selected page directly; that's
+        // only possible through manipulation of the bound ViewPager.
+    }
+
+    @Test
+    fun testBottomPaddingDelegate_default() {
+        val container =
+            mock<View> {
+                whenever(getPaddingLeft()).thenReturn(1)
+                whenever(getPaddingTop()).thenReturn(2)
+                whenever(getPaddingRight()).thenReturn(3)
+                whenever(getPaddingBottom()).thenReturn(4)
+            }
+        val pagerAdapter =
+            GenericMultiProfilePagerAdapter(
+                context,
+                { listAdapter: ResolverListAdapter -> listAdapter },
+                { listView: ListView, bindAdapter: ResolverListAdapter ->
+                    listView.setAdapter(bindAdapter)
+                },
+                ImmutableList.of(),
+                object : AbstractMultiProfilePagerAdapter.EmptyStateProvider {},
+                { false },
+                PROFILE_PERSONAL,
+                null,
+                null,
+                { ListView(context) },
+                { Optional.empty() }
+            )
+        pagerAdapter.setupContainerPadding(container)
+        verify(container, never()).setPadding(any(), any(), any(), any())
+    }
+
+    @Test
+    fun testBottomPaddingDelegate_override() {
+        val container =
+            mock<View> {
+                whenever(getPaddingLeft()).thenReturn(1)
+                whenever(getPaddingTop()).thenReturn(2)
+                whenever(getPaddingRight()).thenReturn(3)
+                whenever(getPaddingBottom()).thenReturn(4)
+            }
+        val pagerAdapter =
+            GenericMultiProfilePagerAdapter(
+                context,
+                { listAdapter: ResolverListAdapter -> listAdapter },
+                { listView: ListView, bindAdapter: ResolverListAdapter ->
+                    listView.setAdapter(bindAdapter)
+                },
+                ImmutableList.of(),
+                object : AbstractMultiProfilePagerAdapter.EmptyStateProvider {},
+                { false },
+                PROFILE_PERSONAL,
+                null,
+                null,
+                { ListView(context) },
+                { Optional.of(42) }
+            )
+        pagerAdapter.setupContainerPadding(container)
+        verify(container).setPadding(1, 2, 3, 42)
+    }
+
+    @Test
+    fun testPresumedQuietModeEmptyStateForWorkProfile_whenQuiet() {
+        // TODO: this is "presumed" because the conditions to determine whether we "should" show an
+        // empty state aren't enforced to align with the conditions when we actually *would* -- I
+        // believe `shouldShowEmptyStateScreen` should be implemented in terms of the provider?
+        val personalListAdapter =
+            mock<ResolverListAdapter> {
+                whenever(getUserHandle()).thenReturn(PERSONAL_USER_HANDLE)
+                whenever(getUnfilteredCount()).thenReturn(1)
+            }
+        val workListAdapter =
+            mock<ResolverListAdapter> {
+                whenever(getUserHandle()).thenReturn(WORK_USER_HANDLE)
+                whenever(getUnfilteredCount()).thenReturn(1)
+            }
+        val pagerAdapter =
+            GenericMultiProfilePagerAdapter(
+                context,
+                { listAdapter: ResolverListAdapter -> listAdapter },
+                { listView: ListView, bindAdapter: ResolverListAdapter ->
+                    listView.setAdapter(bindAdapter)
+                },
+                ImmutableList.of(personalListAdapter, workListAdapter),
+                object : AbstractMultiProfilePagerAdapter.EmptyStateProvider {},
+                { true }, // <-- Work mode is quiet.
+                PROFILE_WORK,
+                WORK_USER_HANDLE,
+                null,
+                { ListView(context) },
+                { Optional.empty() }
+            )
+        assertThat(pagerAdapter.shouldShowEmptyStateScreen(workListAdapter)).isTrue()
+        assertThat(pagerAdapter.shouldShowEmptyStateScreen(personalListAdapter)).isFalse()
+    }
+
+    @Test
+    fun testPresumedQuietModeEmptyStateForWorkProfile_notWhenNotQuiet() {
+        // TODO: this is "presumed" because the conditions to determine whether we "should" show an
+        // empty state aren't enforced to align with the conditions when we actually *would* -- I
+        // believe `shouldShowEmptyStateScreen` should be implemented in terms of the provider?
+        val personalListAdapter =
+            mock<ResolverListAdapter> {
+                whenever(getUserHandle()).thenReturn(PERSONAL_USER_HANDLE)
+                whenever(getUnfilteredCount()).thenReturn(1)
+            }
+        val workListAdapter =
+            mock<ResolverListAdapter> {
+                whenever(getUserHandle()).thenReturn(WORK_USER_HANDLE)
+                whenever(getUnfilteredCount()).thenReturn(1)
+            }
+        val pagerAdapter =
+            GenericMultiProfilePagerAdapter(
+                context,
+                { listAdapter: ResolverListAdapter -> listAdapter },
+                { listView: ListView, bindAdapter: ResolverListAdapter ->
+                    listView.setAdapter(bindAdapter)
+                },
+                ImmutableList.of(personalListAdapter, workListAdapter),
+                object : AbstractMultiProfilePagerAdapter.EmptyStateProvider {},
+                { false }, // <-- Work mode is not quiet.
+                PROFILE_WORK,
+                WORK_USER_HANDLE,
+                null,
+                { ListView(context) },
+                { Optional.empty() }
+            )
+        assertThat(pagerAdapter.shouldShowEmptyStateScreen(workListAdapter)).isFalse()
+        assertThat(pagerAdapter.shouldShowEmptyStateScreen(personalListAdapter)).isFalse()
+    }
+}