Merge "Fix typo in metrics constants"
This commit is contained in:
committed by
Android (Google) Code Review
commit
c26ae0677e
@@ -392,13 +392,13 @@ public class SearchFragment extends InstrumentedFragment implements SearchView.O
|
|||||||
taggedData.addAll(Arrays.asList(logTaggedData));
|
taggedData.addAll(Arrays.asList(logTaggedData));
|
||||||
}
|
}
|
||||||
taggedData.add(Pair.create(
|
taggedData.add(Pair.create(
|
||||||
MetricsEvent.FIELD_SETTINGS_SERACH_RESULT_COUNT,
|
MetricsEvent.FIELD_SETTINGS_SEARCH_RESULT_COUNT,
|
||||||
mSearchAdapter.getItemCount()));
|
mSearchAdapter.getItemCount()));
|
||||||
taggedData.add(Pair.create(
|
taggedData.add(Pair.create(
|
||||||
MetricsEvent.FIELD_SETTINGS_SERACH_RESULT_RANK,
|
MetricsEvent.FIELD_SETTINGS_SEARCH_RESULT_RANK,
|
||||||
resultViewHolder.getAdapterPosition()));
|
resultViewHolder.getAdapterPosition()));
|
||||||
taggedData.add(Pair.create(
|
taggedData.add(Pair.create(
|
||||||
MetricsEvent.FIELD_SETTINGS_SERACH_QUERY_LENGTH,
|
MetricsEvent.FIELD_SETTINGS_SEARCH_QUERY_LENGTH,
|
||||||
TextUtils.isEmpty(mQuery) ? 0 : mQuery.length()));
|
TextUtils.isEmpty(mQuery) ? 0 : mQuery.length()));
|
||||||
|
|
||||||
mMetricsFeatureProvider.action(getContext(),
|
mMetricsFeatureProvider.action(getContext(),
|
||||||
|
@@ -435,9 +435,9 @@ public class SearchFragmentTest {
|
|||||||
nullable(Context.class),
|
nullable(Context.class),
|
||||||
eq(MetricsProto.MetricsEvent.ACTION_CLICK_SETTINGS_SEARCH_RESULT),
|
eq(MetricsProto.MetricsEvent.ACTION_CLICK_SETTINGS_SEARCH_RESULT),
|
||||||
eq("test_setting"),
|
eq("test_setting"),
|
||||||
argThat(pairMatches(MetricsProto.MetricsEvent.FIELD_SETTINGS_SERACH_RESULT_COUNT)),
|
argThat(pairMatches(MetricsProto.MetricsEvent.FIELD_SETTINGS_SEARCH_RESULT_COUNT)),
|
||||||
argThat(pairMatches(MetricsProto.MetricsEvent.FIELD_SETTINGS_SERACH_RESULT_RANK)),
|
argThat(pairMatches(MetricsProto.MetricsEvent.FIELD_SETTINGS_SEARCH_RESULT_RANK)),
|
||||||
argThat(pairMatches(MetricsProto.MetricsEvent.FIELD_SETTINGS_SERACH_QUERY_LENGTH)));
|
argThat(pairMatches(MetricsProto.MetricsEvent.FIELD_SETTINGS_SEARCH_QUERY_LENGTH)));
|
||||||
|
|
||||||
verify(mFeatureFactory.searchFeatureProvider).searchResultClicked(nullable(Context.class),
|
verify(mFeatureFactory.searchFeatureProvider).searchResultClicked(nullable(Context.class),
|
||||||
nullable(String.class), eq(searchResult));
|
nullable(String.class), eq(searchResult));
|
||||||
|
Reference in New Issue
Block a user