Merge "Fix test failures under settings/wallpaper" into main

This commit is contained in:
Fan Wu
2024-03-22 04:13:11 +00:00
committed by Android (Google) Code Review
3 changed files with 16 additions and 17 deletions

View File

@@ -27,14 +27,15 @@ import android.os.Handler;
import androidx.annotation.NonNull; import androidx.annotation.NonNull;
import androidx.annotation.Nullable; import androidx.annotation.Nullable;
import androidx.test.core.app.ApplicationProvider;
import com.android.settings.FallbackHome; import com.android.settings.FallbackHome;
import org.junit.Before; import org.junit.Before;
import org.junit.Ignore;
import org.junit.Test; import org.junit.Test;
import org.junit.runner.RunWith; import org.junit.runner.RunWith;
import org.mockito.MockitoAnnotations; import org.mockito.junit.MockitoJUnit;
import org.mockito.junit.MockitoRule;
import org.robolectric.Robolectric; import org.robolectric.Robolectric;
import org.robolectric.RobolectricTestRunner; import org.robolectric.RobolectricTestRunner;
import org.robolectric.RuntimeEnvironment; import org.robolectric.RuntimeEnvironment;
@@ -43,6 +44,7 @@ import org.robolectric.android.controller.ActivityController;
import org.robolectric.annotation.Config; import org.robolectric.annotation.Config;
import org.robolectric.annotation.Implementation; import org.robolectric.annotation.Implementation;
import org.robolectric.annotation.Implements; import org.robolectric.annotation.Implements;
import org.robolectric.annotation.LooperMode;
import org.robolectric.shadow.api.Shadow; import org.robolectric.shadow.api.Shadow;
import org.robolectric.shadows.ShadowApplication; import org.robolectric.shadows.ShadowApplication;
@@ -50,15 +52,15 @@ import java.util.ArrayList;
import java.util.List; import java.util.List;
@RunWith(RobolectricTestRunner.class) @RunWith(RobolectricTestRunner.class)
@LooperMode(LooperMode.Mode.LEGACY)
public class FallbackHomeActivityTest { public class FallbackHomeActivityTest {
public final MockitoRule mMockitoRule = MockitoJUnit.rule();
private ActivityController<FallbackHome> mController; private ActivityController<FallbackHome> mController;
@Before @Before
public void setUp() { public void setUp() {
MockitoAnnotations.initMocks(this); final Application application = ApplicationProvider.getApplicationContext();
final Application application = RuntimeEnvironment.application;
WallpaperManager wallpaperManager = WallpaperManager.getInstance(application); WallpaperManager wallpaperManager = WallpaperManager.getInstance(application);
ShadowApplication shadowApplication = Shadows.shadowOf(application); ShadowApplication shadowApplication = Shadows.shadowOf(application);
shadowApplication.setSystemService(Context.WALLPAPER_SERVICE, wallpaperManager); shadowApplication.setSystemService(Context.WALLPAPER_SERVICE, wallpaperManager);
@@ -66,7 +68,6 @@ public class FallbackHomeActivityTest {
mController = Robolectric.buildActivity(FallbackHome.class); mController = Robolectric.buildActivity(FallbackHome.class);
} }
@Ignore("b/315124270")
@Test @Test
@Config(shadows = ShadowWallpaperManager.class) @Config(shadows = ShadowWallpaperManager.class)
public void wallpaperColorsChangedListener_ensured_removed() { public void wallpaperColorsChangedListener_ensured_removed() {
@@ -83,7 +84,7 @@ public class FallbackHomeActivityTest {
@Implements(WallpaperManager.class) @Implements(WallpaperManager.class)
public static class ShadowWallpaperManager extends public static class ShadowWallpaperManager extends
org.robolectric.shadows.ShadowWallpaperManager { org.robolectric.shadows.ShadowWallpaperManager {
private final List<OnColorsChangedListener> mListener = new ArrayList<>(); private final List<OnColorsChangedListener> mListener = new ArrayList<>();

View File

@@ -31,7 +31,6 @@ import com.google.android.setupcompat.util.WizardManagerHelper;
import org.junit.After; import org.junit.After;
import org.junit.Before; import org.junit.Before;
import org.junit.Ignore;
import org.junit.Test; import org.junit.Test;
import org.junit.runner.RunWith; import org.junit.runner.RunWith;
import org.mockito.Mock; import org.mockito.Mock;
@@ -43,10 +42,12 @@ import org.robolectric.Shadows;
import org.robolectric.annotation.Config; import org.robolectric.annotation.Config;
import org.robolectric.annotation.Implementation; import org.robolectric.annotation.Implementation;
import org.robolectric.annotation.Implements; import org.robolectric.annotation.Implements;
import org.robolectric.annotation.LooperMode;
import org.robolectric.annotation.Resetter; import org.robolectric.annotation.Resetter;
import org.robolectric.shadows.ShadowApplication; import org.robolectric.shadows.ShadowApplication;
@RunWith(RobolectricTestRunner.class) @RunWith(RobolectricTestRunner.class)
@LooperMode(LooperMode.Mode.LEGACY)
public class WallpaperSuggestionActivityTest { public class WallpaperSuggestionActivityTest {
@Mock @Mock
@@ -101,7 +102,6 @@ public class WallpaperSuggestionActivityTest {
.isTrue(); .isTrue();
} }
@Ignore("b/315124270")
@Test @Test
public void addExtras_intentFromSetupWizard_extrasHasWallpaperOnlyAndLaunchedSuw() { public void addExtras_intentFromSetupWizard_extrasHasWallpaperOnlyAndLaunchedSuw() {
WallpaperSuggestionActivity activity = WallpaperSuggestionActivity activity =
@@ -119,7 +119,6 @@ public class WallpaperSuggestionActivityTest {
.isEqualTo("app_launched_suw"); .isEqualTo("app_launched_suw");
} }
@Ignore("b/315124270")
@Test @Test
public void addExtras_intentNotFromSetupWizard_extrasHasFocusWallpaper() { public void addExtras_intentNotFromSetupWizard_extrasHasFocusWallpaper() {
WallpaperSuggestionActivity activity = Robolectric.buildActivity( WallpaperSuggestionActivity activity = Robolectric.buildActivity(
@@ -135,7 +134,7 @@ public class WallpaperSuggestionActivityTest {
@Implements(WallpaperManager.class) @Implements(WallpaperManager.class)
public static class ShadowWallpaperManager extends public static class ShadowWallpaperManager extends
org.robolectric.shadows.ShadowWallpaperManager { org.robolectric.shadows.ShadowWallpaperManager {
private static int sWallpaperId; private static int sWallpaperId;

View File

@@ -27,15 +27,18 @@ import androidx.preference.Preference;
import com.android.settings.core.BasePreferenceController; import com.android.settings.core.BasePreferenceController;
import org.junit.Before; import org.junit.Before;
import org.junit.Ignore;
import org.junit.Test; import org.junit.Test;
import org.junit.runner.RunWith; import org.junit.runner.RunWith;
import org.mockito.MockitoAnnotations; import org.mockito.junit.MockitoJUnit;
import org.mockito.junit.MockitoRule;
import org.robolectric.Robolectric; import org.robolectric.Robolectric;
import org.robolectric.RobolectricTestRunner; import org.robolectric.RobolectricTestRunner;
import org.robolectric.annotation.LooperMode;
@RunWith(RobolectricTestRunner.class) @RunWith(RobolectricTestRunner.class)
@LooperMode(LooperMode.Mode.LEGACY)
public class WallpaperTypePreferenceControllerTest { public class WallpaperTypePreferenceControllerTest {
public final MockitoRule mMockitoRule = MockitoJUnit.rule();
private Context mContext; private Context mContext;
private WallpaperTypePreferenceController mController; private WallpaperTypePreferenceController mController;
@@ -44,21 +47,18 @@ public class WallpaperTypePreferenceControllerTest {
@Before @Before
public void setUp() { public void setUp() {
MockitoAnnotations.initMocks(this);
mContext = Robolectric.setupActivity(Activity.class); mContext = Robolectric.setupActivity(Activity.class);
mController = new WallpaperTypePreferenceController(mContext, "pref_key"); mController = new WallpaperTypePreferenceController(mContext, "pref_key");
mIntent = new Intent(); mIntent = new Intent();
mPreference = new Preference(mContext); mPreference = new Preference(mContext);
} }
@Ignore("b/315124270")
@Test @Test
public void getAvailabilityStatus_byDefault_shouldBeShown() { public void getAvailabilityStatus_byDefault_shouldBeShown() {
assertThat(mController.getAvailabilityStatus()) assertThat(mController.getAvailabilityStatus())
.isEqualTo(BasePreferenceController.AVAILABLE); .isEqualTo(BasePreferenceController.AVAILABLE);
} }
@Ignore("b/315124270")
@Test @Test
public void testHandlePreferenceTreeClick_intentNull_shouldDoNothing() { public void testHandlePreferenceTreeClick_intentNull_shouldDoNothing() {
mPreference.setIntent(null); mPreference.setIntent(null);
@@ -68,7 +68,6 @@ public class WallpaperTypePreferenceControllerTest {
assertThat(handled).isFalse(); assertThat(handled).isFalse();
} }
@Ignore("b/315124270")
@Test @Test
public void testHandlePreferenceTreeClick_shouldLaunchIntent() { public void testHandlePreferenceTreeClick_shouldLaunchIntent() {
mPreference.setIntent(mIntent); mPreference.setIntent(mIntent);