Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit bddf4257 authored by Gabriel Biren's avatar Gabriel Biren Committed by Gerrit Code Review
Browse files

Merge "Disable Wi-Fi framework during VTS" into main

parents 87e6b60e 46538346

wifi/hostapd/1.0/vts/functional/hostapd_hidl_test.cpp

+16 −2
Original line number Diff line number Diff line
@@ -48,6 +48,13 @@ class HostapdHidlTest 
    virtual void SetUp() override {

        wifi_instance_name_ = std::get<0>(GetParam());

        hostapd_instance_name_ = std::get<1>(GetParam());



        // Disable Wi-Fi framework to avoid interference

        isWifiEnabled_ = isWifiFrameworkEnabled();

        isScanAlwaysEnabled_ = isWifiScanAlwaysAvailable();

        toggleWifiFramework(false);

        toggleWifiScanAlwaysAvailable(false);



        stopSupplicantIfNeeded(wifi_instance_name_);

        startHostapdAndWaitForHidlService(wifi_instance_name_,

                                          hostapd_instance_name_);
@@ -58,9 +65,16 @@ class HostapdHidlTest 
    virtual void TearDown() override {

        HIDL_INVOKE_VOID_WITHOUT_ARGUMENTS(hostapd_, terminate);

        stopHostapd(wifi_instance_name_);



        // Restore Wi-Fi framework state

        toggleWifiFramework(isWifiEnabled_);

        toggleWifiScanAlwaysAvailable(isScanAlwaysEnabled_);

    }



   protected:

     bool isWifiEnabled_ = false;

     bool isScanAlwaysEnabled_ = false;



     std::string getPrimaryWlanIfaceName() {

        std::array<char, PROPERTY_VALUE_MAX> buffer;

        property_get("wifi.interface", buffer.data(), "wlan0");

wifi/hostapd/1.0/vts/functional/hostapd_hidl_test_utils.cpp

+21 −0
Original line number Diff line number Diff line
@@ -100,3 +100,24 @@ bool is_1_1(const sp<IHostapd>& hostapd) { 
        ::android::hardware::wifi::hostapd::V1_1::IHostapd::castFrom(hostapd);

    return hostapd_1_1.get() != nullptr;

}



void toggleWifiFramework(bool enable) {

    std::string cmd = "/system/bin/cmd wifi set-wifi-enabled ";

    cmd += enable ? "enabled" : "disabled";

    testing::checkSubstringInCommandOutput(cmd.c_str(), "X");

}



void toggleWifiScanAlwaysAvailable(bool enable) {

    std::string cmd = "/system/bin/cmd wifi set-scan-always-available ";

    cmd += enable ? "enabled" : "disabled";

    testing::checkSubstringInCommandOutput(cmd.c_str(), "X");

}



bool isWifiFrameworkEnabled() {

    return testing::checkSubstringInCommandOutput("/system/bin/cmd wifi status", "Wifi is enabled");

}



bool isWifiScanAlwaysAvailable() {

    return testing::checkSubstringInCommandOutput("/system/bin/cmd wifi status",

                                                  "Wifi scanning is always available");

}

wifi/hostapd/1.0/vts/functional/hostapd_hidl_test_utils.h

+5 −2
Original line number Diff line number Diff line
@@ -17,14 +17,17 @@ 
#ifndef HOSTAPD_HIDL_TEST_UTILS_H

#define HOSTAPD_HIDL_TEST_UTILS_H



#include <VtsCoreUtil.h>

#include <android/hardware/wifi/hostapd/1.0/IHostapd.h>

#include <android/hardware/wifi/hostapd/1.1/IHostapd.h>



// Used to stop the android wifi framework before every test.

void stopWifiFramework(const std::string& instance_name);

void startWifiFramework(const std::string& instance_name);

void stopSupplicantIfNeeded(const std::string& instance_name);

void stopHostapd(const std::string& instance_name);

void toggleWifiFramework(bool enable);

void toggleWifiScanAlwaysAvailable(bool enable);

bool isWifiFrameworkEnabled();

bool isWifiScanAlwaysAvailable();

// Used to configure the chip, driver and start wpa_hostapd before every

// test.

void startHostapdAndWaitForHidlService(