tree 91020c5c3ee51e1660802e5d69933cb4a856a233
parent ca7cebc1f65a58fcdf2646b3583bc18a09f0ec4d
author Lorenzo Colitti <lorenzo@google.com> 1605510344 +0900
committer Lorenzo Colitti <lorenzo@google.com> 1611883151 +0000

Simplify MockVpn.

This CL removes four methods in MockVpn by slightly changing the
test code to leverage the actual methods implemented by the
(production) Vpn superclass.

This works because setting mInterface results in
isRunningLocked() returning true, which makes a number of methods
behave as if the VPN is connected (which is what the test
expects).

The more realistic behaviour exposes a minor bug in the treatment
of underlying networks. Add a TODO to fix it.

Bug: 172870110
Test: test-only change
Change-Id: I49421183538ba61ca790af71e309ece36b653bf9
Merged-In: I49421183538ba61ca790af71e309ece36b653bf9