Merge "[Regional Preference] Need to remove the metrics by the metrics concil team's comment." into udc-qpr-dev am: 6116750110
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Settings/+/24521607 Change-Id: I3462de66a8f19b3531848ccfd6f97fb99562a593 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -19,14 +19,12 @@ package com.android.settings.localepicker;
|
||||
import static org.mockito.Mockito.anyString;
|
||||
import static org.mockito.Mockito.verify;
|
||||
|
||||
import android.app.settings.SettingsEnums;
|
||||
import android.content.Context;
|
||||
import android.os.Looper;
|
||||
|
||||
import androidx.test.core.app.ApplicationProvider;
|
||||
import androidx.test.ext.junit.runners.AndroidJUnit4;
|
||||
|
||||
import com.android.settings.testutils.FakeFeatureFactory;
|
||||
import com.android.settingslib.widget.FooterPreference;
|
||||
|
||||
import org.junit.Before;
|
||||
@@ -39,7 +37,6 @@ import org.mockito.MockitoAnnotations;
|
||||
public class LocaleHelperPreferenceControllerTest {
|
||||
private Context mContext;
|
||||
private LocaleHelperPreferenceController mLocaleHelperPreferenceController;
|
||||
private FakeFeatureFactory mFeatureFactory;
|
||||
|
||||
@Mock
|
||||
private FooterPreference mMockFooterPreference;
|
||||
@@ -52,16 +49,11 @@ public class LocaleHelperPreferenceControllerTest {
|
||||
}
|
||||
mContext = ApplicationProvider.getApplicationContext();
|
||||
mLocaleHelperPreferenceController = new LocaleHelperPreferenceController(mContext);
|
||||
mFeatureFactory = FakeFeatureFactory.setupForTest();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void updateFooterPreference_setFooterPreference_hasClickAction() {
|
||||
mLocaleHelperPreferenceController.updateFooterPreference(mMockFooterPreference);
|
||||
verify(mMockFooterPreference).setLearnMoreText(anyString());
|
||||
mMockFooterPreference.setLearnMoreAction(v -> {
|
||||
verify(mFeatureFactory.metricsFeatureProvider).action(
|
||||
mContext, SettingsEnums.ACTION_LANGUAGES_LEARN_MORE);
|
||||
});
|
||||
}
|
||||
}
|
||||
|
@@ -24,7 +24,6 @@ import static org.mockito.ArgumentMatchers.any;
|
||||
import static org.mockito.Mockito.spy;
|
||||
import static org.mockito.Mockito.verify;
|
||||
|
||||
import android.app.settings.SettingsEnums;
|
||||
import android.content.Context;
|
||||
import android.os.Bundle;
|
||||
import android.os.LocaleList;
|
||||
@@ -52,7 +51,6 @@ public class NumberingSystemItemControllerTest {
|
||||
private NumberingPreferencesFragment mFragment;
|
||||
private PreferenceScreen mPreferenceScreen;
|
||||
private LocaleList mCacheLocale;
|
||||
private FakeFeatureFactory mFeatureFactory;
|
||||
|
||||
@Before
|
||||
@UiThreadTest
|
||||
@@ -61,7 +59,6 @@ public class NumberingSystemItemControllerTest {
|
||||
Looper.prepare();
|
||||
}
|
||||
mApplicationContext = ApplicationProvider.getApplicationContext();
|
||||
mFeatureFactory = FakeFeatureFactory.setupForTest();
|
||||
mFragment = spy(new NumberingPreferencesFragment());
|
||||
PreferenceManager preferenceManager = new PreferenceManager(mApplicationContext);
|
||||
mPreferenceScreen = preferenceManager.createPreferenceScreen(mApplicationContext);
|
||||
@@ -97,10 +94,6 @@ public class NumberingSystemItemControllerTest {
|
||||
}
|
||||
|
||||
assertTrue(isCallingStartActivity);
|
||||
verify(mFeatureFactory.metricsFeatureProvider).action(
|
||||
mApplicationContext,
|
||||
SettingsEnums.ACTION_CHOOSE_LANGUAGE_FOR_NUMBERS_PREFERENCES,
|
||||
"I_am_the_key");
|
||||
}
|
||||
|
||||
@Test
|
||||
@@ -121,9 +114,6 @@ public class NumberingSystemItemControllerTest {
|
||||
mController.handlePreferenceTreeClick(preference);
|
||||
|
||||
verify(mFragment).setArguments(any());
|
||||
verify(mFeatureFactory.metricsFeatureProvider).action(
|
||||
mApplicationContext, SettingsEnums.ACTION_SET_NUMBERS_PREFERENCES,
|
||||
"test_key");
|
||||
}
|
||||
|
||||
@Test
|
||||
|
Reference in New Issue
Block a user