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

Commit b5cc0223 authored by Alisher Alikhodjaev's avatar Alisher Alikhodjaev Committed by Cherrypicker Worker
Browse files

Temporary disable one of the transmit checks 

The transmit without openning a channel passes on some platforms,
because a basic channel can be opened by external applications
and the state is maintained in the HAL.

Bug: 300502872
Test: run vts -m VtsHalSecureElementTargetTest
(cherry picked from https://android-review.googlesource.com/q/commit:d0ed43bb7eeb8c2cdc23d23d0d926e083322b7f3)
Merged-In: If727c613e8575802b99604f7242e16cf85fc97a0
Change-Id: If727c613e8575802b99604f7242e16cf85fc97a0
parent 1c91d9cc

secure_element/aidl/vts/VtsHalSecureElementTargetTest.cpp

+2 −0
Original line number Original line Diff line number Diff line
@@ -293,11 +293,13 @@ TEST_P(SecureElementAidl, transmit) { 
    std::vector<uint8_t> response;

    std::vector<uint8_t> response;

    LogicalChannelResponse logical_channel_response;

    LogicalChannelResponse logical_channel_response;





    /* Temporaly disable this check to clarify Basic Channel behavior (b/300502872)

    // Note: no channel is opened for this test

    // Note: no channel is opened for this test

    // transmit() will return an empty response with the error

    // transmit() will return an empty response with the error

    // code CHANNEL_NOT_AVAILABLE when the SE cannot be

    // code CHANNEL_NOT_AVAILABLE when the SE cannot be

    // communicated with.

    // communicated with.

    EXPECT_ERR(secure_element_->transmit(kDataApdu, &response));

    EXPECT_ERR(secure_element_->transmit(kDataApdu, &response));

    */





    EXPECT_OK(secure_element_->openLogicalChannel(kSelectableAid, 0x00, &logical_channel_response));

    EXPECT_OK(secure_element_->openLogicalChannel(kSelectableAid, 0x00, &logical_channel_response));

    EXPECT_GE(logical_channel_response.selectResponse.size(), 2u);

    EXPECT_GE(logical_channel_response.selectResponse.size(), 2u);