diff options
| author | 2022-12-12 16:26:16 -0500 | |
|---|---|---|
| committer | 2022-12-15 16:03:11 +0000 | |
| commit | 9ebd81953ce60236381a068a0b7c52c8164bbfea (patch) | |
| tree | dc6daacf40273cef8dc782f582ce6ccd73436edc /java/tests/AndroidTest.xml | |
| parent | 92160d6e9fb1caa1142599ee252f53b796298be5 (diff) | |
Extract remainining ChooserActivity logging
This especially covers other kinds of logging in ChooserActivity that
hadn't previously been a responsibility of ChooserActivityLogger (e.g.
because they instead went directly through MetricsLogger).
There's a few reasons to make these changes, in rough priority order:
1. Tests can make assertions about the high-level logging methods we
expect to call; the implementation of those methods (in terms of
low-level event streams) is unit-tested for the logger itself.
2. We can encapsulate all the types of loggers (UiEventLogger,
MetricsLogger, etc.) in a single object, which is easier to inject
as a dependency as we continue decomposing ChooserActivity
responsibilities into other components.
3. High-level logger APIs reduce the clutter that clients previously
spent building up the low-level event data that's now abstracted
away.
Test: atest IntentResolverUnitTests
Bug: 202167050
Change-Id: Ib3197bca12ec5ea3c925946c0f6d37a6be19d8fa
Diffstat (limited to 'java/tests/AndroidTest.xml')
0 files changed, 0 insertions, 0 deletions