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

Commit 57218af3 authored by Chiachang Wang's avatar Chiachang Wang
Browse files

Prevent race in NetworkMonitorTest 

Move setupTcpDataStall to do before preparing test NM.

Bug: 157612829
Test: atest com.android.server.connectivity.NetworkMonitorTest\
       #testDataStall_StallTcpSuspectedAndSendMetricsOnWifi\
       --iterations 30
Change-Id: I764a97b1007613e7d1c19ec6294faac27bea1fef
parent 5576d8e4

tests/unit/src/com/android/server/connectivity/NetworkMonitorTest.java

+2 −1
Original line number Diff line number Diff line
@@ -1834,10 +1834,11 @@ public class NetworkMonitorTest { 
    private void testDataStall_StallTcpSuspectedAndSendMetrics(NetworkCapabilities nc)

            throws Exception {

        assumeTrue(ShimUtils.isReleaseOrDevelopmentApiAbove(Build.VERSION_CODES.Q));

        setupTcpDataStall();

        // NM suspects data stall from TCP signal and sends data stall metrics.

        setDataStallEvaluationType(DATA_STALL_EVALUATION_TYPE_TCP);

        final WrappedNetworkMonitor nm = prepareNetworkMonitorForVerifyDataStall(nc);

        setupTcpDataStall();



        // Trigger a tcp event immediately.

        setTcpPollingInterval(0);

        nm.sendTcpPollingEvent();