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

Commit 465d8d7d authored by terryguan's avatar terryguan
Browse files

Add refreshPropertyConfigs to DumpResult proto 

refreshPropertyConfigs was not previously used in dumpresult proto
resulting in failing tests that require the argument.

Flag: EXEMPT test
Bug: 357135536
Test: atest android.car.apitest.CarPropertyManagerTest
Change-Id: I9c6dd3053aa54f37786edc3810a9342f5efb39cd
parent 201694df

automotive/vehicle/aidl/impl/grpc/GRPCVehicleHardware.cpp

+1 −0
Original line number Diff line number Diff line
@@ -276,6 +276,7 @@ DumpResult GRPCVehicleHardware::dump(const std::vector<std::string>& options) { 
    return {

            .callerShouldDumpState = protoDumpResult.caller_should_dump_state(),

            .buffer = protoDumpResult.buffer(),

            .refreshPropertyConfigs = protoDumpResult.refresh_property_configs(),

    };

}

automotive/vehicle/aidl/impl/grpc/GRPCVehicleProxyServer.cpp

+1 −0
Original line number Diff line number Diff line
@@ -226,6 +226,7 @@ GrpcVehicleProxyServer::GrpcVehicleProxyServer(std::vector<std::string> serverAd 
    auto dumpResult = mHardware->dump(dumpOptionStrings);

    result->set_caller_should_dump_state(dumpResult.callerShouldDumpState);

    result->set_buffer(dumpResult.buffer);

    result->set_refresh_property_configs(dumpResult.refreshPropertyConfigs);

    return ::grpc::Status::OK;

}

automotive/vehicle/aidl/impl/proto/android/hardware/automotive/vehicle/DumpResult.proto

+2 −0
Original line number Diff line number Diff line
@@ -25,4 +25,6 @@ message DumpResult { 
    bool caller_should_dump_state = 1;

    /* The dumped information for the caller to print. */

    string buffer = 2;

    /* To pass if DefaultVehicleHal should refresh the property configs. */

    bool refresh_property_configs = 3;

}