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

Commit b9d3ffd1 authored by Massimo Carli's avatar Massimo Carli
Browse files

[33/n] Remove redundant spyOn()

spyOn() invocations on AppCompatConfiguration is redundant because
already done in the AppCompatConfigurationRobot.

Flag: EXEMPT refactor
Bug: 351763164
Test: atest WmTests:AppCompatCameraOverridesTest
Test: atest WmTests:AppCompatFocusOverridesTest
Test: atest WmTests:AppCompatOrientationPolicyTest
Test: atest WmTests:AppCompatReachabilityOverridesTest
Test: atest WmTests:AppCompatReachabilityPolicyTest
Test: atest WmTests:AppCompatResizeOverridesTest

Change-Id: Ib2755c1be7c9ffa73f4cb72dbed13c6ed454c6cf
parent 4575a97d
Loading
Loading
Loading
Loading
+0 −1
Original line number Diff line number Diff line
@@ -374,7 +374,6 @@ public class AppCompatCameraOverridesTest extends WindowTestsBase {
     * Runs a test scenario providing a Robot.
     */
    void runTestScenario(@NonNull Consumer<CameraOverridesRobotTest> consumer) {
        spyOn(mWm.mAppCompatConfiguration);
        final CameraOverridesRobotTest robot = new CameraOverridesRobotTest(mWm, mAtm, mSupervisor);
        consumer.accept(robot);
    }
+0 −3
Original line number Diff line number Diff line
@@ -19,8 +19,6 @@ package com.android.server.wm;
import static android.content.pm.ActivityInfo.OVERRIDE_ENABLE_COMPAT_FAKE_FOCUS;
import static android.view.WindowManager.PROPERTY_COMPAT_ENABLE_FAKE_FOCUS;

import static com.android.dx.mockito.inline.extended.ExtendedMockito.spyOn;

import android.compat.testing.PlatformCompatChangeRule;
import android.platform.test.annotations.Presubmit;

@@ -177,7 +175,6 @@ public class AppCompatFocusOverridesTest extends WindowTestsBase {
     * Runs a test scenario providing a Robot.
     */
    void runTestScenario(@NonNull Consumer<FocusOverridesRobotTest> consumer) {
        spyOn(mWm.mAppCompatConfiguration);
        final FocusOverridesRobotTest robot = new FocusOverridesRobotTest(mWm, mAtm, mSupervisor);
        consumer.accept(robot);
    }
+0 −1
Original line number Diff line number Diff line
@@ -514,7 +514,6 @@ public class AppCompatOrientationPolicyTest extends WindowTestsBase {
     */
    void runTestScenario(boolean withActivity,
                         @NonNull Consumer<OrientationPolicyRobotTest> consumer) {
        spyOn(mWm.mAppCompatConfiguration);
        final OrientationPolicyRobotTest robot =
                new OrientationPolicyRobotTest(mWm, mAtm, mSupervisor, withActivity);
        consumer.accept(robot);
+0 −1
Original line number Diff line number Diff line
@@ -164,7 +164,6 @@ public class AppCompatReachabilityOverridesTest extends WindowTestsBase {
     * Runs a test scenario providing a Robot.
     */
    void runTestScenario(@NonNull Consumer<ReachabilityOverridesRobotTest> consumer) {
        spyOn(mWm.mAppCompatConfiguration);
        final ReachabilityOverridesRobotTest robot =
                new ReachabilityOverridesRobotTest(mWm, mAtm, mSupervisor);
        consumer.accept(robot);
+0 −1
Original line number Diff line number Diff line
@@ -228,7 +228,6 @@ public class AppCompatReachabilityPolicyTest extends WindowTestsBase {
     * Runs a test scenario providing a Robot.
     */
    void runTestScenario(@NonNull Consumer<ReachabilityPolicyRobotTest> consumer) {
        spyOn(mWm.mAppCompatConfiguration);
        final ReachabilityPolicyRobotTest robot =
                new ReachabilityPolicyRobotTest(mWm, mAtm, mSupervisor);
        consumer.accept(robot);
Loading