From e3509d455b5ada4083cceac1e6afff6d3cd14aa1 Mon Sep 17 00:00:00 2001 From: Roshan Pius Date: Tue, 21 Mar 2017 15:44:40 -0700 Subject: [PATCH] supplicant(vts): Compilation fixes Fixed couple of compilation failures from .hal changes and made change to stop/start framework during tests. Bug: 33457575 Test: mmma -j32 hardware/interfaces/wifi/supplicant/1.0/vts/functional Change-Id: I1bac68bd71e92b025acdebd0fef8834b826bcfcb --- wifi/supplicant/1.0/vts/Android.mk | 2 ++ .../1.0/vts/functional/supplicant_hidl_test_utils.cpp | 4 ++-- .../vts/functional/supplicant_sta_iface_hidl_test.cpp | 11 +++++++++-- 3 files changed, 13 insertions(+), 4 deletions(-) create mode 100644 wifi/supplicant/1.0/vts/Android.mk diff --git a/wifi/supplicant/1.0/vts/Android.mk b/wifi/supplicant/1.0/vts/Android.mk new file mode 100644 index 0000000000..6361f9b8ae --- /dev/null +++ b/wifi/supplicant/1.0/vts/Android.mk @@ -0,0 +1,2 @@ +LOCAL_PATH := $(call my-dir) +include $(call all-makefiles-under,$(LOCAL_PATH)) diff --git a/wifi/supplicant/1.0/vts/functional/supplicant_hidl_test_utils.cpp b/wifi/supplicant/1.0/vts/functional/supplicant_hidl_test_utils.cpp index 1fcfc8c723..df4bfa95ca 100644 --- a/wifi/supplicant/1.0/vts/functional/supplicant_hidl_test_utils.cpp +++ b/wifi/supplicant/1.0/vts/functional/supplicant_hidl_test_utils.cpp @@ -136,14 +136,14 @@ class ServiceNotificationListener : public IServiceNotification { }; void stopWifiFramework() { - ASSERT_EQ(std::system("svc wifi disable"), 0); + ASSERT_EQ(std::system("stop"), 0); // TODO: Use some other mechanism to wait for the framework to // finish disabling. sleep(5); } void startWifiFramework() { - ASSERT_EQ(std::system("svc wifi enable"), 0); + ASSERT_EQ(std::system("start"), 0); // These tests don't care whether the framework // finished enabling or not. } diff --git a/wifi/supplicant/1.0/vts/functional/supplicant_sta_iface_hidl_test.cpp b/wifi/supplicant/1.0/vts/functional/supplicant_sta_iface_hidl_test.cpp index c2a58b6009..5abf4e04d9 100644 --- a/wifi/supplicant/1.0/vts/functional/supplicant_sta_iface_hidl_test.cpp +++ b/wifi/supplicant/1.0/vts/functional/supplicant_sta_iface_hidl_test.cpp @@ -104,11 +104,13 @@ class IfaceCallback : public ISupplicantStaIfaceCallback { } Return onDisconnected(const hidl_array& /* bssid */, bool /* locallyGenerated */, - uint32_t /* reasonCode */) override { + ISupplicantStaIfaceCallback::ReasonCode + /* reasonCode */) override { return Void(); } Return onAssociationRejected( - const hidl_array& /* bssid */, uint32_t /* statusCode */, + const hidl_array& /* bssid */, + ISupplicantStaIfaceCallback::StatusCode /* statusCode */, bool /*timedOut */) override { return Void(); } @@ -116,6 +118,11 @@ class IfaceCallback : public ISupplicantStaIfaceCallback { const hidl_array& /* bssid */) override { return Void(); } + Return onBssidChanged( + ISupplicantStaIfaceCallback::BssidChangeReason /* reason */, + const hidl_array& /* bssid */) override { + return Void(); + } Return onEapFailure() override { return Void(); } Return onWpsEventSuccess() override { return Void(); } Return onWpsEventFail(