Remove the replaced activity checking to impact launching performance
Bug: 209081502 Test: make SettingsRoboTests Change-Id: Id8e2144a5965601be9c290bc4df6db0d0beb0917
This commit is contained in:
@@ -21,11 +21,7 @@ import static com.android.settings.core.BasePreferenceController.UNSUPPORTED_ON_
|
||||
|
||||
import static com.google.common.truth.Truth.assertThat;
|
||||
|
||||
import static org.mockito.Mockito.any;
|
||||
import static org.mockito.Mockito.doNothing;
|
||||
import static org.mockito.Mockito.mock;
|
||||
import static org.mockito.Mockito.spy;
|
||||
import static org.mockito.Mockito.times;
|
||||
import static org.mockito.Mockito.verify;
|
||||
import static org.mockito.Mockito.when;
|
||||
|
||||
@@ -37,14 +33,11 @@ import android.content.Intent;
|
||||
import androidx.preference.Preference;
|
||||
|
||||
import com.android.settings.R;
|
||||
import com.android.settings.testutils.FakeFeatureFactory;
|
||||
|
||||
import org.junit.After;
|
||||
import org.junit.Before;
|
||||
import org.junit.Ignore;
|
||||
import org.junit.Test;
|
||||
import org.junit.runner.RunWith;
|
||||
import org.mockito.ArgumentCaptor;
|
||||
import org.mockito.MockitoAnnotations;
|
||||
import org.robolectric.Robolectric;
|
||||
import org.robolectric.RobolectricTestRunner;
|
||||
@@ -53,24 +46,14 @@ import org.robolectric.annotation.Config;
|
||||
@RunWith(RobolectricTestRunner.class)
|
||||
public class TopLevelBatteryPreferenceControllerTest {
|
||||
private Context mContext;
|
||||
private FakeFeatureFactory mFeatureFactory;
|
||||
private TopLevelBatteryPreferenceController mController;
|
||||
private BatterySettingsFeatureProvider mBatterySettingsFeatureProvider;
|
||||
private ArgumentCaptor<Intent> mIntentArgumentCaptor;
|
||||
|
||||
@Before
|
||||
public void setUp() {
|
||||
MockitoAnnotations.initMocks(this);
|
||||
mFeatureFactory = FakeFeatureFactory.setupForTest();
|
||||
mContext = spy(Robolectric.setupActivity(Activity.class));
|
||||
mController = new TopLevelBatteryPreferenceController(mContext, "test_key");
|
||||
mBatterySettingsFeatureProvider =
|
||||
mFeatureFactory.batterySettingsFeatureProvider;
|
||||
}
|
||||
|
||||
@After
|
||||
public void cleanUp() {
|
||||
TopLevelBatteryPreferenceController.sReplacingActivityMap.clear();
|
||||
}
|
||||
|
||||
@Test
|
||||
@@ -84,56 +67,6 @@ public class TopLevelBatteryPreferenceControllerTest {
|
||||
assertThat(mController.getAvailabilityStatus()).isEqualTo(UNSUPPORTED_ON_DEVICE);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void handlePreferenceTreeClick_noFragment_noCustomActivityCalled() {
|
||||
Preference preference = new Preference(mContext);
|
||||
|
||||
assertThat(mController.handlePreferenceTreeClick(preference)).isFalse();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void handlePreferenceTreeClick_sameActivityReturned_noCustomActivityCalled() {
|
||||
String fragmentPath = "my.fragment.ClassName";
|
||||
Preference preference = mock(Preference.class);
|
||||
when(preference.getFragment()).thenReturn(fragmentPath);
|
||||
ComponentName pathName = mController.convertClassPathToComponentName(fragmentPath);
|
||||
when(mBatterySettingsFeatureProvider.getReplacingActivity(any())).thenReturn(pathName);
|
||||
|
||||
assertThat(mController.handlePreferenceTreeClick(preference)).isFalse();
|
||||
}
|
||||
|
||||
@Test
|
||||
@Ignore
|
||||
public void handlePreferenceTreeClick_newActivityReturned_newActivityRedirected() {
|
||||
String fragmentPath = "my.fragment.ClassName";
|
||||
Preference preference = mock(Preference.class);
|
||||
when(preference.getFragment()).thenReturn(fragmentPath);
|
||||
String newFragmentPath = "my.fragment.NewClassName";
|
||||
ComponentName newPathName = mController.convertClassPathToComponentName(newFragmentPath);
|
||||
when(mBatterySettingsFeatureProvider.getReplacingActivity(any())).thenReturn(
|
||||
newPathName);
|
||||
doNothing().when(mContext).startActivity(mIntentArgumentCaptor.capture());
|
||||
|
||||
assertThat(mIntentArgumentCaptor.getValue().getComponent()).isEqualTo(newPathName);
|
||||
assertThat(mController.handlePreferenceTreeClick(preference)).isTrue();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void handlePreferenceTreeClick_calledMultipleTimes_fetchedFromCache() {
|
||||
String fragmentPath = "my.fragment.ClassName";
|
||||
Preference preference = mock(Preference.class);
|
||||
when(preference.getFragment()).thenReturn(fragmentPath);
|
||||
String newFragmentPath = "my.fragment.NewClassName";
|
||||
ComponentName newPathName = mController.convertClassPathToComponentName(newFragmentPath);
|
||||
when(mBatterySettingsFeatureProvider.getReplacingActivity(any())).thenReturn(
|
||||
newPathName);
|
||||
doNothing().when(mContext).startActivity(any());
|
||||
|
||||
assertThat(mController.handlePreferenceTreeClick(preference)).isTrue();
|
||||
assertThat(mController.handlePreferenceTreeClick(preference)).isTrue();
|
||||
verify(mBatterySettingsFeatureProvider, times(1)).getReplacingActivity(any());
|
||||
}
|
||||
|
||||
@Test
|
||||
public void convertClassPathToComponentName_nullInput_returnsNull() {
|
||||
assertThat(mController.convertClassPathToComponentName(null)).isNull();
|
||||
|
Reference in New Issue
Block a user