From 16d82ccd0a0f95cdc6c5b9d1abc59c5e178954ee Mon Sep 17 00:00:00 2001 From: marcusge Date: Wed, 28 Feb 2024 20:28:59 +0000 Subject: [PATCH] [Accessibility] Add basic metrics logging for color contrast proto was merged in ag/26383954 Bug: 241805782 Test: atest Change-Id: Ic2e6bcb27aecc386582e6f268d3b4f40387f2742 --- .../settings/accessibility/ColorContrastFragment.java | 5 +++-- .../settings/accessibility/ColorContrastFragmentTest.java | 4 +++- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/src/com/android/settings/accessibility/ColorContrastFragment.java b/src/com/android/settings/accessibility/ColorContrastFragment.java index a5726579b38..904826e47ec 100644 --- a/src/com/android/settings/accessibility/ColorContrastFragment.java +++ b/src/com/android/settings/accessibility/ColorContrastFragment.java @@ -16,6 +16,8 @@ package com.android.settings.accessibility; +import android.app.settings.SettingsEnums; + import com.android.settings.R; import com.android.settings.dashboard.DashboardFragment; import com.android.settings.search.BaseSearchIndexProvider; @@ -40,8 +42,7 @@ public class ColorContrastFragment extends DashboardFragment { @Override public int getMetricsCategory() { - // TODO(b/326539398): Add metrics tracking for color contrast. - return 0; + return SettingsEnums.ACCESSIBILITY_COLOR_CONTRAST; } public static final BaseSearchIndexProvider SEARCH_INDEX_DATA_PROVIDER = diff --git a/tests/robotests/src/com/android/settings/accessibility/ColorContrastFragmentTest.java b/tests/robotests/src/com/android/settings/accessibility/ColorContrastFragmentTest.java index f90374508e1..3077637a8e4 100644 --- a/tests/robotests/src/com/android/settings/accessibility/ColorContrastFragmentTest.java +++ b/tests/robotests/src/com/android/settings/accessibility/ColorContrastFragmentTest.java @@ -22,6 +22,7 @@ import static org.mockito.Mockito.spy; import static org.mockito.Mockito.when; import android.app.UiModeManager; +import android.app.settings.SettingsEnums; import android.content.Context; import androidx.test.core.app.ApplicationProvider; @@ -59,7 +60,8 @@ public class ColorContrastFragmentTest { @Test public void getMetricsCategory_returnsCorrectCategory() { - assertThat(mFragment.getMetricsCategory()).isEqualTo(0); + assertThat(mFragment.getMetricsCategory()).isEqualTo( + SettingsEnums.ACCESSIBILITY_COLOR_CONTRAST); } @Test