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

Commit 976c9242 authored by Ivan Zaitsev's avatar Ivan Zaitsev
Browse files

Revert "Floss: Add new Suspend Topshim test" 

This reverts commit bfaa61f5.

Reason for revert: DroidMonitor: Potential culprit for Bug 220174310 - verifying through ABTD before revert submission. This is part of the standard investigation process, and does not mean your CL will be reverted.

Change-Id: I2d9f9c5e18e513a4c5458e707cfa8429ab3df7e2
parent bfaa61f5

system/blueberry/facade/topshim/facade.proto

+0 −3
Original line number Original line Diff line number Diff line 
syntax = "proto3";

syntax = "proto3";





import "google/protobuf/empty.proto";



package blueberry.facade.topshim;

package blueberry.facade.topshim;





service AdapterService {

service AdapterService {

  rpc FetchEvents(FetchEventsRequest) returns (stream FetchEventsResponse) {}

  rpc FetchEvents(FetchEventsRequest) returns (stream FetchEventsResponse) {}

  rpc ToggleStack(ToggleStackRequest) returns (ToggleStackResponse) {}

  rpc ToggleStack(ToggleStackRequest) returns (ToggleStackResponse) {}

  rpc SetDiscoveryMode(SetDiscoveryModeRequest) returns (SetDiscoveryModeResponse) {}

  rpc SetDiscoveryMode(SetDiscoveryModeRequest) returns (SetDiscoveryModeResponse) {}

  rpc ClearEventFilter(google.protobuf.Empty) returns (google.protobuf.Empty) {}

}

}





enum EventType {

enum EventType {

system/blueberry/tests/gd/rust/topshim/facade/automation_helper.py

+0 −5
Original line number Original line Diff line number Diff line
@@ -20,8 +20,6 @@ import grpc 
from blueberry.facade.topshim import facade_pb2

from blueberry.facade.topshim import facade_pb2

from blueberry.facade.topshim import facade_pb2_grpc

from blueberry.facade.topshim import facade_pb2_grpc





from google.protobuf import empty_pb2 as empty_proto







class AdapterAutomationHelper():

class AdapterAutomationHelper():

    # Timeout for async wait

    # Timeout for async wait
@@ -62,9 +60,6 @@ class AdapterAutomationHelper(): 
    async def verify_adapter_started(self):

    async def verify_adapter_started(self):

        await asyncio.wait_for(self.pending_future, AdapterAutomationHelper.DEFAULT_TIMEOUT)

        await asyncio.wait_for(self.pending_future, AdapterAutomationHelper.DEFAULT_TIMEOUT)





    async def clear_event_filter(self):

        await self.adapter_stub.ClearEventFilter(empty_proto.Empty())







class A2dpAutomationHelper():

class A2dpAutomationHelper():

    """Invoke gRPC on topshim for A2DP testing"""

    """Invoke gRPC on topshim for A2DP testing"""

system/blueberry/tests/gd/rust/topshim/facade/suspend_test.py

deleted100644 → 0
+0 −42
Original line number Original line Diff line number Diff line 
#!/usr/bin/env python3

#

#   Copyright 2021 - The Android Open Source Project

#

#   Licensed under the Apache License, Version 2.0 (the "License");

#   you may not use this file except in compliance with the License.

#   You may obtain a copy of the License at

#

#       http://www.apache.org/licenses/LICENSE-2.0

#

#   Unless required by applicable law or agreed to in writing, software

#   distributed under the License is distributed on an "AS IS" BASIS,

#   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

#   See the License for the specific language governing permissions and

#   limitations under the License.



import asyncio

from mobly import test_runner

from blueberry.tests.gd.rust.topshim.facade import topshim_base_test

from blueberry.tests.gd.rust.topshim.facade.automation_helper import AdapterAutomationHelper

import time





class SuspendTest(topshim_base_test.TopshimBaseTest):



    async def _test_verify_event_filter_cleared(self):

        self.dut_adapter = AdapterAutomationHelper(port=self.dut_port)

        event_loop = asyncio.get_running_loop()

        self.dut_adapter.fetch_events(event_loop)

        self.dut_adapter.pending_future = event_loop.create_future()

        await self.dut_adapter.clear_event_filter()

        #TODO(optedoblivion): Replace sleep with a call to LeGetRandom and synchronize on

        # the callback

        time.sleep(1)

        self.dut_adapter.event_handler.cancel()



    def test_verify_event_filter_cleared(self):

        asyncio.run(self._test_verify_event_filter_cleared())





if __name__ == "__main__":

    test_runner.main()

system/gd/cert/run_topshim

+2 −3
Original line number Original line Diff line number Diff line
@@ -20,8 +20,7 @@ import os 
import argparse

import argparse





TEST_SUITES = [

TEST_SUITES = [

    "blueberry.tests.gd.rust.topshim.facade.adapter_test",

    "blueberry.tests.gd.rust.topshim.facade.adapter_test"

    "blueberry.tests.gd.rust.topshim.facade.suspend_test"

]

]





SOONG_UI_BASH = 'build/soong/soong_ui.bash'

SOONG_UI_BASH = 'build/soong/soong_ui.bash'
@@ -136,7 +135,7 @@ def main(): 
    if not all(test_results):

    if not all(test_results):

        failures = [i for i, x in enumerate(test_results) if not x]

        failures = [i for i, x in enumerate(test_results) if not x]

        for index in failures:

        for index in failures:

            print('TEST FAILED: ' + TEST_SUITES[index])

            print('TEST FAILLED: ' + TEST_SUITES[index])

        sys.exit(0)

        sys.exit(0)

    print('TEST PASSED ' + str(len(test_results)) + ' tests were run')

    print('TEST PASSED ' + str(len(test_results)) + ' tests were run')

system/gd/rust/topshim/facade/src/adapter_service.rs

+0 −8
Original line number Original line Diff line number Diff line
@@ -3,7 +3,6 @@ 
use bt_topshim::btif;

use bt_topshim::btif;

use bt_topshim::btif::{BaseCallbacks, BaseCallbacksDispatcher, BluetoothInterface};

use bt_topshim::btif::{BaseCallbacks, BaseCallbacksDispatcher, BluetoothInterface};





use bt_topshim_facade_protobuf::empty::Empty;

use bt_topshim_facade_protobuf::facade::{

use bt_topshim_facade_protobuf::facade::{

    EventType, FetchEventsRequest, FetchEventsResponse, SetDiscoveryModeRequest,

    EventType, FetchEventsRequest, FetchEventsResponse, SetDiscoveryModeRequest,

    SetDiscoveryModeResponse, ToggleStackRequest, ToggleStackResponse,

    SetDiscoveryModeResponse, ToggleStackRequest, ToggleStackResponse,
@@ -119,11 +118,4 @@ impl AdapterService for AdapterServiceImpl { 
            sink.success(SetDiscoveryModeResponse::default()).await.unwrap();

            sink.success(SetDiscoveryModeResponse::default()).await.unwrap();

        })

        })

    }

    }



    fn clear_event_filter(&mut self, ctx: RpcContext<'_>, _req: Empty, sink: UnarySink<Empty>) {

        self.btif_intf.lock().unwrap().clear_event_filter();

        ctx.spawn(async move {

            sink.success(Empty::default()).await.unwrap();

        })

    }

}
 
}