diff options
author | Xiao Ma <xiaom@google.com> | 2020-04-16 05:56:46 +0000 |
---|---|---|
committer | Xiao Ma <xiaom@google.com> | 2020-04-16 06:43:37 +0000 |
commit | bfe73b182d85930ea04d0c2cdce2c73674fc3b6f (patch) | |
tree | 000824460d58cc6d8901bf74840f365efd89c6a0 /tests | |
parent | 73ecfa71e9e35568b55e00745c2dc83cde44d2d1 (diff) |
Replace confirmConfiguration with updateLayer2Information.
Since updateLayer2Information has updated such BSSID, L2Key and
GroupHint information, we can check if the initial connection has
been established or L2 roaming happened with this method instead of
confirmConfiguration. Otherwise, it seems that we impose a calling
rule for the caller: *must* call updateLayer2Information first and
then confirmConfiguration followed. Ideally, we should deal with
this case only with one metohd.
Bug: 131797393
Test: atest FrameworksWifiTests
Merged-In: I623c83dbcc6b14ea59fda1d68e9b15f09fc64f06
Change-Id: I623c83dbcc6b14ea59fda1d68e9b15f09fc64f06
Diffstat (limited to 'tests')
-rw-r--r-- | tests/wifitests/src/com/android/server/wifi/ClientModeImplTest.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/wifitests/src/com/android/server/wifi/ClientModeImplTest.java b/tests/wifitests/src/com/android/server/wifi/ClientModeImplTest.java index 17fc1a7f8..66bde2fc0 100644 --- a/tests/wifitests/src/com/android/server/wifi/ClientModeImplTest.java +++ b/tests/wifitests/src/com/android/server/wifi/ClientModeImplTest.java @@ -3227,7 +3227,7 @@ public class ClientModeImplTest extends WifiBaseTest { verify(mWifiScoreCard).noteIpConfiguration(any()); ArgumentCaptor<Layer2InformationParcelable> captor = ArgumentCaptor.forClass(Layer2InformationParcelable.class); - verify(mIpClient).updateLayer2Information(captor.capture()); + verify(mIpClient, atLeastOnce()).updateLayer2Information(captor.capture()); final Layer2InformationParcelable info = captor.getValue(); assertEquals(info.l2Key, "Wad"); assertEquals(info.groupHint, "Gab"); |