Merge "Properly index developer options."
This commit is contained in:
committed by
Android (Google) Code Review
commit
ea63c97690
@@ -18,13 +18,16 @@ package com.android.settings.development;
|
||||
|
||||
import android.app.Activity;
|
||||
import android.content.Context;
|
||||
import android.provider.SearchIndexableResource;
|
||||
import android.support.v7.preference.Preference;
|
||||
import android.support.v7.preference.PreferenceManager;
|
||||
import android.support.v7.preference.PreferenceScreen;
|
||||
|
||||
import com.android.settings.R;
|
||||
import com.android.settings.SettingsRobolectricTestRunner;
|
||||
import com.android.settings.TestConfig;
|
||||
import com.android.settings.testutils.FakeFeatureFactory;
|
||||
import com.android.settings.testutils.shadow.SettingsShadowResources;
|
||||
import com.android.settingslib.drawer.CategoryKey;
|
||||
|
||||
import org.junit.Before;
|
||||
@@ -33,12 +36,14 @@ import org.junit.runner.RunWith;
|
||||
import org.mockito.Answers;
|
||||
import org.mockito.Mock;
|
||||
import org.mockito.MockitoAnnotations;
|
||||
import org.robolectric.RuntimeEnvironment;
|
||||
import org.robolectric.annotation.Config;
|
||||
import org.robolectric.shadows.ShadowApplication;
|
||||
|
||||
import java.util.ArrayList;
|
||||
import java.util.List;
|
||||
|
||||
import static com.google.common.truth.Truth.assertThat;
|
||||
import static org.mockito.Answers.RETURNS_DEEP_STUBS;
|
||||
import static org.mockito.Matchers.any;
|
||||
import static org.mockito.Mockito.doReturn;
|
||||
@@ -48,7 +53,11 @@ import static org.mockito.Mockito.verify;
|
||||
import static org.mockito.Mockito.when;
|
||||
|
||||
@RunWith(SettingsRobolectricTestRunner.class)
|
||||
@Config(manifest = TestConfig.MANIFEST_PATH, sdk = TestConfig.SDK_VERSION)
|
||||
@Config(manifest = TestConfig.MANIFEST_PATH, sdk = TestConfig.SDK_VERSION,
|
||||
shadows = {
|
||||
SettingsShadowResources.class,
|
||||
SettingsShadowResources.SettingsShadowTheme.class
|
||||
})
|
||||
public class DevelopmentSettingsTest {
|
||||
|
||||
@Mock(answer = Answers.RETURNS_DEEP_STUBS)
|
||||
@@ -90,4 +99,38 @@ public class DevelopmentSettingsTest {
|
||||
|
||||
verify(mScreen, times(2)).addPreference(any(Preference.class));
|
||||
}
|
||||
|
||||
@Test
|
||||
public void searchIndex_shouldIndexFromPrefXml() {
|
||||
final List<SearchIndexableResource> index =
|
||||
DevelopmentSettings.SEARCH_INDEX_DATA_PROVIDER.getXmlResourcesToIndex(
|
||||
RuntimeEnvironment.application, true);
|
||||
|
||||
assertThat(index.size()).isEqualTo(1);
|
||||
assertThat(index.get(0).xmlResId).isEqualTo(R.xml.development_prefs);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void searchIndex_pageDisabled_shouldAddAllKeysToNonIndexable() {
|
||||
final Context appContext = RuntimeEnvironment.application;
|
||||
new DevelopmentSettingsEnabler(appContext, null /* lifecycle */)
|
||||
.disableDevelopmentSettings();
|
||||
|
||||
final List<String> nonIndexableKeys =
|
||||
DevelopmentSettings.SEARCH_INDEX_DATA_PROVIDER.getNonIndexableKeys(appContext);
|
||||
|
||||
assertThat(nonIndexableKeys).contains("development_prefs_screen");
|
||||
}
|
||||
|
||||
@Test
|
||||
public void searchIndex_pageEnabled_shouldNotAddKeysToNonIndexable() {
|
||||
final Context appContext = RuntimeEnvironment.application;
|
||||
new DevelopmentSettingsEnabler(appContext, null /* lifecycle */)
|
||||
.enableDevelopmentSettings();
|
||||
|
||||
final List<String> nonIndexableKeys =
|
||||
DevelopmentSettings.SEARCH_INDEX_DATA_PROVIDER.getNonIndexableKeys(appContext);
|
||||
|
||||
assertThat(nonIndexableKeys).doesNotContain("development_prefs_screen");
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user