Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit eb979c4c authored by Edgar Wang's avatar Edgar Wang Committed by Android (Google) Code Review
Browse files

Merge "Public logSettingsTileClick for log some widget in Settings" into rvc-dev

parents 06e29c96 36ecb3de
Loading
Loading
Loading
Loading
+6 −1
Original line number Original line Diff line number Diff line
@@ -185,7 +185,12 @@ public class MetricsFeatureProvider {
                sourceMetricsCategory);
                sourceMetricsCategory);
    }
    }


    private boolean logSettingsTileClick(String logKey, int sourceMetricsCategory) {
    /**
     * Logs an event when the setting key is clicked.
     *
     * @return true if the key is loggable, otherwise false
     */
    public boolean logSettingsTileClick(String logKey, int sourceMetricsCategory) {
        if (TextUtils.isEmpty(logKey)) {
        if (TextUtils.isEmpty(logKey)) {
            // Not loggable
            // Not loggable
            return false;
            return false;
+25 −0
Original line number Original line Diff line number Diff line
@@ -218,4 +218,29 @@ public class MetricsFeatureProviderTest {


        assertThat(mProvider.getAttribution(activity)).isEqualTo(100);
        assertThat(mProvider.getAttribution(activity)).isEqualTo(100);
    }
    }

    @Test
    public void logSettingsTileClick_hasKey_shouldLog() {
        final String key = "abc";
        final boolean loggable = mProvider.logSettingsTileClick(key,
                MetricsEvent.SETTINGS_GESTURES);

        assertThat(loggable).isTrue();
        verify(mLogWriter).action(
                MetricsEvent.SETTINGS_GESTURES,
                MetricsEvent.ACTION_SETTINGS_TILE_CLICK,
                SettingsEnums.PAGE_UNKNOWN,
                key,
                0);
    }

    @Test
    public void logSettingsTileClick_keyEmpty_shouldNotLog() {
        final String key = "";
        boolean loggable = mProvider.logSettingsTileClick(key,
                MetricsEvent.SETTINGS_GESTURES);

        assertThat(loggable).isFalse();
        verifyNoMoreInteractions(mLogWriter);
    }
}
}