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

Commit 147ec300 authored by Ruchi Kandoi's avatar Ruchi Kandoi
Browse files

secure_element: vts: Fix openBasicChannel and transmit test case 

Status Response for basic channel can be only 2 bytes as well.
Update the test to test for 2 or more bytes.
Basic Channel if opened successfully should be closed.

Transmit test case should specify the channel number in the command.

Test: atest VtsHalSecureElementV1_0TargetTest
Bug: 109833790
Change-Id: I91c943b4297067f6ae6872efb038f1295f92f5c8
(cherry picked from commit d7700d216f0d854405bb9d99972c79f1af5b1b68)
parent 0be95494

secure_element/1.0/vts/functional/VtsHalSecureElementV1_0TargetTest.cpp

+3 −1
Original line number Diff line number Diff line
@@ -134,6 +134,7 @@ TEST_F(SecureElementHidlTest, transmit) { 
    EXPECT_LE((unsigned int)2, response.selectResponse.size());

    EXPECT_LE(1, response.channelNumber);

    std::vector<uint8_t> command = DATA_APDU;

    command[0] |= response.channelNumber;

    std::vector<uint8_t> transmitResponse;

    se_->transmit(command, [&transmitResponse](std::vector<uint8_t> res) {

        transmitResponse.resize(res.size());
@@ -168,7 +169,8 @@ TEST_F(SecureElementHidlTest, openBasicChannel) { 
                              }

                          });

    if (statusReturned == SecureElementStatus::SUCCESS) {

        EXPECT_LE((unsigned int)3, response.size());

        EXPECT_LE((unsigned int)2, response.size());

        se_->closeChannel(0);

        return;

    }

    EXPECT_EQ(SecureElementStatus::UNSUPPORTED_OPERATION, statusReturned);