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

Commit 779678f5 authored by Jeff Tinker's avatar Jeff Tinker
Browse files

VtsHalDrmV1_0Target test failure 

The DrmHalVendorPluginTest.RestoreKeysClosedSession test wasn't
taking into account the policy of the asset, so both streaming
and offline assets were included in the test which is incorrect.

Test: drm vts test passes

bug:37358379
Change-Id: I41116e77a1ae306bb1a484e3714360c8553590f0
parent 04b086dd

drm/1.0/vts/functional/drm_hal_vendor_test.cpp

+11 −8
Original line number Diff line number Diff line
@@ -591,8 +591,10 @@ TEST_P(DrmHalVendorPluginTest, RestoreKeysNull) { 
 */

TEST_P(DrmHalVendorPluginTest, RestoreKeysClosedSession) {

    for (auto config : contentConfigurations) {

        if (config.policy.allowOffline) {

            auto sessionId = openSession();

        hidl_vec<uint8_t> keySetId = loadKeys(sessionId, config);

            hidl_vec<uint8_t> keySetId =

                    loadKeys(sessionId, config, KeyType::OFFLINE);

            EXPECT_NE(0u, keySetId.size());

            closeSession(sessionId);

            sessionId = openSession();
@@ -601,6 +603,7 @@ TEST_P(DrmHalVendorPluginTest, RestoreKeysClosedSession) { 
            EXPECT_EQ(Status::ERROR_DRM_SESSION_NOT_OPENED, status);

        }

    }

}



/**

 * Test that the plugin either doesn't support getting