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

Commit 71ab5f86 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: Iae8daad78ca310c54d057006857c6ea3e0b5b8de
parents 16d2cb74 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);
    }
}