summaryrefslogtreecommitdiff
path: root/tests/src
AgeCommit message (Collapse)Author
2019-05-21Move the NetworkStack tests to unit/ to add an integration test.Lorenzo Colitti
Test: m Change-Id: If638183f45956b60d694c859a759dd761c0598b2
2019-05-15Merge "Fix a bug where private DNS to v4-only servers don't validate."Chiachang Wang
2019-05-15Fix a bug where private DNS to v4-only servers don't validate.Chiachang Wang
Bug: 132670008 Test: Manual Test: atest NetworkStackTests Change-Id: I8ae6fa6f3800f5f6c9ee88ffe3b8ada34b96dee6 (cherry picked from commit aea0808d3b26dab24005afc39eb7ec4f4f333bae)
2019-05-14apf: Remove IPv6 NAT-T keepalive related codes and ApfTest improvementAaron Huang
Remove IPv6 related codes since we don't support IPv6 NAT-T keepalive as the kernel doesn't support IPv6 UDP encapsulation. Renaming keepaliveAck to keepaliveResponce since NAT-T keeaplive response is not an ack. Also, add generateV4NattKeepaliveFilters() if multicast filter is disabled. Verify incoming packet contains 1 byte payload but it is not 0xff will pass NAT-T keepalive filter. Bug: 33530442 Test: atest FrameworksNetTests atest NetworkStackTests Change-Id: Ifb9e088c0c8d71c763ebd9ad122103d0f97ca278
2019-05-10Dropping NAT-T keepalive packet from APFAaron Huang
Add NAT Traversal keepalive filter to APF program to drop NAT-T keepalive packets when NAT-T keepalive offload is starting. Bug: 33530442 Test: - atest NetworkStackTests - atest FrameworksNetTests Change-Id: I1c537485e11b31e5a6e0d8b7b6a1f396f9441746
2019-05-09Use async dns query to resolve all addressesChiachang Wang
Currently, it looks like private DNS server resolution uses OneAddressPerFamilyNetwork and only returns one server address. It should return all addresses. Use async dns api for this. Bug: 123435238 Test: atest NetworkStacktests Change-Id: I9f50da3c8c2e3b12b29bc8844291e4bf1559cd1f
2019-05-09Switch to positive naming for NetworkMonitor's Networks.Lorenzo Colitti
"mNonPrivateDnsBypassNetwork" is not very easy to understand. Bug: 122652057 Test: atest FrameworksNetTests NetworkStackTests Change-Id: If5801c81c6ef222b46f631bd6570dcd0fea465bd
2019-05-09Add tests for strict mode private DNS validation.Lorenzo Colitti
Test successful and failed validation, and updating the config. In order to do this, add a FakeDns class so we can change DNS responses dynamically while the test is running. Also a couple of minor fixes: 1. Make sure the DNS timeout is set. Before this CL, it was always 0. Not sure why. It does seem to be set to the default value (12500) when actually running on device. We didn't catch this because the only tests that use the timeout set it explicitly. 2. Make runNetworkTest a bit more realistic: always send NetworkCapabilities *before* calling notifyNetworkConnected. This is what ConnectivityService does. Bug: 122652057 Test: atest FrameworksNetTests NetworkStackTests Test: atest --generate-new-metrics 50 NetworkStackTests:com.android.server.connectivity.NetworkMonitorTest Change-Id: Ifd6694262501874f3261c864a049cb35c6afb9c8
2019-04-29Support adding NATT keepalive packet filterAaron Huang
Support adding NATT keepalive packet filter to APF filter. Generating APF program will be addressed in another CL. Bug: 33530442 Test: - atest NetworkStackTests - atest FrameworksNetTests Change-Id: I4961d5da343d8700600269632787c28112f0f9f1
2019-04-27Fix some checkstyle nits in DhcpPacketTest.Lorenzo Colitti
Use the Generated code suppression comment filter instead of the IndentationCheck suppression comment filter because the latter complains that '+' should be on a new line. Bug: 120584519 Bug: 127423755 Test: atest NetworkStackTests Test: atest FrameworksNetTests Change-Id: Ia9865c75f940e2fbb3aef374a297339e8ea2fd1b
2019-04-27Additional code for server name in DHCP packets.Lorenzo Colitti
- Add the hostname to DhcpResultsParcelable. - Don't store the server hostname if option overload is in use, as this is not valid. - Add unit tests. Bug: 120584519 Bug: 127423755 Test: atest NetworkStackTests Test: atest FrameworksNetTests Change-Id: I47d9d53d1fb58968322cc8b071a44fbc8f7156e1
2019-04-23Merge "Add a timeout for the DNS probe."Lorenzo Colitti
2019-04-23Freeze the networkstack-aidl-interfaces interfaceRemi NGUYEN VAN
This freezes the interface as of the latest beta build, not the tip of tree. IIpClient#setL2KeyAndGroupHint is not in the frozen definition in particular. Generated with: m networkstack-aidl-interfaces-freeze-api \ ipmemorystore-aidl-interfaces-freeze-api Test: flashed, booted, WiFi and captive portal working Bug: 128803828 Change-Id: Ideabe73fc93bbefca2d624ee9ca190cf31419424
2019-04-22Add a timeout for the DNS probe.Lorenzo Colitti
Bug: 129312219 Test: atest NetworkStackTests Test: atest --generate-new-metrics 50 NetworkStackTests:com.android.server.connectivity.NetworkMonitorTest Change-Id: Ib3ab9105d7ae39f551b51e8d5a04b9ec5e549655
2019-04-22Merge changes Id598ae1d,I475bd011Chalard Jean
* changes: Fix a possible crash when the listener is null Straighten AIDL interface for the memory store
2019-04-22Straighten AIDL interface for the memory storeChalard Jean
Some names were still wrong somehow, and the wrappers were missing. Test: NetworkStack & FrameworkNetTests Change-Id: I475bd011ad9bc714a07021a9dfd85c4876f8e9ad Merged-In: I475bd011ad9bc714a07021a9dfd85c4876f8e9ad
2019-04-19Merge changes from topic "StatsLog.write_migrate"Chiachang Wang
* changes: Java code for mainline modules to log to statsd. Migrate NetworkStack metrics off StatsLog.write
2019-04-18Migrate NetworkStack metrics off StatsLog.writeChiachang Wang
Now that the new metrics API can be used, NetworkStack metrics need to migrate off the legacy StatsLog.write. Bug: 130323000 Test: Test with ./out/host/linux-x86/bin/statsd_testdrive 121 to ensure log properly Test: Ensure NetworkStackStatsLog will be generated as expected Test: atest NetworkStackTest cherry-picking commit e512b269132. Change-Id: I8c70503cf1d9f1d06f30a1936d2d88857d0cff53 Merged-In: I1bfe856d771bf56fe33790aea22eceae2001921a
2019-04-17Move Setting constants for NetworkStackChiachang Wang
From mainline perspective, we should use android flag api instead of using Settings. Thus, move the definitions into NetworkStack and apply new flag design. Bug: 123167629 Test: atest NetworkStackTests Change-Id: I9602263f0bff5d8e942bd652de69ccfcb3034a2f
2019-04-10Merge "Add lifecycle tests to NetworkMonitorTest"Lorenzo Colitti
2019-04-10Merge "Move data stall definitions out from Setting"Chiachang Wang
2019-04-10Move data stall definitions out from SettingChiachang Wang
From mainline perspective, we should use android flag api instead of using Settings. Thus, move the definitions into NetworkStack. Bug:120013793 Test: atest NetworkStackTests SettingsBackupTest Change-Id: I8e1fb5b47fff3bf624131ba1f5732daabd991e6d Merged-In: I8e1fb5b47fff3bf624131ba1f5732daabd991e6d Merged-In: I0a2a1e5d3de14ea86556ca471e26064df3481412
2019-04-09Add lifecycle tests to NetworkMonitorTestLorenzo Colitti
- Ensure that NetworkMonitor quits when notifyNetworkDisconnected is called. - Ensure that BroadcastReceivers are registered when a captive portal is detected and unregistered when the network validates or when NetworkMonitor quits. Bug: 122164725 Test: atest FrameworksNetTests NetworkStackTests Test: atest --generate-new-metrics 10 NetworkStackTests:com.android.server.connectivity.NetworkMonitorTest Test: test fails if "Don't leak captive portal broadcast receivers." change reverted Change-Id: I1202626face33cc1e1ec02016b95da487b3335c8
2019-04-09Expose captive portal urls for configurationNiklas Lindgren
Carriers in Mainland China need to customize certain captive portal urls. The main issue is that google servers are not accessible in Mainland China. Added the following captive portal resources to be targeted for overlay. - config_captive_portal_http_url - config_captive_portal_https_url - config_captive_portal_fallback_urls (string-array) - config_captive_portal_fallback_probe_specs (string-array) These values can be customized for e g diffent countries Bug: 111819230 Test: atest FrameworksNetTests NetworkStackTests Test: Add a product RRO that targets a specific country code, insert a SIM card that matches that country code and check the log what URL is used. Change-Id: I38a294a5e14417f65e96e76235ea153c53c4d7cc
2019-04-05Add a mechanism to pass the L2Key and group hint to IpClientChalard Jean
Test: IpClientTest Change-Id: I399b20ce29a385a74971b7a7248a48cb4d66deb6
2019-04-02Make CaptivePortalProbeSpec and CaptivePortalProbeResult as a librarypaulhu
These two classes were added to @SystemApi because they are used both by NetworkMonitor and CaptivePortalLogin. However it turns out they are not needed in the framework, so having them as a library sounds better. Change-Id: Iadf77ec5952b6da8812dc6d006a39bd4e93d2bd9 Fix: 129433264 Test: atest NetworkStackTests FrameworksNetTests
2019-04-01[IPMS] Implement regular maintenancepaulhu
Implement regular maintenance of IpMemoryStoreService. Regular maintenance is scheduled for when the device is idle with access power and a minimum interval of one day. Bug: 113554482 Test: atest NetworkStackTests Change-Id: Id3985e30d12307fc2e9fcbe782caaf97a627cef3
2019-04-01Merge "Add DHCP address lease expiry in IpMemoryStore."Xiao Ma
2019-04-01Merge "Fix race when starting NetworkMonitor"Remi NGUYEN VAN
2019-04-01Fix race when starting NetworkMonitorRemi NGUYEN VAN
NetworkMonitor obtained LinkProperties and NetworkCapabilities via synchronous calls to ConnectivityManager after receiving an asynchronous notification, which is prone to races: the network could be gone before the LinkProperties/NetworkCapabilities can be fetched. Fix the race by passing LinkProperties/NetworkCapabilities directly to NetworkMonitor in the asynchronous notifications. Test: atest FrameworksNetTests NetworkStackTests Test: booted, WiFi works Bug: 129375892 Change-Id: I200ac7ca6ff79590b11c9be705f650c92fd3cb63
2019-03-29Add DHCP address lease expiry in IpMemoryStore.Xiao Ma
Bug:122710829 Test: atest FrameworksNetTests Change-Id: I643fe1231edcd18923514ab66c64a6cf83e69443
2019-03-25apf: Documentation improvements and code optimizations for keepalive filtersAaron Huang
Optimizing instruction to reduce code size of keepalive filters. Bug: 123992088 Test: atest NetworkStackTests Change-Id: I9ae55977c6d6740277ba55603ee957f33a97ecaa
2019-03-21Merge "Improve partial connectivity"Lucas Lin
2019-03-20Improve partial connectivitylucaslin
Improve the design and fix some nits. Bug: 113450764 Test: 1. Build pass 2. atest FrameworksNetTests 3. atest NetworkStackTests 4. Change captive_portal_https_url to https://invalid.com to simulate partial connectivity. Change-Id: Ia56645841d00d2ed8406cfeacb86a4a27fd58650
2019-03-18Move the IpMemoryStore to the network stack.Chalard Jean
Test: atest FrameworksNetTests Change-Id: Ic5bd6ff54b09a6fa92b6863f240a5b053011acb6
2019-03-16Remove aidl wrappers for framework parcelablesRemi NGUYEN VAN
The new @JavaOnlyStableParcelable annotation allows using the framework parcelables directly, which removes a lot of boilerplate. Includes new copy constructors for IpPrefix and LinkAddress so they can be copied easily as-is into parcelables. Test: atest FrameworksNetTests Test: booted, WiFi works Bug: 126477266 Change-Id: Icac8afe498d0b5ebf9a0d0b9eceb14d64a29b381
2019-03-12Check if network has partial connectivitylucaslin
In some networks, network validation may only get success result for http probe but fail result for https probe. For this kind of network, it may still work at some websites or apps, but user didn't know about that. In order to fix this issue, we will check if network has partial connectivity and notify user to make a choice if they want to use this partial connectivity or not. Bug: 113450764 Test: 1. Build pass. 2. Fake partial connectivity case for testing. 3. atest FrameworksNetTests 4. atest NetworkStackTests Change-Id: I69ed00ac4850904ff708c9fef22e148879a10e92
2019-03-11Merge changes I9dbbf904,Iebc174edRemi NGUYEN VAN
* changes: Move NetworkStackTests to presubmit Remove NetworkStack test dependency on testutils
2019-03-08Remove NetworkStack test dependency on testutilsRemi NGUYEN VAN
frameworks-base-testutils is close to unused in the tests, and has the side-effect of adding some unrelated tests to be run in NetworkStackTests like TestLooperTest. TestLooperTest is also failing in some cases. Bug: 127908503 Test: atest NetworkStackTests Change-Id: Iebc174ed1362e8c341d765129a7fc51f8ed1999d
2019-03-08Not to use any() to verify DataStallDetectionStatsChiachang Wang
Build a empty DataStallDetectionStats object instead of use any() to do the verification to prevent NPE while trying to write the metrics. Bug: 120452078 Test: atest NetworkStackTests Change-Id: Ib9f3e56195a539a5ca4826b4aad0e38945a9a3a7
2019-03-05Update description based on API feedbackChiachang Wang
Bug: 126883311 Test: Build pass atest NetworkStackTests Change-Id: If3d21829bbe98d125215abe96247ca7d5a66d1f3
2019-03-01Migrate frameworks/base/tests/testables and its users to androidx.testBrett Chabot
See go/jetpack-test-android-migration Test: atest TestablesTests NetworkStackTests FrameworksServicesTests SystemUITests FrameworksUiServicesTests ExtServicesUnitTests Exempt-From-Owner-Approval: already reviewed Change-Id: I083fcdaa71b503535aa2b3c257740f3ecf055373 Merged-In: Ib04d80954bd8536914d88c66b28e6632e60a6245
2019-02-26Add more test case and some clean upChiachang Wang
Bug: 120452078 Test: atest NetworkStackTests Change-Id: Ifb981ccdaa7818bf2bebec98e91979007222910b
2019-02-22Merge "Injecting data stall event to statsd"Chiachang Wang
2019-02-22Injecting data stall event to statsdChiachang Wang
Bug: 121185319 Bug: 120452078 Test: - Build and test against data stall code - atest NetworkStackTests Change-Id: I2ee16085e7861d38bcd203c98ffd682f86fdc867
2019-02-15Replace TcpSocketInfo with TcpKeepalivePacketDataParcelable in ApfTest.Xiao Ma
Test: atest NetworkStackTests Change-Id: I95fde70c07b398a2a2f734f35165870284a3959c
2019-02-15Merge "Remove NetworkMonitor dependency on ICaptivePortal"Remi NGUYEN VAN
2019-02-15Merge "Fix testApfFilterKeepaliveAck."Xiao Ma
2019-02-15Remove NetworkMonitor dependency on ICaptivePortalRemi NGUYEN VAN
ICaptivePortal is used in the framework and cannot be used as a dependency in NetworkMonitor, as the framework class takes precedence when linking. Also fix NetworkMonitorTest that was not verifying the right startCaptivePortalApp call. Test: atest FrameworksNetTests NetworkStackTests Bug: 124033493 Change-Id: I8e7bb79e50650ae182a2e4277fb49abf5fb6d910 Merged-In: Ib6a89e54312628662b130fbeec18d11e139f09fa
2019-02-14Fix testApfFilterKeepaliveAck.Xiao Ma
Apf program for TCP keep alive ACK also compares protocol field in IPv4 header and the total length of TCP keep alive ACK packet. Besides, should set the ACK flag in the the TCP header of keep alive ACK packet. Bug: 123992564 Test: atest NetworkStackTests Change-Id: I997cd6807a33f7e1af25566226d803dcc2e257dd