* commit '4e7386bda93c41b622c64b1ce17b7d1b8bd503fd': Fix CTS build
This commit is contained in:
@@ -34,6 +34,7 @@ import java.util.concurrent.TimeUnit;
|
||||
|
||||
public class TrafficStatsTest extends AndroidTestCase {
|
||||
private static final String LOG_TAG = "TrafficStatsTest";
|
||||
|
||||
public void testValidMobileStats() {
|
||||
// We can't assume a mobile network is even present in this test, so
|
||||
// we simply assert that a valid value is returned.
|
||||
@@ -217,17 +218,6 @@ public class TrafficStatsTest extends AndroidTestCase {
|
||||
assertTrue("too many non-localhost packets on the sam UID", deltaTxOtherPackets + deltaTxOtherPackets < 20);
|
||||
}
|
||||
|
||||
// Some other tests don't cleanup connections correctly.
|
||||
// They have the same UID, so we discount their lingering traffic
|
||||
// which happens only on non-localhost, such as TCP FIN retranmission packets
|
||||
long deltaTxOtherPackets = (totalTxPacketsAfter - totalTxPacketsBefore) - uidTxDeltaPackets;
|
||||
long deltaRxOtherPackets = (totalRxPacketsAfter - totalRxPacketsBefore) - uidRxDeltaPackets;
|
||||
if (deltaTxOtherPackets > 0 || deltaRxOtherPackets > 0) {
|
||||
Log.i(LOG_TAG, "lingering traffic data: " + deltaTxOtherPackets + "/" + deltaRxOtherPackets);
|
||||
// Make sure that not too many non-localhost packets are accounted for
|
||||
assertTrue("too many non-localhost packets on the sam UID", deltaTxOtherPackets + deltaTxOtherPackets < 20);
|
||||
}
|
||||
|
||||
assertTrue("uidtxp: " + uidTxPacketsBefore + " -> " + uidTxPacketsAfter + " delta=" + uidTxDeltaPackets +
|
||||
" Wanted: " + uidTxDeltaPackets + ">=" + packetCount + "+" + minExpectedExtraPackets + " && " +
|
||||
uidTxDeltaPackets + "<=" + packetCount + "+" + packetCount + "+" + maxExpectedExtraPackets + "+" + deltaTxOtherPackets,
|
||||
|
||||
Reference in New Issue
Block a user