Commit 402ae22
authored
[tests] Fix network interface tests for Android 30 (#4868)
Context: https://developer.android.com/preview/behavior-changes-11#netlink-mac
Context: https://developer.android.com/reference/java/net/NetworkInterface#getNetworkInterfaces()
If an application targets Android 11/API-30 (and later), then the
number of reported network interfaces as well as the amount of
information about those interfaces includes less detail.
First of all, only interfaces with associated IP addresses will be
reported by both the `NetworkInterface` Java API and the
[**getifaddrs**(3)][0] native libc API:
* Application targetting API-28 on API-30 device:
* name: wlan0; type: Ethernet
* name: dummy0; type: Ethernet
* name: lo; type: Loopback
* name: hwsim0; type: Unknown
* name: ip6tnl0; type: Tunnel
* name: sit0; type: Tunnel
* name: ip6_vti0; type: Tunnel
* name: ip_vti0; type: Tunnel
* Application targetting API-30 on API-30 device:
* name: wlan0; type: 0
* name: dummy0; type: 0
* name: lo; type: 0
Second of all, as can be seen above, device information does not
include the type of interface. This bit of information is what the
`DotNetInterfacesShouldEqualJavaInterfaces()` test relied on so far.
That change breaks `DotNetInterfacesShouldEqualJavaInterfaces()`:
DotNetInterfacesShouldEqualJavaInterfaces
Mono interfaces:
[InterfaceInfo: Name=radio0, IsLoopback=False, IsUp=True, HardwareAddress=<>, Addresses=<fe80::d4d9:5aff:fe48:60be%11, fec0::d4d9:5aff:fe48:60be, fec0::397c:d582:6278:ca2c, 10.0.2.15>]
[InterfaceInfo: Name=wlan0, IsLoopback=False, IsUp=True, HardwareAddress=<>, Addresses=<fe80::e00a:8bb6:415b:4c97%9, fec0::f925:950b:4c31:eea7, fec0::315a:4cc4:796:6915, 10.0.2.16>]
[InterfaceInfo: Name=dummy0, IsLoopback=False, IsUp=True, HardwareAddress=<>, Addresses=<fe80::4c41:86ff:fe93:6354%2>]
[InterfaceInfo: Name=lo, IsLoopback=False, IsUp=True, HardwareAddress=<>, Addresses=<::1, 127.0.0.1>]
Java interfaces:
[InterfaceInfo: Name=dummy0, IsLoopback=False, IsUp=True, HardwareAddress=<>, Addresses=<fe80::4c41:86ff:fe93:6354%2>]
[InterfaceInfo: Name=wlan0, IsLoopback=False, IsUp=True, HardwareAddress=<>, Addresses=<fe80::e00a:8bb6:415b:4c97%9, fec0::f925:950b:4c31:eea7, fec0::315a:4cc4:796:6915, 10.0.2.16>]
[InterfaceInfo: Name=lo, IsLoopback=True, IsUp=True, HardwareAddress=<>, Addresses=<::1, 127.0.0.1>]
[InterfaceInfo: Name=radio0, IsLoopback=False, IsUp=True, HardwareAddress=<>, Addresses=<fe80::d4d9:5aff:fe48:60be%11, fec0::d4d9:5aff:fe48:60be, fec0::397c:d582:6278:ca2c, 10.0.2.15>]
Interface radio0: passed
Interface wlan0: passed
Interface dummy0: passed
[FAIL] : #8.3 (lo not found in Java interfaces)
Expected: True
But was: False
at System.NetTests.NetworkInterfacesTest.DotNetInterfacesShouldEqualJavaInterfaces ()
at (wrapper managed-to-native) System.Reflection.RuntimeMethodInfo.InternalInvoke(System.Reflection.RuntimeMethodInfo,object,object[],System.Exception&)
at (wrapper managed-to-native) System.Reflection.RuntimeMethodInfo.InternalInvoke(System.Reflection.RuntimeMethodInfo,object,object[],System.Exception&)
Note that both Java and Mono return the same number of interfaces, with
the same names, but the `lo` interface in Mono list is not marked as a
loopback (`IsLoopback=False`), while the corresponding Java entry
reports the interface is a loopback.
In order to account for the above Android changes, we now look not only
at the hardware type but also at its name in order to determine whether
the device is a loopback one or not.
[0]: https://www.man7.org/linux/man-pages/man3/getifaddrs.3.html1 parent 09218b5 commit 402ae22
1 file changed
+10
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
170 | 170 | | |
171 | 171 | | |
172 | 172 | | |
173 | | - | |
| 173 | + | |
174 | 174 | | |
175 | 175 | | |
176 | 176 | | |
| |||
188 | 188 | | |
189 | 189 | | |
190 | 190 | | |
| 191 | + | |
191 | 192 | | |
192 | 193 | | |
193 | | - | |
| 194 | + | |
194 | 195 | | |
195 | 196 | | |
196 | 197 | | |
| |||
220 | 221 | | |
221 | 222 | | |
222 | 223 | | |
| 224 | + | |
| 225 | + | |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
223 | 231 | | |
224 | 232 | | |
0 commit comments