From 3ea610af84224d0a5919441da38beb0ab5214954 Mon Sep 17 00:00:00 2001 From: Caitlin Cassidy Date: Fri, 13 Aug 2021 18:16:20 +0000 Subject: [PATCH 1/2] [Dagger] Inject NotificationEntryManager into StatusBarNotificationPresenter. Test: atest Bug: 138786270 Change-Id: Ib84badab344205fe039c4b2d94531e6aaed06fb3 --- .../src/com/android/systemui/statusbar/phone/StatusBar.java | 5 +++++ .../statusbar/phone/StatusBarNotificationPresenter.java | 5 +++-- .../statusbar/phone/dagger/StatusBarPhoneModule.java | 3 +++ .../statusbar/phone/StatusBarNotificationPresenterTest.java | 4 +--- .../com/android/systemui/statusbar/phone/StatusBarTest.java | 2 ++ 5 files changed, 14 insertions(+), 5 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBar.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBar.java index 9c2b2793c4673..bc72baf3081d1 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBar.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBar.java @@ -197,6 +197,7 @@ import com.android.systemui.statusbar.SysuiStatusBarStateController; import com.android.systemui.statusbar.events.SystemStatusAnimationScheduler; import com.android.systemui.statusbar.notification.DynamicPrivacyController; import com.android.systemui.statusbar.notification.NotificationActivityStarter; +import com.android.systemui.statusbar.notification.NotificationEntryManager; import com.android.systemui.statusbar.notification.NotificationLaunchAnimatorControllerProvider; import com.android.systemui.statusbar.notification.NotificationWakeUpCoordinator; import com.android.systemui.statusbar.notification.collection.legacy.VisualStabilityManager; @@ -529,6 +530,7 @@ public class StatusBar extends SystemUI implements private final int[] mAbsPos = new int[2]; + protected final NotificationEntryManager mEntryManager; private final NotificationGutsManager mGutsManager; private final NotificationLogger mNotificationLogger; private final NotificationViewHierarchyManager mViewHierarchyManager; @@ -805,6 +807,7 @@ public class StatusBar extends SystemUI implements FalsingManager falsingManager, FalsingCollector falsingCollector, BroadcastDispatcher broadcastDispatcher, + NotificationEntryManager notificationEntryManager, NotificationGutsManager notificationGutsManager, NotificationLogger notificationLogger, NotificationInterruptStateProvider notificationInterruptStateProvider, @@ -894,6 +897,7 @@ public class StatusBar extends SystemUI implements mFalsingCollector = falsingCollector; mFalsingManager = falsingManager; mBroadcastDispatcher = broadcastDispatcher; + mEntryManager = notificationEntryManager; mGutsManager = notificationGutsManager; mNotificationLogger = notificationLogger; mNotificationInterruptStateProvider = notificationInterruptStateProvider; @@ -1530,6 +1534,7 @@ public class StatusBar extends SystemUI implements mViewHierarchyManager, mLockscreenUserManager, mStatusBarStateController, + mEntryManager, mMediaManager, mGutsManager, mKeyguardUpdateMonitor, diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenter.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenter.java index 608697df79058..29dd21cc42f59 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenter.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenter.java @@ -93,8 +93,7 @@ public class StatusBarNotificationPresenter implements NotificationPresenter, private final NotificationViewHierarchyManager mViewHierarchyManager; private final NotificationLockscreenUserManager mLockscreenUserManager; private final SysuiStatusBarStateController mStatusBarStateController; - private final NotificationEntryManager mEntryManager = - Dependency.get(NotificationEntryManager.class); + private final NotificationEntryManager mEntryManager; private final NotificationMediaManager mMediaManager; private final NotificationGutsManager mGutsManager; private final KeyguardUpdateMonitor mKeyguardUpdateMonitor; @@ -139,6 +138,7 @@ public class StatusBarNotificationPresenter implements NotificationPresenter, NotificationViewHierarchyManager notificationViewHierarchyManager, NotificationLockscreenUserManager lockscreenUserManager, SysuiStatusBarStateController sysuiStatusBarStateController, + NotificationEntryManager notificationEntryManager, NotificationMediaManager notificationMediaManager, NotificationGutsManager notificationGutsManager, KeyguardUpdateMonitor keyguardUpdateMonitor, @@ -159,6 +159,7 @@ public class StatusBarNotificationPresenter implements NotificationPresenter, mViewHierarchyManager = notificationViewHierarchyManager; mLockscreenUserManager = lockscreenUserManager; mStatusBarStateController = sysuiStatusBarStateController; + mEntryManager = notificationEntryManager; mMediaManager = notificationMediaManager; mGutsManager = notificationGutsManager; mKeyguardUpdateMonitor = keyguardUpdateMonitor; diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/dagger/StatusBarPhoneModule.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/dagger/StatusBarPhoneModule.java index eabb2ab696ee2..83d0474414b1f 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/dagger/StatusBarPhoneModule.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/dagger/StatusBarPhoneModule.java @@ -60,6 +60,7 @@ import com.android.systemui.statusbar.SuperStatusBarViewFactory; import com.android.systemui.statusbar.SysuiStatusBarStateController; import com.android.systemui.statusbar.events.SystemStatusAnimationScheduler; import com.android.systemui.statusbar.notification.DynamicPrivacyController; +import com.android.systemui.statusbar.notification.NotificationEntryManager; import com.android.systemui.statusbar.notification.NotificationWakeUpCoordinator; import com.android.systemui.statusbar.notification.collection.legacy.VisualStabilityManager; import com.android.systemui.statusbar.notification.init.NotificationsController; @@ -143,6 +144,7 @@ public interface StatusBarPhoneModule { FalsingManager falsingManager, FalsingCollector falsingCollector, BroadcastDispatcher broadcastDispatcher, + NotificationEntryManager notificationEntryManager, NotificationGutsManager notificationGutsManager, NotificationLogger notificationLogger, NotificationInterruptStateProvider notificationInterruptStateProvider, @@ -231,6 +233,7 @@ public interface StatusBarPhoneModule { falsingManager, falsingCollector, broadcastDispatcher, + notificationEntryManager, notificationGutsManager, notificationLogger, notificationInterruptStateProvider, diff --git a/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenterTest.java b/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenterTest.java index 03a353b6c2b47..cc9c2b4448c2c 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenterTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenterTest.java @@ -96,9 +96,6 @@ public class StatusBarNotificationPresenterTest extends SysuiTestCase { mDependency.injectMockDependency(NotificationRemoteInputManager.Callback.class); mDependency.injectMockDependency(NotificationShadeWindowController.class); mDependency.injectMockDependency(ForegroundServiceNotificationListener.class); - NotificationEntryManager entryManager = - mDependency.injectMockDependency(NotificationEntryManager.class); - when(entryManager.getActiveNotificationsForCurrentUser()).thenReturn(new ArrayList<>()); NotificationShadeWindowView notificationShadeWindowView = mock(NotificationShadeWindowView.class); @@ -122,6 +119,7 @@ public class StatusBarNotificationPresenterTest extends SysuiTestCase { mock(NotificationViewHierarchyManager.class), mock(NotificationLockscreenUserManager.class), mock(SysuiStatusBarStateController.class), + mock(NotificationEntryManager.class), mock(NotificationMediaManager.class), mock(NotificationGutsManager.class), mock(KeyguardUpdateMonitor.class), diff --git a/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarTest.java b/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarTest.java index e4f6e13db607f..46bc3f3bb44cd 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarTest.java @@ -205,6 +205,7 @@ public class StatusBarTest extends SysuiTestCase { @Mock private NotificationShadeWindowView mNotificationShadeWindowView; @Mock private BroadcastDispatcher mBroadcastDispatcher; @Mock private AssistManager mAssistManager; + @Mock private NotificationEntryManager mNotificationEntryManager; @Mock private NotificationGutsManager mNotificationGutsManager; @Mock private NotificationMediaManager mNotificationMediaManager; @Mock private NavigationBarController mNavigationBarController; @@ -359,6 +360,7 @@ public class StatusBarTest extends SysuiTestCase { new FalsingManagerFake(), new FalsingCollectorFake(), mBroadcastDispatcher, + mNotificationEntryManager, mNotificationGutsManager, notificationLogger, mNotificationInterruptStateProvider, From 59de6b39a4c1a1c4868b4fd83541390006d3bc0e Mon Sep 17 00:00:00 2001 From: Caitlin Cassidy Date: Fri, 13 Aug 2021 19:47:12 +0000 Subject: [PATCH 2/2] [Dagger] Inject LockscreenGestureLogger into PanelViewController and StatusBarNotificationPresenter. Test: atest Bug: 138786270 Change-Id: I81cb369944ceccb9b7552a681a37b0d1ff66834c --- .../phone/LockscreenGestureLogger.java | 6 +++--- .../phone/NotificationPanelViewController.java | 18 +++++++++++++----- .../statusbar/phone/PanelViewController.java | 4 +++- .../systemui/statusbar/phone/StatusBar.java | 4 ++++ .../phone/StatusBarNotificationPresenter.java | 7 +++---- .../phone/dagger/StatusBarPhoneModule.java | 3 +++ .../phone/NotificationPanelViewTest.java | 3 +++ .../StatusBarNotificationPresenterTest.java | 9 +++------ .../statusbar/phone/StatusBarTest.java | 2 ++ 9 files changed, 37 insertions(+), 19 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/LockscreenGestureLogger.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/LockscreenGestureLogger.java index 094ebb9ef0a7b..5f222afd77e50 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/LockscreenGestureLogger.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/LockscreenGestureLogger.java @@ -24,7 +24,6 @@ import com.android.internal.logging.UiEvent; import com.android.internal.logging.UiEventLogger; import com.android.internal.logging.UiEventLoggerImpl; import com.android.internal.logging.nano.MetricsProto.MetricsEvent; -import com.android.systemui.Dependency; import com.android.systemui.EventLogConstants; import com.android.systemui.EventLogTags; import com.android.systemui.dagger.SysUISingleton; @@ -88,10 +87,11 @@ public class LockscreenGestureLogger { } private ArrayMap mLegacyMap; - private final MetricsLogger mMetricsLogger = Dependency.get(MetricsLogger.class); + private final MetricsLogger mMetricsLogger; @Inject - public LockscreenGestureLogger() { + public LockscreenGestureLogger(MetricsLogger metricsLogger) { + mMetricsLogger = metricsLogger; mLegacyMap = new ArrayMap<>(EventLogConstants.METRICS_GESTURE_TYPE_MAP.length); for (int i = 0; i < EventLogConstants.METRICS_GESTURE_TYPE_MAP.length ; i++) { mLegacyMap.put(EventLogConstants.METRICS_GESTURE_TYPE_MAP[i], i); diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/NotificationPanelViewController.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/NotificationPanelViewController.java index 30cb1dc9bd16d..5b724f7c10295 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/NotificationPanelViewController.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/NotificationPanelViewController.java @@ -482,7 +482,6 @@ public class NotificationPanelViewController extends PanelViewController { private float mLinearDarkAmount; private boolean mPulsing; - private LockscreenGestureLogger mLockscreenGestureLogger = new LockscreenGestureLogger(); private boolean mUserSetupComplete; private int mQsNotificationTopPadding; private boolean mHideIconsDuringLaunchAnimation = true; @@ -742,12 +741,21 @@ public class NotificationPanelViewController extends PanelViewController { SplitShadeHeaderController splitShadeHeaderController, LockscreenSmartspaceController lockscreenSmartspaceController, UnlockedScreenOffAnimationController unlockedScreenOffAnimationController, + LockscreenGestureLogger lockscreenGestureLogger, NotificationRemoteInputManager remoteInputManager, ControlsComponent controlsComponent) { - super(view, falsingManager, dozeLog, keyguardStateController, - (SysuiStatusBarStateController) statusBarStateController, vibratorHelper, - statusBarKeyguardViewManager, latencyTracker, flingAnimationUtilsBuilder.get(), - statusBarTouchableRegionManager, ambientState); + super(view, + falsingManager, + dozeLog, + keyguardStateController, + (SysuiStatusBarStateController) statusBarStateController, + vibratorHelper, + statusBarKeyguardViewManager, + latencyTracker, + flingAnimationUtilsBuilder.get(), + statusBarTouchableRegionManager, + lockscreenGestureLogger, + ambientState); mView = view; mVibratorHelper = vibratorHelper; mKeyguardMediaController = keyguardMediaController; diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/PanelViewController.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/PanelViewController.java index 27f08a860f28b..310f02e9f222e 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/PanelViewController.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/PanelViewController.java @@ -79,7 +79,6 @@ public abstract class PanelViewController { protected long mDownTime; protected boolean mTouchSlopExceededBeforeDown; private float mMinExpandHeight; - private LockscreenGestureLogger mLockscreenGestureLogger = new LockscreenGestureLogger(); private boolean mPanelUpdateWhenAnimatorEnds; private boolean mVibrateOnOpening; protected boolean mIsLaunchAnimationRunning; @@ -182,6 +181,7 @@ public abstract class PanelViewController { protected final KeyguardStateController mKeyguardStateController; protected final SysuiStatusBarStateController mStatusBarStateController; protected final AmbientState mAmbientState; + protected final LockscreenGestureLogger mLockscreenGestureLogger; protected void onExpandingFinished() { mBar.onExpandingFinished(); @@ -217,10 +217,12 @@ public abstract class PanelViewController { LatencyTracker latencyTracker, FlingAnimationUtils.Builder flingAnimationUtilsBuilder, StatusBarTouchableRegionManager statusBarTouchableRegionManager, + LockscreenGestureLogger lockscreenGestureLogger, AmbientState ambientState) { mAmbientState = ambientState; mView = view; mStatusBarKeyguardViewManager = statusBarKeyguardViewManager; + mLockscreenGestureLogger = lockscreenGestureLogger; mView.addOnAttachStateChangeListener(new View.OnAttachStateChangeListener() { @Override public void onViewAttachedToWindow(View v) { diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBar.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBar.java index bc72baf3081d1..565ca507df50a 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBar.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBar.java @@ -455,6 +455,7 @@ public class StatusBar extends SystemUI implements private BiometricUnlockController mBiometricUnlockController; private final LightBarController mLightBarController; private final Lazy mLockscreenWallpaperLazy; + private final LockscreenGestureLogger mLockscreenGestureLogger; @Nullable protected LockscreenWallpaper mLockscreenWallpaper; private final AutoHideController mAutoHideController; @@ -837,6 +838,7 @@ public class StatusBar extends SystemUI implements DozeParameters dozeParameters, ScrimController scrimController, Lazy lockscreenWallpaperLazy, + LockscreenGestureLogger lockscreenGestureLogger, Lazy biometricUnlockControllerLazy, DozeServiceHost dozeServiceHost, PowerManager powerManager, @@ -929,6 +931,7 @@ public class StatusBar extends SystemUI implements mDozeParameters = dozeParameters; mScrimController = scrimController; mLockscreenWallpaperLazy = lockscreenWallpaperLazy; + mLockscreenGestureLogger = lockscreenGestureLogger; mScreenPinningRequest = screenPinningRequest; mDozeScrimController = dozeScrimController; mBiometricUnlockControllerLazy = biometricUnlockControllerLazy; @@ -1538,6 +1541,7 @@ public class StatusBar extends SystemUI implements mMediaManager, mGutsManager, mKeyguardUpdateMonitor, + mLockscreenGestureLogger, mInitController, mNotificationInterruptStateProvider, mRemoteInputManager, diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenter.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenter.java index 29dd21cc42f59..8ebaf9188e912 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenter.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenter.java @@ -82,10 +82,6 @@ public class StatusBarNotificationPresenter implements NotificationPresenter, ConfigurationController.ConfigurationListener, NotificationRowBinderImpl.BindRowCallback, CommandQueue.Callbacks { - - private final LockscreenGestureLogger mLockscreenGestureLogger = - Dependency.get(LockscreenGestureLogger.class); - private static final String TAG = "StatusBarNotificationPresenter"; private final ActivityStarter mActivityStarter = Dependency.get(ActivityStarter.class); @@ -97,6 +93,7 @@ public class StatusBarNotificationPresenter implements NotificationPresenter, private final NotificationMediaManager mMediaManager; private final NotificationGutsManager mGutsManager; private final KeyguardUpdateMonitor mKeyguardUpdateMonitor; + private final LockscreenGestureLogger mLockscreenGestureLogger; private final NotificationPanelViewController mNotificationPanel; private final HeadsUpManagerPhone mHeadsUpManager; @@ -142,6 +139,7 @@ public class StatusBarNotificationPresenter implements NotificationPresenter, NotificationMediaManager notificationMediaManager, NotificationGutsManager notificationGutsManager, KeyguardUpdateMonitor keyguardUpdateMonitor, + LockscreenGestureLogger lockscreenGestureLogger, InitController initController, NotificationInterruptStateProvider notificationInterruptStateProvider, NotificationRemoteInputManager remoteInputManager, @@ -163,6 +161,7 @@ public class StatusBarNotificationPresenter implements NotificationPresenter, mMediaManager = notificationMediaManager; mGutsManager = notificationGutsManager; mKeyguardUpdateMonitor = keyguardUpdateMonitor; + mLockscreenGestureLogger = lockscreenGestureLogger; mAboveShelfObserver = new AboveShelfObserver(stackScrollerController.getView()); mNotificationShadeWindowController = notificationShadeWindowController; mAboveShelfObserver.setListener(statusBarWindow.findViewById( diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/dagger/StatusBarPhoneModule.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/dagger/StatusBarPhoneModule.java index 83d0474414b1f..4074caaa2fb44 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/dagger/StatusBarPhoneModule.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/dagger/StatusBarPhoneModule.java @@ -78,6 +78,7 @@ import com.android.systemui.statusbar.phone.KeyguardBypassController; import com.android.systemui.statusbar.phone.KeyguardDismissUtil; import com.android.systemui.statusbar.phone.LightBarController; import com.android.systemui.statusbar.phone.LightsOutNotifController; +import com.android.systemui.statusbar.phone.LockscreenGestureLogger; import com.android.systemui.statusbar.phone.LockscreenWallpaper; import com.android.systemui.statusbar.phone.NotificationIconAreaController; import com.android.systemui.statusbar.phone.PhoneStatusBarPolicy; @@ -174,6 +175,7 @@ public interface StatusBarPhoneModule { DozeParameters dozeParameters, ScrimController scrimController, Lazy lockscreenWallpaperLazy, + LockscreenGestureLogger lockscreenGestureLogger, Lazy biometricUnlockControllerLazy, DozeServiceHost dozeServiceHost, PowerManager powerManager, @@ -263,6 +265,7 @@ public interface StatusBarPhoneModule { dozeParameters, scrimController, lockscreenWallpaperLazy, + lockscreenGestureLogger, biometricUnlockControllerLazy, dozeServiceHost, powerManager, diff --git a/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/NotificationPanelViewTest.java b/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/NotificationPanelViewTest.java index 1387b8e1f1b96..0ea619dfadc89 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/NotificationPanelViewTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/NotificationPanelViewTest.java @@ -306,6 +306,8 @@ public class NotificationPanelViewTest extends SysuiTestCase { private LockscreenSmartspaceController mLockscreenSmartspaceController; @Mock private FrameLayout mSplitShadeSmartspaceContainer; + @Mock + private LockscreenGestureLogger mLockscreenGestureLogger; private SysuiStatusBarStateController mStatusBarStateController; private NotificationPanelViewController mNotificationPanelViewController; @@ -452,6 +454,7 @@ public class NotificationPanelViewTest extends SysuiTestCase { mSplitShadeHeaderController, mLockscreenSmartspaceController, mUnlockedScreenOffAnimationController, + mLockscreenGestureLogger, mNotificationRemoteInputManager, mControlsComponent); mNotificationPanelViewController.initDependencies( diff --git a/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenterTest.java b/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenterTest.java index cc9c2b4448c2c..c80c07249cc57 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenterTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarNotificationPresenterTest.java @@ -31,7 +31,6 @@ import android.testing.TestableLooper.RunWithLooper; import androidx.test.filters.SmallTest; -import com.android.internal.logging.MetricsLogger; import com.android.internal.logging.nano.MetricsProto.MetricsEvent; import com.android.internal.logging.testing.FakeMetricsLogger; import com.android.keyguard.KeyguardUpdateMonitor; @@ -67,14 +66,10 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.mockito.ArgumentCaptor; -import java.util.ArrayList; - @SmallTest @RunWith(AndroidTestingRunner.class) @RunWithLooper() public class StatusBarNotificationPresenterTest extends SysuiTestCase { - - private StatusBarNotificationPresenter mStatusBarNotificationPresenter; private NotificationInterruptStateProvider mNotificationInterruptStateProvider = mock(NotificationInterruptStateProvider.class); @@ -88,7 +83,8 @@ public class StatusBarNotificationPresenterTest extends SysuiTestCase { @Before public void setup() { mMetricsLogger = new FakeMetricsLogger(); - mDependency.injectTestDependency(MetricsLogger.class, mMetricsLogger); + LockscreenGestureLogger lockscreenGestureLogger = new LockscreenGestureLogger( + mMetricsLogger); mCommandQueue = new CommandQueue(mContext); mDependency.injectTestDependency(StatusBarStateController.class, mock(SysuiStatusBarStateController.class)); @@ -123,6 +119,7 @@ public class StatusBarNotificationPresenterTest extends SysuiTestCase { mock(NotificationMediaManager.class), mock(NotificationGutsManager.class), mock(KeyguardUpdateMonitor.class), + lockscreenGestureLogger, mInitController, mNotificationInterruptStateProvider, mock(NotificationRemoteInputManager.class), diff --git a/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarTest.java b/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarTest.java index 46bc3f3bb44cd..216be628620a9 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarTest.java @@ -227,6 +227,7 @@ public class StatusBarTest extends SysuiTestCase { @Mock private NotificationShadeWindowViewController mNotificationShadeWindowViewController; @Mock private DozeParameters mDozeParameters; @Mock private Lazy mLockscreenWallpaperLazy; + @Mock private LockscreenGestureLogger mLockscreenGestureLogger; @Mock private LockscreenWallpaper mLockscreenWallpaper; @Mock private DozeServiceHost mDozeServiceHost; @Mock private ViewMediatorCallback mKeyguardVieMediatorCallback; @@ -390,6 +391,7 @@ public class StatusBarTest extends SysuiTestCase { mDozeParameters, mScrimController, mLockscreenWallpaperLazy, + mLockscreenGestureLogger, mBiometricUnlockControllerLazy, mDozeServiceHost, mPowerManager, mScreenPinningRequest,