From a0e13d6060362db037941db8973d92fb86575a4b Mon Sep 17 00:00:00 2001 From: Jerry Wong Date: Thu, 12 Nov 2020 14:50:03 -0800 Subject: [PATCH] supplicant(vts): Skip p2p tests on non-p2p devices as intended The logic to skip the p2p tests on non-p2p devices appear to have inadventently flipped since d6436731. Bug: 173134041 Test: atest VtsHalWifiSupplicantV1_1TargetTest Change-Id: I7945950e8483241e47e308a2c5ad97571d74a769 --- wifi/supplicant/1.1/vts/functional/supplicant_hidl_test.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/wifi/supplicant/1.1/vts/functional/supplicant_hidl_test.cpp b/wifi/supplicant/1.1/vts/functional/supplicant_hidl_test.cpp index 76d12d7efe..6ad4290795 100644 --- a/wifi/supplicant/1.1/vts/functional/supplicant_hidl_test.cpp +++ b/wifi/supplicant/1.1/vts/functional/supplicant_hidl_test.cpp @@ -75,7 +75,7 @@ TEST_P(SupplicantHidlTest, AddStaInterface) { * AddP2pInterface */ TEST_P(SupplicantHidlTest, AddP2pInterface) { - if (isP2pOn_) return; + if (!isP2pOn_) return; ISupplicant::IfaceInfo iface_info; iface_info.name = getP2pIfaceName(); iface_info.type = IfaceType::P2P; @@ -115,7 +115,7 @@ TEST_P(SupplicantHidlTest, RemoveStaInterface) { * RemoveP2pInterface */ TEST_P(SupplicantHidlTest, RemoveP2pInterface) { - if (isP2pOn_) return; + if (!isP2pOn_) return; ISupplicant::IfaceInfo iface_info; iface_info.name = getP2pIfaceName(); iface_info.type = IfaceType::P2P;