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

Commit dd2d0503 authored by Edgar Wang's avatar Edgar Wang Committed by Automerger Merge Worker
Browse files

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

Change-Id: I359b10efbc20088726a9f08697a59a7b93325506
parents f6a9b172 eb979c4c
Loading
Loading
Loading
Loading
+6 −1
Original line number Diff line number Diff line
@@ -185,7 +185,12 @@ public class MetricsFeatureProvider {
                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)) {
            // Not loggable
            return false;
+25 −0
Original line number Diff line number Diff line
@@ -218,4 +218,29 @@ public class MetricsFeatureProviderTest {

        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);
    }
}