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

Commit cfff46ef authored by Ahmed ElArabawy's avatar Ahmed ElArabawy Committed by Presubmit Automerger Backend
Browse files

[automerge] Wifi: Fix VTS tests broken due to uprev to Vendor HAL 1.6 2p: 7f40eee6 

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

Bug: 220201314
Change-Id: I8b45e09c81d439d278c5aef0f6851f7bccde3b0d
parents bce4d0d2 7f40eee6

wifi/1.4/vts/functional/wifi_rtt_controller_hidl_test.cpp

+9 −2
Original line number Original line Diff line number Diff line
@@ -66,7 +66,9 @@ class WifiRttControllerHidlTest : public ::testing::TestWithParam<std::string> { 
        stopWifi(GetInstanceName());

        stopWifi(GetInstanceName());





        wifi_rtt_controller_ = getWifiRttController();

        wifi_rtt_controller_ = getWifiRttController();

        ASSERT_NE(nullptr, wifi_rtt_controller_.get());

        if (wifi_rtt_controller_.get() == nullptr) {

            GTEST_SKIP() << "Skipping this test since API is deprecated.";

        }





        // Check RTT support before we run the test.

        // Check RTT support before we run the test.

        std::pair<WifiStatus, RttCapabilities> status_and_caps;

        std::pair<WifiStatus, RttCapabilities> status_and_caps;
@@ -79,7 +81,7 @@ class WifiRttControllerHidlTest : public ::testing::TestWithParam<std::string> { 




    virtual void TearDown() override { stopWifi(GetInstanceName()); }

    virtual void TearDown() override { stopWifi(GetInstanceName()); }





    // A simple test implementation of WifiChipEventCallback.

    // A simple test implementation of WifiRttControllerEventCallback.

    class WifiRttControllerEventCallback

    class WifiRttControllerEventCallback

        : public ::testing::VtsHalHidlTargetCallbackBase<

        : public ::testing::VtsHalHidlTargetCallbackBase<

              WifiRttControllerHidlTest>,

              WifiRttControllerHidlTest>,
@@ -122,6 +124,11 @@ class WifiRttControllerHidlTest : public ::testing::TestWithParam<std::string> { 




        const auto& status_and_controller =

        const auto& status_and_controller =

            HIDL_INVOKE(wifi_chip, createRttController_1_4, wifi_sta_iface);

            HIDL_INVOKE(wifi_chip, createRttController_1_4, wifi_sta_iface);



        if (status_and_controller.first.code == WifiStatusCode::ERROR_NOT_SUPPORTED) {

            return nullptr;

        }



        EXPECT_EQ(WifiStatusCode::SUCCESS, status_and_controller.first.code);

        EXPECT_EQ(WifiStatusCode::SUCCESS, status_and_controller.first.code);

        EXPECT_NE(nullptr, status_and_controller.second.get());

        EXPECT_NE(nullptr, status_and_controller.second.get());

wifi/1.5/vts/functional/wifi_sta_iface_hidl_test.cpp

+5 −0
Original line number Original line Diff line number Diff line
@@ -94,6 +94,11 @@ TEST_P(WifiStaIfaceHidlTest, GetLinkLayerStats_1_5) { 
    // Retrieve link layer stats.

    // Retrieve link layer stats.

    const auto& status_and_stats =

    const auto& status_and_stats =

        HIDL_INVOKE(wifi_sta_iface_, getLinkLayerStats_1_5);

        HIDL_INVOKE(wifi_sta_iface_, getLinkLayerStats_1_5);



    if (status_and_stats.first.code == WifiStatusCode::ERROR_NOT_SUPPORTED) {

        GTEST_SKIP() << "Skipping this test since API is deprecated.";

    }



    EXPECT_EQ(WifiStatusCode::SUCCESS, status_and_stats.first.code);

    EXPECT_EQ(WifiStatusCode::SUCCESS, status_and_stats.first.code);

    EXPECT_GT(status_and_stats.second.timeStampInMs, 0u);

    EXPECT_GT(status_and_stats.second.timeStampInMs, 0u);

    // Try to create 2nd iface. If yes, it should fill in the duty cycle field.

    // Try to create 2nd iface. If yes, it should fill in the duty cycle field.

wifi/1.6/default/wifi_chip.cpp

+51 −2
Original line number Original line Diff line number Diff line
@@ -791,8 +791,57 @@ std::pair<WifiStatus, uint32_t> WifiChip::getCapabilitiesInternal() { 




std::pair<WifiStatus, std::vector<V1_0::IWifiChip::ChipMode>>

std::pair<WifiStatus, std::vector<V1_0::IWifiChip::ChipMode>>

WifiChip::getAvailableModesInternal() {

WifiChip::getAvailableModesInternal() {

    // Deprecated support -- use getAvailableModes_1_6.

    // Deprecated support -- use getAvailableModes_1_6 for more granular concurrency combinations.

    return {createWifiStatus(WifiStatusCode::ERROR_NOT_SUPPORTED), {}};

    std::vector<V1_0::IWifiChip::ChipMode> modes_1_0 = {};

    for (const auto& mode_1_6 : modes_) {

        std::vector<V1_0::IWifiChip::ChipIfaceCombination> combos_1_0;

        for (const auto& combo_1_6 : mode_1_6.availableCombinations) {

            std::vector<V1_0::IWifiChip::ChipIfaceCombinationLimit> limits_1_0;

            for (const auto& limit_1_6 : combo_1_6.limits) {

                std::vector<IfaceType> types_1_0;

                for (IfaceConcurrencyType type_1_6 : limit_1_6.types) {

                    switch (type_1_6) {

                        case IfaceConcurrencyType::STA:

                            types_1_0.push_back(IfaceType::STA);

                            break;

                        case IfaceConcurrencyType::AP:

                            types_1_0.push_back(IfaceType::AP);

                            break;

                        case IfaceConcurrencyType::AP_BRIDGED:

                            // Ignore AP_BRIDGED

                            break;

                        case IfaceConcurrencyType::P2P:

                            types_1_0.push_back(IfaceType::P2P);

                            break;

                        case IfaceConcurrencyType::NAN:

                            types_1_0.push_back(IfaceType::NAN);

                            break;

                    }

                }

                if (types_1_0.empty()) {

                    continue;

                }

                V1_0::IWifiChip::ChipIfaceCombinationLimit limit_1_0;

                limit_1_0.types = hidl_vec(types_1_0);

                limit_1_0.maxIfaces = limit_1_6.maxIfaces;

                limits_1_0.push_back(limit_1_0);

            }

            if (limits_1_0.empty()) {

                continue;

            }

            V1_0::IWifiChip::ChipIfaceCombination combo_1_0;

            combo_1_0.limits = hidl_vec(limits_1_0);

            combos_1_0.push_back(combo_1_0);

        }

        if (combos_1_0.empty()) {

            continue;

        }

        V1_0::IWifiChip::ChipMode mode_1_0;

        mode_1_0.id = mode_1_6.id;

        mode_1_0.availableCombinations = hidl_vec(combos_1_0);

        modes_1_0.push_back(mode_1_0);

    }

    return {createWifiStatus(WifiStatusCode::SUCCESS), modes_1_0};

}

}





WifiStatus WifiChip::configureChipInternal(

WifiStatus WifiChip::configureChipInternal(