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

Commit dc90e226 authored by Aaqib Ismail's avatar Aaqib Ismail
Browse files

Print out content of VehiclePropValue when test fails 

Currently when a test fails, it prints out a byte representation of the
object which is not helpful.

Bug: 277359330
Bug: 277359630
Bug: 277359374
Bug: 277359394
Bug: 277359419
Bug: 277359421
Bug: 277359396
Bug: 277359260

Test: atest FakeVehicleHardwareTest
Change-Id: I26c4260b2dbf5a71fbfe94e11c111b482337e6f5
parent 5638a351

automotive/vehicle/aidl/impl/fake_impl/hardware/test/FakeVehicleHardwareTest.cpp

+72 −62
Original line number Diff line number Diff line
@@ -37,6 +37,22 @@ 
#include <unordered_set>

#include <vector>



namespace aidl {

namespace android {

namespace hardware {

namespace automotive {

namespace vehicle {



void PrintTo(const VehiclePropValue& value, std::ostream* os) {

    *os << "\n( " << value.toString() << " )\n";

}



}  // namespace vehicle

}  // namespace automotive

}  // namespace hardware

}  // namespace android

}  // namespace aidl



namespace android {

namespace hardware {

namespace automotive {
@@ -1079,11 +1095,12 @@ TEST_F(FakeVehicleHardwareTest, testSetWaitForVhalAfterCarServiceCrash) { 
    ASSERT_EQ(events.size(), 1u);

    // Erase the timestamp for comparison.

    events[0].timestamp = 0;

    ASSERT_EQ(events[0], (VehiclePropValue{

    auto expectedValue = VehiclePropValue{

            .prop = toInt(VehicleProperty::AP_POWER_STATE_REQ),

            .status = VehiclePropertyStatus::AVAILABLE,

            .value.int32Values = {toInt(VehicleApPowerStateReq::ON), 0},

                         }));

    };

    ASSERT_EQ(events[0], expectedValue);

}



TEST_F(FakeVehicleHardwareTest, testGetObd2FreezeFrame) {
@@ -1340,7 +1357,7 @@ TEST_F(FakeVehicleHardwareTest, testSwitchUser) { 
    events = getChangedProperties();

    ASSERT_EQ(events.size(), static_cast<size_t>(1));

    events[0].timestamp = 0;

    ASSERT_EQ(events[0], (VehiclePropValue{

    auto expectedValue = VehiclePropValue{

            .areaId = 0,

            .prop = toInt(VehicleProperty::SWITCH_USER),

            .value.int32Values =
@@ -1352,7 +1369,8 @@ TEST_F(FakeVehicleHardwareTest, testSwitchUser) { 
                            // SUCCESS

                            1,

                    },

                         }));

    };

    ASSERT_EQ(events[0], expectedValue);

}



TEST_F(FakeVehicleHardwareTest, testCreateUser) {
@@ -1396,7 +1414,7 @@ TEST_F(FakeVehicleHardwareTest, testCreateUser) { 
    events = getChangedProperties();

    ASSERT_EQ(events.size(), static_cast<size_t>(1));

    events[0].timestamp = 0;

    ASSERT_EQ(events[0], (VehiclePropValue{

    auto expectedValue = VehiclePropValue{

            .areaId = 0,

            .prop = toInt(VehicleProperty::CREATE_USER),

            .value.int32Values =
@@ -1406,7 +1424,8 @@ TEST_F(FakeVehicleHardwareTest, testCreateUser) { 
                            // SUCCESS

                            1,

                    },

                         }));

    };

    ASSERT_EQ(events[0], expectedValue);

}



TEST_F(FakeVehicleHardwareTest, testInitialUserInfo) {
@@ -1438,11 +1457,12 @@ TEST_F(FakeVehicleHardwareTest, testInitialUserInfo) { 
    auto events = getChangedProperties();

    ASSERT_EQ(events.size(), static_cast<size_t>(1));

    events[0].timestamp = 0;

    EXPECT_EQ(events[0], (VehiclePropValue{

    auto expectedValue = VehiclePropValue{

            .areaId = 0,

            .prop = toInt(VehicleProperty::INITIAL_USER_INFO),

            .value.int32Values = {3, 1, 11},

                         }));

    };

    EXPECT_EQ(events[0], expectedValue);



    // Try to get create_user again, should return default value.

    clearChangedProperties();
@@ -1452,7 +1472,7 @@ TEST_F(FakeVehicleHardwareTest, testInitialUserInfo) { 
    events = getChangedProperties();

    ASSERT_EQ(events.size(), static_cast<size_t>(1));

    events[0].timestamp = 0;

    EXPECT_EQ(events[0], (VehiclePropValue{

    expectedValue = VehiclePropValue{

            .areaId = 0,

            .prop = toInt(VehicleProperty::INITIAL_USER_INFO),

            .value.int32Values =
@@ -1467,7 +1487,8 @@ TEST_F(FakeVehicleHardwareTest, testInitialUserInfo) { 
                            0,

                    },

            .value.stringValue = "||",

                         }));

    };

    EXPECT_EQ(events[0], expectedValue);

}



TEST_F(FakeVehicleHardwareTest, testDumpAllProperties) {
@@ -2580,18 +2601,7 @@ TEST_F(FakeVehicleHardwareTest, testSetHvacTemperatureValueSuggestion) { 
        EXPECT_EQ(events.size(), static_cast<size_t>(1));

        events[0].timestamp = 0;



        EXPECT_EQ(events[0], (tc.expectedValuesToGet[0]))

                << "Failed Test: " << tc.name << "\n"

                << "Received - prop: " << events[0].prop << ", areaId: " << events[0].areaId

                << ", floatValues: {" << events[0].value.floatValues[0] << ", "

                << events[0].value.floatValues[1] << ", " << events[0].value.floatValues[2] << ", "

                << events[0].value.floatValues[3] << "}\n"

                << "Expected - prop: " << tc.expectedValuesToGet[0].prop

                << ", areaId: " << tc.expectedValuesToGet[0].areaId << ", floatValues: {"

                << tc.expectedValuesToGet[0].value.floatValues[0] << ", "

                << tc.expectedValuesToGet[0].value.floatValues[1] << ", "

                << tc.expectedValuesToGet[0].value.floatValues[2] << ", "

                << tc.expectedValuesToGet[0].value.floatValues[3] << "}\n";

        EXPECT_EQ(events[0], tc.expectedValuesToGet[0]);

        clearChangedProperties();

    }

}