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

Commit 849b81b7 authored by Hugo Benichi's avatar Hugo Benichi
Browse files

@Ignore ConnectivityServiceTest#testRequestBenchmark 

Ignore the last remaining test in ConnectivityServiceTest with spurious
failures. testRequestBenchmark has some intrinsic chances of failure due
to the fact it attempts to assert elapsed time durations against a
reference target.

Bug: 32561414
Test: no functional change
Change-Id: Ib25d76581b47997b2ef84df3e6a9fd9224b85d92
parent 475d9b28

tests/net/java/com/android/server/ConnectivityServiceTest.java

+2 −0
Original line number Diff line number Diff line
@@ -97,6 +97,7 @@ import com.android.server.connectivity.NetworkMonitor; 
import com.android.server.connectivity.NetworkMonitor.CaptivePortalProbeResult;

import com.android.server.net.NetworkPinner;



import org.junit.Ignore;

import org.mockito.Mock;

import org.mockito.MockitoAnnotations;

import org.mockito.Spy;
@@ -2409,6 +2410,7 @@ public class ConnectivityServiceTest extends AndroidTestCase { 
        mCm.unregisterNetworkCallback(fgCallback);

    }



    @Ignore // This test has instrinsic chances of spurious failures: ignore for continuous testing.

    @SmallTest

    public void testRequestBenchmark() throws Exception {

        // TODO: turn this unit test into a real benchmarking test.