Add some non-connectivity-module-test back to ConnectivityCoverageTests
In S, the tests are listed in the non-connectivity-module-test where their testing files are not in the connectivity module. But some files move into the connectivity module in T. So some tests can be removed from non-connectivity-module-test and will be running on ConnectivityCoverageTests to count their code coverage. Also update some tests to make them compatible with mockito-extended and ignore some tests which need to test with T apis. Bug: 234000451 Test: atest FrameworksNetTests ConnectivityCoverageTests Change-Id: I47fb59334d00ae7fee91c5694548b3cf89b09b2d
This commit is contained in:
@@ -54,42 +54,21 @@ java_defaults {
|
|||||||
filegroup {
|
filegroup {
|
||||||
name: "non-connectivity-module-test",
|
name: "non-connectivity-module-test",
|
||||||
srcs: [
|
srcs: [
|
||||||
"java/android/app/usage/*.java",
|
|
||||||
"java/android/net/EthernetNetworkUpdateRequestTest.java",
|
|
||||||
"java/android/net/Ikev2VpnProfileTest.java",
|
"java/android/net/Ikev2VpnProfileTest.java",
|
||||||
"java/android/net/IpMemoryStoreTest.java",
|
"java/android/net/IpMemoryStoreTest.java",
|
||||||
"java/android/net/IpSecAlgorithmTest.java",
|
|
||||||
"java/android/net/IpSecConfigTest.java",
|
|
||||||
"java/android/net/IpSecManagerTest.java",
|
|
||||||
"java/android/net/IpSecTransformTest.java",
|
|
||||||
"java/android/net/KeepalivePacketDataUtilTest.java",
|
|
||||||
"java/android/net/NetworkIdentitySetTest.kt",
|
|
||||||
"java/android/net/NetworkIdentityTest.kt",
|
|
||||||
"java/android/net/NetworkStats*.java",
|
|
||||||
"java/android/net/NetworkTemplateTest.kt",
|
|
||||||
"java/android/net/TelephonyNetworkSpecifierTest.java",
|
"java/android/net/TelephonyNetworkSpecifierTest.java",
|
||||||
"java/android/net/VpnManagerTest.java",
|
"java/android/net/VpnManagerTest.java",
|
||||||
"java/android/net/ipmemorystore/*.java",
|
"java/android/net/ipmemorystore/*.java",
|
||||||
"java/android/net/netstats/NetworkStatsDataMigrationUtilsTest.kt",
|
"java/android/net/netstats/NetworkStatsDataMigrationUtilsTest.kt",
|
||||||
"java/android/net/nsd/*.java",
|
|
||||||
"java/com/android/internal/net/NetworkUtilsInternalTest.java",
|
"java/com/android/internal/net/NetworkUtilsInternalTest.java",
|
||||||
"java/com/android/internal/net/VpnProfileTest.java",
|
"java/com/android/internal/net/VpnProfileTest.java",
|
||||||
"java/com/android/server/IpSecServiceParameterizedTest.java",
|
|
||||||
"java/com/android/server/IpSecServiceRefcountedResourceTest.java",
|
|
||||||
"java/com/android/server/IpSecServiceTest.java",
|
|
||||||
"java/com/android/server/NetworkManagementServiceTest.java",
|
"java/com/android/server/NetworkManagementServiceTest.java",
|
||||||
"java/com/android/server/NsdServiceTest.java",
|
|
||||||
"java/com/android/server/connectivity/IpConnectivityEventBuilderTest.java",
|
"java/com/android/server/connectivity/IpConnectivityEventBuilderTest.java",
|
||||||
"java/com/android/server/connectivity/IpConnectivityMetricsTest.java",
|
"java/com/android/server/connectivity/IpConnectivityMetricsTest.java",
|
||||||
"java/com/android/server/connectivity/MultipathPolicyTrackerTest.java",
|
"java/com/android/server/connectivity/MultipathPolicyTrackerTest.java",
|
||||||
"java/com/android/server/connectivity/NetdEventListenerServiceTest.java",
|
"java/com/android/server/connectivity/NetdEventListenerServiceTest.java",
|
||||||
"java/com/android/server/connectivity/VpnTest.java",
|
"java/com/android/server/connectivity/VpnTest.java",
|
||||||
"java/com/android/server/ethernet/*.java",
|
|
||||||
"java/com/android/server/net/ipmemorystore/*.java",
|
"java/com/android/server/net/ipmemorystore/*.java",
|
||||||
"java/com/android/server/net/BpfInterfaceMapUpdaterTest.java",
|
|
||||||
"java/com/android/server/net/IpConfigStoreTest.java",
|
|
||||||
"java/com/android/server/net/NetworkStats*.java",
|
|
||||||
"java/com/android/server/net/TestableUsageCallback.kt",
|
|
||||||
]
|
]
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -54,7 +54,7 @@ import org.mockito.invocation.InvocationOnMock;
|
|||||||
|
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class NetworkStatsManagerTest {
|
public class NetworkStatsManagerTest {
|
||||||
private static final String TEST_SUBSCRIBER_ID = "subid";
|
private static final String TEST_SUBSCRIBER_ID = "subid";
|
||||||
|
|
||||||
|
|||||||
@@ -47,7 +47,7 @@ import java.util.Set;
|
|||||||
/** Unit tests for {@link IpSecAlgorithm}. */
|
/** Unit tests for {@link IpSecAlgorithm}. */
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class IpSecAlgorithmTest {
|
public class IpSecAlgorithmTest {
|
||||||
private static final byte[] KEY_MATERIAL;
|
private static final byte[] KEY_MATERIAL;
|
||||||
|
|
||||||
|
|||||||
@@ -36,7 +36,7 @@ import org.junit.runner.RunWith;
|
|||||||
/** Unit tests for {@link IpSecConfig}. */
|
/** Unit tests for {@link IpSecConfig}. */
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class IpSecConfigTest {
|
public class IpSecConfigTest {
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
|
|||||||
@@ -52,7 +52,7 @@ import java.net.UnknownHostException;
|
|||||||
/** Unit tests for {@link IpSecManager}. */
|
/** Unit tests for {@link IpSecManager}. */
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class IpSecManagerTest {
|
public class IpSecManagerTest {
|
||||||
|
|
||||||
private static final int TEST_UDP_ENCAP_PORT = 34567;
|
private static final int TEST_UDP_ENCAP_PORT = 34567;
|
||||||
|
|||||||
@@ -32,7 +32,7 @@ import org.junit.runner.RunWith;
|
|||||||
/** Unit tests for {@link IpSecTransform}. */
|
/** Unit tests for {@link IpSecTransform}. */
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class IpSecTransformTest {
|
public class IpSecTransformTest {
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
|
|||||||
@@ -47,7 +47,7 @@ private const val TEST_SUBID1 = 1
|
|||||||
private const val TEST_SUBID2 = 2
|
private const val TEST_SUBID2 = 2
|
||||||
|
|
||||||
@RunWith(DevSdkIgnoreRunner::class)
|
@RunWith(DevSdkIgnoreRunner::class)
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
class NetworkIdentityTest {
|
class NetworkIdentityTest {
|
||||||
private val mockContext = mock(Context::class.java)
|
private val mockContext = mock(Context::class.java)
|
||||||
|
|
||||||
|
|||||||
@@ -19,6 +19,7 @@ package android.net;
|
|||||||
import static com.android.testutils.DevSdkIgnoreRuleKt.SC_V2;
|
import static com.android.testutils.DevSdkIgnoreRuleKt.SC_V2;
|
||||||
|
|
||||||
import static org.junit.Assert.assertEquals;
|
import static org.junit.Assert.assertEquals;
|
||||||
|
import static org.mockito.Mockito.doCallRealMethod;
|
||||||
import static org.mockito.Mockito.when;
|
import static org.mockito.Mockito.when;
|
||||||
|
|
||||||
import android.Manifest;
|
import android.Manifest;
|
||||||
@@ -66,6 +67,10 @@ public class NetworkStatsAccessTest {
|
|||||||
when(mContext.getSystemServiceName(DevicePolicyManager.class))
|
when(mContext.getSystemServiceName(DevicePolicyManager.class))
|
||||||
.thenReturn(Context.DEVICE_POLICY_SERVICE);
|
.thenReturn(Context.DEVICE_POLICY_SERVICE);
|
||||||
when(mContext.getSystemService(Context.DEVICE_POLICY_SERVICE)).thenReturn(mDpm);
|
when(mContext.getSystemService(Context.DEVICE_POLICY_SERVICE)).thenReturn(mDpm);
|
||||||
|
if (mContext.getSystemService(DevicePolicyManager.class) == null) {
|
||||||
|
// Test is using mockito-extended
|
||||||
|
doCallRealMethod().when(mContext).getSystemService(DevicePolicyManager.class);
|
||||||
|
}
|
||||||
|
|
||||||
setHasCarrierPrivileges(false);
|
setHasCarrierPrivileges(false);
|
||||||
setIsDeviceOwner(false);
|
setIsDeviceOwner(false);
|
||||||
|
|||||||
@@ -60,7 +60,7 @@ import java.util.Random;
|
|||||||
|
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class NetworkStatsHistoryTest {
|
public class NetworkStatsHistoryTest {
|
||||||
private static final String TAG = "NetworkStatsHistoryTest";
|
private static final String TAG = "NetworkStatsHistoryTest";
|
||||||
|
|
||||||
|
|||||||
@@ -61,7 +61,7 @@ import java.util.Iterator;
|
|||||||
|
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class NetworkStatsTest {
|
public class NetworkStatsTest {
|
||||||
|
|
||||||
private static final String TEST_IFACE = "test0";
|
private static final String TEST_IFACE = "test0";
|
||||||
|
|||||||
@@ -70,7 +70,7 @@ private const val TEST_WIFI_KEY1 = "wifiKey1"
|
|||||||
private const val TEST_WIFI_KEY2 = "wifiKey2"
|
private const val TEST_WIFI_KEY2 = "wifiKey2"
|
||||||
|
|
||||||
@RunWith(DevSdkIgnoreRunner::class)
|
@RunWith(DevSdkIgnoreRunner::class)
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
class NetworkTemplateTest {
|
class NetworkTemplateTest {
|
||||||
private val mockContext = mock(Context::class.java)
|
private val mockContext = mock(Context::class.java)
|
||||||
private val mockWifiInfo = mock(WifiInfo::class.java)
|
private val mockWifiInfo = mock(WifiInfo::class.java)
|
||||||
|
|||||||
@@ -51,7 +51,7 @@ import org.mockito.MockitoAnnotations;
|
|||||||
|
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class NsdManagerTest {
|
public class NsdManagerTest {
|
||||||
|
|
||||||
static final int PROTOCOL = NsdManager.PROTOCOL_DNS_SD;
|
static final int PROTOCOL = NsdManager.PROTOCOL_DNS_SD;
|
||||||
|
|||||||
@@ -42,7 +42,7 @@ import java.util.Map;
|
|||||||
|
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class NsdServiceInfoTest {
|
public class NsdServiceInfoTest {
|
||||||
|
|
||||||
public final static InetAddress LOCALHOST;
|
public final static InetAddress LOCALHOST;
|
||||||
|
|||||||
@@ -89,7 +89,7 @@ import java.util.Set;
|
|||||||
public class IpSecServiceParameterizedTest {
|
public class IpSecServiceParameterizedTest {
|
||||||
@Rule
|
@Rule
|
||||||
public final DevSdkIgnoreRule mIgnoreRule = new DevSdkIgnoreRule(
|
public final DevSdkIgnoreRule mIgnoreRule = new DevSdkIgnoreRule(
|
||||||
Build.VERSION_CODES.R /* ignoreClassUpTo */);
|
Build.VERSION_CODES.S_V2 /* ignoreClassUpTo */);
|
||||||
|
|
||||||
private static final int TEST_SPI = 0xD1201D;
|
private static final int TEST_SPI = 0xD1201D;
|
||||||
|
|
||||||
|
|||||||
@@ -54,7 +54,7 @@ import java.util.concurrent.ThreadLocalRandom;
|
|||||||
/** Unit tests for {@link IpSecService.RefcountedResource}. */
|
/** Unit tests for {@link IpSecService.RefcountedResource}. */
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class IpSecServiceRefcountedResourceTest {
|
public class IpSecServiceRefcountedResourceTest {
|
||||||
Context mMockContext;
|
Context mMockContext;
|
||||||
IpSecService.Dependencies mMockDeps;
|
IpSecService.Dependencies mMockDeps;
|
||||||
|
|||||||
@@ -75,7 +75,7 @@ import java.util.List;
|
|||||||
/** Unit tests for {@link IpSecService}. */
|
/** Unit tests for {@link IpSecService}. */
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class IpSecServiceTest {
|
public class IpSecServiceTest {
|
||||||
|
|
||||||
private static final int DROID_SPI = 0xD1201D;
|
private static final int DROID_SPI = 0xD1201D;
|
||||||
|
|||||||
@@ -23,6 +23,7 @@ import static org.junit.Assert.assertNotNull;
|
|||||||
import static org.mockito.ArgumentMatchers.anyInt;
|
import static org.mockito.ArgumentMatchers.anyInt;
|
||||||
import static org.mockito.ArgumentMatchers.anyString;
|
import static org.mockito.ArgumentMatchers.anyString;
|
||||||
import static org.mockito.Mockito.any;
|
import static org.mockito.Mockito.any;
|
||||||
|
import static org.mockito.Mockito.doCallRealMethod;
|
||||||
import static org.mockito.Mockito.doReturn;
|
import static org.mockito.Mockito.doReturn;
|
||||||
import static org.mockito.Mockito.eq;
|
import static org.mockito.Mockito.eq;
|
||||||
import static org.mockito.Mockito.mock;
|
import static org.mockito.Mockito.mock;
|
||||||
@@ -74,7 +75,7 @@ import java.util.Queue;
|
|||||||
// - test NSD_ON ENABLE/DISABLED listening
|
// - test NSD_ON ENABLE/DISABLED listening
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class NsdServiceTest {
|
public class NsdServiceTest {
|
||||||
|
|
||||||
static final int PROTOCOL = NsdManager.PROTOCOL_DNS_SD;
|
static final int PROTOCOL = NsdManager.PROTOCOL_DNS_SD;
|
||||||
@@ -114,6 +115,10 @@ public class NsdServiceTest {
|
|||||||
doReturn(MDnsManager.MDNS_SERVICE).when(mContext)
|
doReturn(MDnsManager.MDNS_SERVICE).when(mContext)
|
||||||
.getSystemServiceName(MDnsManager.class);
|
.getSystemServiceName(MDnsManager.class);
|
||||||
doReturn(mMockMDnsM).when(mContext).getSystemService(MDnsManager.MDNS_SERVICE);
|
doReturn(mMockMDnsM).when(mContext).getSystemService(MDnsManager.MDNS_SERVICE);
|
||||||
|
if (mContext.getSystemService(MDnsManager.class) == null) {
|
||||||
|
// Test is using mockito-extended
|
||||||
|
doCallRealMethod().when(mContext).getSystemService(MDnsManager.class);
|
||||||
|
}
|
||||||
doReturn(true).when(mMockMDnsM).registerService(
|
doReturn(true).when(mMockMDnsM).registerService(
|
||||||
anyInt(), anyString(), anyString(), anyInt(), any(), anyInt());
|
anyInt(), anyString(), anyString(), anyInt(), any(), anyInt());
|
||||||
doReturn(true).when(mMockMDnsM).stopOperation(anyInt());
|
doReturn(true).when(mMockMDnsM).stopOperation(anyInt());
|
||||||
|
|||||||
@@ -16,8 +16,6 @@
|
|||||||
|
|
||||||
package com.android.server.ethernet;
|
package com.android.server.ethernet;
|
||||||
|
|
||||||
import static com.android.testutils.DevSdkIgnoreRuleKt.SC_V2;
|
|
||||||
|
|
||||||
import static org.junit.Assert.assertEquals;
|
import static org.junit.Assert.assertEquals;
|
||||||
import static org.junit.Assert.assertFalse;
|
import static org.junit.Assert.assertFalse;
|
||||||
import static org.junit.Assert.assertNotSame;
|
import static org.junit.Assert.assertNotSame;
|
||||||
@@ -55,16 +53,17 @@ import android.net.NetworkRequest;
|
|||||||
import android.net.StaticIpConfiguration;
|
import android.net.StaticIpConfiguration;
|
||||||
import android.net.ip.IpClientCallbacks;
|
import android.net.ip.IpClientCallbacks;
|
||||||
import android.net.ip.IpClientManager;
|
import android.net.ip.IpClientManager;
|
||||||
|
import android.os.Build;
|
||||||
import android.os.Handler;
|
import android.os.Handler;
|
||||||
import android.os.IBinder;
|
import android.os.IBinder;
|
||||||
import android.os.Looper;
|
import android.os.Looper;
|
||||||
import android.os.test.TestLooper;
|
import android.os.test.TestLooper;
|
||||||
|
|
||||||
import androidx.test.filters.SmallTest;
|
import androidx.test.filters.SmallTest;
|
||||||
import androidx.test.runner.AndroidJUnit4;
|
|
||||||
|
|
||||||
import com.android.net.module.util.InterfaceParams;
|
import com.android.net.module.util.InterfaceParams;
|
||||||
import com.android.testutils.DevSdkIgnoreRule;
|
import com.android.testutils.DevSdkIgnoreRule;
|
||||||
|
import com.android.testutils.DevSdkIgnoreRunner;
|
||||||
|
|
||||||
import org.junit.After;
|
import org.junit.After;
|
||||||
import org.junit.Before;
|
import org.junit.Before;
|
||||||
@@ -79,8 +78,9 @@ import java.util.concurrent.CompletableFuture;
|
|||||||
import java.util.concurrent.ExecutionException;
|
import java.util.concurrent.ExecutionException;
|
||||||
import java.util.concurrent.TimeUnit;
|
import java.util.concurrent.TimeUnit;
|
||||||
|
|
||||||
@RunWith(AndroidJUnit4.class)
|
|
||||||
@SmallTest
|
@SmallTest
|
||||||
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class EthernetNetworkFactoryTest {
|
public class EthernetNetworkFactoryTest {
|
||||||
private static final int TIMEOUT_MS = 2_000;
|
private static final int TIMEOUT_MS = 2_000;
|
||||||
private static final String TEST_IFACE = "test123";
|
private static final String TEST_IFACE = "test123";
|
||||||
@@ -608,7 +608,6 @@ public class EthernetNetworkFactoryTest {
|
|||||||
assertEquals(listener.expectOnResult(), TEST_IFACE);
|
assertEquals(listener.expectOnResult(), TEST_IFACE);
|
||||||
}
|
}
|
||||||
|
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(SC_V2) // TODO: Use to Build.VERSION_CODES.SC_V2 when available
|
|
||||||
@Test
|
@Test
|
||||||
public void testUpdateInterfaceAbortsOnConcurrentRemoveInterface() throws Exception {
|
public void testUpdateInterfaceAbortsOnConcurrentRemoveInterface() throws Exception {
|
||||||
initEthernetNetworkFactory();
|
initEthernetNetworkFactory();
|
||||||
@@ -617,7 +616,6 @@ public class EthernetNetworkFactoryTest {
|
|||||||
() -> mNetFactory.removeInterface(TEST_IFACE));
|
() -> mNetFactory.removeInterface(TEST_IFACE));
|
||||||
}
|
}
|
||||||
|
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(SC_V2) // TODO: Use to Build.VERSION_CODES.SC_V2 when available
|
|
||||||
@Test
|
@Test
|
||||||
public void testUpdateInterfaceAbortsOnConcurrentUpdateInterfaceLinkState() throws Exception {
|
public void testUpdateInterfaceAbortsOnConcurrentUpdateInterfaceLinkState() throws Exception {
|
||||||
initEthernetNetworkFactory();
|
initEthernetNetworkFactory();
|
||||||
@@ -626,7 +624,6 @@ public class EthernetNetworkFactoryTest {
|
|||||||
() -> mNetFactory.updateInterfaceLinkState(TEST_IFACE, false, NULL_LISTENER));
|
() -> mNetFactory.updateInterfaceLinkState(TEST_IFACE, false, NULL_LISTENER));
|
||||||
}
|
}
|
||||||
|
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(SC_V2) // TODO: Use to Build.VERSION_CODES.SC_V2 when available
|
|
||||||
@Test
|
@Test
|
||||||
public void testUpdateInterfaceCallsListenerCorrectlyOnConcurrentRequests() throws Exception {
|
public void testUpdateInterfaceCallsListenerCorrectlyOnConcurrentRequests() throws Exception {
|
||||||
initEthernetNetworkFactory();
|
initEthernetNetworkFactory();
|
||||||
|
|||||||
@@ -25,6 +25,7 @@ import static org.mockito.ArgumentMatchers.eq;
|
|||||||
import static org.mockito.ArgumentMatchers.isNull;
|
import static org.mockito.ArgumentMatchers.isNull;
|
||||||
import static org.mockito.Mockito.doReturn;
|
import static org.mockito.Mockito.doReturn;
|
||||||
import static org.mockito.Mockito.doThrow;
|
import static org.mockito.Mockito.doThrow;
|
||||||
|
import static org.mockito.Mockito.mock;
|
||||||
import static org.mockito.Mockito.never;
|
import static org.mockito.Mockito.never;
|
||||||
import static org.mockito.Mockito.reset;
|
import static org.mockito.Mockito.reset;
|
||||||
import static org.mockito.Mockito.verify;
|
import static org.mockito.Mockito.verify;
|
||||||
@@ -38,19 +39,21 @@ import android.net.EthernetNetworkUpdateRequest;
|
|||||||
import android.net.INetworkInterfaceOutcomeReceiver;
|
import android.net.INetworkInterfaceOutcomeReceiver;
|
||||||
import android.net.IpConfiguration;
|
import android.net.IpConfiguration;
|
||||||
import android.net.NetworkCapabilities;
|
import android.net.NetworkCapabilities;
|
||||||
|
import android.os.Build;
|
||||||
import android.os.Handler;
|
import android.os.Handler;
|
||||||
|
|
||||||
import androidx.test.filters.SmallTest;
|
import androidx.test.filters.SmallTest;
|
||||||
import androidx.test.runner.AndroidJUnit4;
|
|
||||||
|
import com.android.testutils.DevSdkIgnoreRule;
|
||||||
|
import com.android.testutils.DevSdkIgnoreRunner;
|
||||||
|
|
||||||
import org.junit.Before;
|
import org.junit.Before;
|
||||||
import org.junit.Test;
|
import org.junit.Test;
|
||||||
import org.junit.runner.RunWith;
|
import org.junit.runner.RunWith;
|
||||||
import org.mockito.Mock;
|
|
||||||
import org.mockito.MockitoAnnotations;
|
|
||||||
|
|
||||||
@RunWith(AndroidJUnit4.class)
|
|
||||||
@SmallTest
|
@SmallTest
|
||||||
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class EthernetServiceImplTest {
|
public class EthernetServiceImplTest {
|
||||||
private static final String TEST_IFACE = "test123";
|
private static final String TEST_IFACE = "test123";
|
||||||
private static final EthernetNetworkUpdateRequest UPDATE_REQUEST =
|
private static final EthernetNetworkUpdateRequest UPDATE_REQUEST =
|
||||||
@@ -68,14 +71,17 @@ public class EthernetServiceImplTest {
|
|||||||
.build();
|
.build();
|
||||||
private static final INetworkInterfaceOutcomeReceiver NULL_LISTENER = null;
|
private static final INetworkInterfaceOutcomeReceiver NULL_LISTENER = null;
|
||||||
private EthernetServiceImpl mEthernetServiceImpl;
|
private EthernetServiceImpl mEthernetServiceImpl;
|
||||||
@Mock private Context mContext;
|
private Context mContext;
|
||||||
@Mock private Handler mHandler;
|
private Handler mHandler;
|
||||||
@Mock private EthernetTracker mEthernetTracker;
|
private EthernetTracker mEthernetTracker;
|
||||||
@Mock private PackageManager mPackageManager;
|
private PackageManager mPackageManager;
|
||||||
|
|
||||||
@Before
|
@Before
|
||||||
public void setup() {
|
public void setup() {
|
||||||
MockitoAnnotations.initMocks(this);
|
mContext = mock(Context.class);
|
||||||
|
mHandler = mock(Handler.class);
|
||||||
|
mEthernetTracker = mock(EthernetTracker.class);
|
||||||
|
mPackageManager = mock(PackageManager.class);
|
||||||
doReturn(mPackageManager).when(mContext).getPackageManager();
|
doReturn(mPackageManager).when(mContext).getPackageManager();
|
||||||
mEthernetServiceImpl = new EthernetServiceImpl(mContext, mHandler, mEthernetTracker);
|
mEthernetServiceImpl = new EthernetServiceImpl(mContext, mHandler, mEthernetTracker);
|
||||||
mEthernetServiceImpl.mStarted.set(true);
|
mEthernetServiceImpl.mStarted.set(true);
|
||||||
|
|||||||
@@ -49,12 +49,14 @@ import android.net.IpConfiguration.ProxySettings;
|
|||||||
import android.net.LinkAddress;
|
import android.net.LinkAddress;
|
||||||
import android.net.NetworkCapabilities;
|
import android.net.NetworkCapabilities;
|
||||||
import android.net.StaticIpConfiguration;
|
import android.net.StaticIpConfiguration;
|
||||||
|
import android.os.Build;
|
||||||
import android.os.HandlerThread;
|
import android.os.HandlerThread;
|
||||||
import android.os.RemoteException;
|
import android.os.RemoteException;
|
||||||
|
|
||||||
import androidx.test.filters.SmallTest;
|
import androidx.test.filters.SmallTest;
|
||||||
import androidx.test.runner.AndroidJUnit4;
|
|
||||||
|
|
||||||
|
import com.android.testutils.DevSdkIgnoreRule;
|
||||||
|
import com.android.testutils.DevSdkIgnoreRunner;
|
||||||
import com.android.testutils.HandlerUtils;
|
import com.android.testutils.HandlerUtils;
|
||||||
|
|
||||||
import org.junit.After;
|
import org.junit.After;
|
||||||
@@ -69,7 +71,8 @@ import java.net.InetAddress;
|
|||||||
import java.util.ArrayList;
|
import java.util.ArrayList;
|
||||||
|
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@RunWith(AndroidJUnit4.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class EthernetTrackerTest {
|
public class EthernetTrackerTest {
|
||||||
private static final String TEST_IFACE = "test123";
|
private static final String TEST_IFACE = "test123";
|
||||||
private static final int TIMEOUT_MS = 1_000;
|
private static final int TIMEOUT_MS = 1_000;
|
||||||
|
|||||||
@@ -24,16 +24,18 @@ import static org.mockito.Mockito.when;
|
|||||||
import android.content.Context;
|
import android.content.Context;
|
||||||
import android.net.INetd;
|
import android.net.INetd;
|
||||||
import android.net.MacAddress;
|
import android.net.MacAddress;
|
||||||
|
import android.os.Build;
|
||||||
import android.os.Handler;
|
import android.os.Handler;
|
||||||
import android.os.test.TestLooper;
|
import android.os.test.TestLooper;
|
||||||
|
|
||||||
import androidx.test.filters.SmallTest;
|
import androidx.test.filters.SmallTest;
|
||||||
import androidx.test.runner.AndroidJUnit4;
|
|
||||||
|
|
||||||
import com.android.net.module.util.BaseNetdUnsolicitedEventListener;
|
import com.android.net.module.util.BaseNetdUnsolicitedEventListener;
|
||||||
import com.android.net.module.util.IBpfMap;
|
import com.android.net.module.util.IBpfMap;
|
||||||
import com.android.net.module.util.InterfaceParams;
|
import com.android.net.module.util.InterfaceParams;
|
||||||
import com.android.net.module.util.Struct.U32;
|
import com.android.net.module.util.Struct.U32;
|
||||||
|
import com.android.testutils.DevSdkIgnoreRule;
|
||||||
|
import com.android.testutils.DevSdkIgnoreRunner;
|
||||||
|
|
||||||
import org.junit.Before;
|
import org.junit.Before;
|
||||||
import org.junit.Test;
|
import org.junit.Test;
|
||||||
@@ -42,8 +44,9 @@ import org.mockito.ArgumentCaptor;
|
|||||||
import org.mockito.Mock;
|
import org.mockito.Mock;
|
||||||
import org.mockito.MockitoAnnotations;
|
import org.mockito.MockitoAnnotations;
|
||||||
|
|
||||||
@RunWith(AndroidJUnit4.class)
|
|
||||||
@SmallTest
|
@SmallTest
|
||||||
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public final class BpfInterfaceMapUpdaterTest {
|
public final class BpfInterfaceMapUpdaterTest {
|
||||||
private static final int TEST_INDEX = 1;
|
private static final int TEST_INDEX = 1;
|
||||||
private static final int TEST_INDEX2 = 2;
|
private static final int TEST_INDEX2 = 2;
|
||||||
|
|||||||
@@ -27,9 +27,11 @@ import android.net.IpConfiguration.ProxySettings;
|
|||||||
import android.net.LinkAddress;
|
import android.net.LinkAddress;
|
||||||
import android.net.ProxyInfo;
|
import android.net.ProxyInfo;
|
||||||
import android.net.StaticIpConfiguration;
|
import android.net.StaticIpConfiguration;
|
||||||
|
import android.os.Build;
|
||||||
import android.util.ArrayMap;
|
import android.util.ArrayMap;
|
||||||
|
|
||||||
import androidx.test.runner.AndroidJUnit4;
|
import com.android.testutils.DevSdkIgnoreRule;
|
||||||
|
import com.android.testutils.DevSdkIgnoreRunner;
|
||||||
|
|
||||||
import org.junit.Test;
|
import org.junit.Test;
|
||||||
import org.junit.runner.RunWith;
|
import org.junit.runner.RunWith;
|
||||||
@@ -46,7 +48,8 @@ import java.util.Arrays;
|
|||||||
/**
|
/**
|
||||||
* Unit tests for {@link IpConfigStore}
|
* Unit tests for {@link IpConfigStore}
|
||||||
*/
|
*/
|
||||||
@RunWith(AndroidJUnit4.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
|
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.S_V2)
|
||||||
public class IpConfigStoreTest {
|
public class IpConfigStoreTest {
|
||||||
private static final int KEY_CONFIG = 17;
|
private static final int KEY_CONFIG = 17;
|
||||||
private static final String IFACE_1 = "eth0";
|
private static final String IFACE_1 = "eth0";
|
||||||
|
|||||||
@@ -29,6 +29,7 @@ import static android.net.NetworkStats.TAG_NONE;
|
|||||||
import static android.net.NetworkStats.UID_ALL;
|
import static android.net.NetworkStats.UID_ALL;
|
||||||
|
|
||||||
import static com.android.server.net.NetworkStatsFactory.kernelToTag;
|
import static com.android.server.net.NetworkStatsFactory.kernelToTag;
|
||||||
|
import static com.android.testutils.DevSdkIgnoreRuleKt.SC_V2;
|
||||||
|
|
||||||
import static org.junit.Assert.assertEquals;
|
import static org.junit.Assert.assertEquals;
|
||||||
import static org.junit.Assert.fail;
|
import static org.junit.Assert.fail;
|
||||||
@@ -38,7 +39,6 @@ import android.content.res.Resources;
|
|||||||
import android.net.NetworkStats;
|
import android.net.NetworkStats;
|
||||||
import android.net.TrafficStats;
|
import android.net.TrafficStats;
|
||||||
import android.net.UnderlyingNetworkInfo;
|
import android.net.UnderlyingNetworkInfo;
|
||||||
import android.os.Build;
|
|
||||||
|
|
||||||
import androidx.test.InstrumentationRegistry;
|
import androidx.test.InstrumentationRegistry;
|
||||||
import androidx.test.filters.SmallTest;
|
import androidx.test.filters.SmallTest;
|
||||||
@@ -67,7 +67,7 @@ import java.io.OutputStream;
|
|||||||
/** Tests for {@link NetworkStatsFactory}. */
|
/** Tests for {@link NetworkStatsFactory}. */
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@SmallTest
|
@SmallTest
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(SC_V2)
|
||||||
public class NetworkStatsFactoryTest extends NetworkStatsBaseTest {
|
public class NetworkStatsFactoryTest extends NetworkStatsBaseTest {
|
||||||
private static final String CLAT_PREFIX = "v4-";
|
private static final String CLAT_PREFIX = "v4-";
|
||||||
|
|
||||||
|
|||||||
@@ -19,6 +19,8 @@ package com.android.server.net;
|
|||||||
import static android.telephony.TelephonyDisplayInfo.OVERRIDE_NETWORK_TYPE_NONE;
|
import static android.telephony.TelephonyDisplayInfo.OVERRIDE_NETWORK_TYPE_NONE;
|
||||||
import static android.telephony.TelephonyDisplayInfo.OVERRIDE_NETWORK_TYPE_NR_NSA;
|
import static android.telephony.TelephonyDisplayInfo.OVERRIDE_NETWORK_TYPE_NR_NSA;
|
||||||
|
|
||||||
|
import static com.android.testutils.DevSdkIgnoreRuleKt.SC_V2;
|
||||||
|
|
||||||
import static org.junit.Assert.assertEquals;
|
import static org.junit.Assert.assertEquals;
|
||||||
import static org.junit.Assert.assertNotNull;
|
import static org.junit.Assert.assertNotNull;
|
||||||
import static org.junit.Assert.fail;
|
import static org.junit.Assert.fail;
|
||||||
@@ -37,7 +39,6 @@ import android.annotation.NonNull;
|
|||||||
import android.annotation.Nullable;
|
import android.annotation.Nullable;
|
||||||
import android.app.usage.NetworkStatsManager;
|
import android.app.usage.NetworkStatsManager;
|
||||||
import android.content.Context;
|
import android.content.Context;
|
||||||
import android.os.Build;
|
|
||||||
import android.os.Looper;
|
import android.os.Looper;
|
||||||
import android.os.Parcel;
|
import android.os.Parcel;
|
||||||
import android.telephony.SubscriptionManager;
|
import android.telephony.SubscriptionManager;
|
||||||
@@ -63,7 +64,7 @@ import java.util.concurrent.Executor;
|
|||||||
import java.util.concurrent.Executors;
|
import java.util.concurrent.Executors;
|
||||||
|
|
||||||
@RunWith(DevSdkIgnoreRunner.class)
|
@RunWith(DevSdkIgnoreRunner.class)
|
||||||
@DevSdkIgnoreRule.IgnoreUpTo(Build.VERSION_CODES.R)
|
@DevSdkIgnoreRule.IgnoreUpTo(SC_V2)
|
||||||
public final class NetworkStatsSubscriptionsMonitorTest {
|
public final class NetworkStatsSubscriptionsMonitorTest {
|
||||||
private static final int TEST_SUBID1 = 3;
|
private static final int TEST_SUBID1 = 3;
|
||||||
private static final int TEST_SUBID2 = 5;
|
private static final int TEST_SUBID2 = 5;
|
||||||
|
|||||||
Reference in New Issue
Block a user