commit 6ec42b402ce3c71b74e7d76f6b977c47d5075ca4
author Chris Ye <lzye@google.com> Wed Feb 23 23:55:18 2022 +0000
committer Presubmit Automerger Backend <android-build-presubmit-automerger-backend@system.gserviceaccount.com> Wed Feb 23 23:55:18 2022 +0000
tree 921285d1b809ba4effdca76b79c5be5ab39f0d72
parent 374c80a6e88e41a00124d054cbc957c07cb32b4a
parent 89b3de718d848a2809d819fc96fa74cb02d40c0f

[automerge] Change the expected AP iface index when dual STAs supported. 2p: 89b3de718d

Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/16993920

Bug: 220338166
Change-Id: Id54ccdd7f6fea5c849622cdf73439b298fe35fa8

wifi/1.6/default/tests/wifi_chip_unit_tests.cpp

diff --git a/wifi/1.6/default/tests/wifi_chip_unit_tests.cpp b/wifi/1.6/default/tests/wifi_chip_unit_tests.cpp
index eaab211..81117c5 100644
--- a/wifi/1.6/default/tests/wifi_chip_unit_tests.cpp
+++ b/wifi/1.6/default/tests/wifi_chip_unit_tests.cpp
@@ -846,13 +846,15 @@
 }
 
 TEST_F(WifiChip_MultiIfaceTest, CreateApStartsWithIdx1) {
+    // WifiChip_MultiIfaceTest iface combo: STAx3 + APx1
+    // When the HAL support dual STAs, AP should start with idx 2.
     findModeAndConfigureForIfaceType(IfaceConcurrencyType::STA);
     // First AP will be slotted to wlan1.
-    ASSERT_EQ(createIface(IfaceType::AP), "wlan1");
+    ASSERT_EQ(createIface(IfaceType::AP), "wlan2");
     // First STA will be slotted to wlan0.
     ASSERT_EQ(createIface(IfaceType::STA), "wlan0");
     // All further STA will be slotted to the remaining free indices.
-    ASSERT_EQ(createIface(IfaceType::STA), "wlan2");
+    ASSERT_EQ(createIface(IfaceType::STA), "wlan1");
     ASSERT_EQ(createIface(IfaceType::STA), "wlan3");
 }
 }  // namespace implementation