diff --git a/libwvdrmengine/cdm/test/cdm_extended_duration_test.cpp b/libwvdrmengine/cdm/test/cdm_extended_duration_test.cpp index 8207015c..b808ba7c 100644 --- a/libwvdrmengine/cdm/test/cdm_extended_duration_test.cpp +++ b/libwvdrmengine/cdm/test/cdm_extended_duration_test.cpp @@ -1316,7 +1316,11 @@ TEST_P(WvCdmStreamingNoPstTest, UsageTest) { if (initial_license_duration_remaining == kUnlimitedDurationValue) { EXPECT_EQ(license_duration_remaining, kUnlimitedDurationValue); - } else { + } else if (license_duration_remaining != kUnlimitedDurationValue) { + // Possible that the initial license duration was limited due to + // the finit rental duration; however, if the license has a soft + // rental duration, it will be considered unlimited after playback + // has begun. EXPECT_NEAR(initial_license_duration_remaining - license_duration_remaining, expected_seconds_since_license_received, kClockTolerance) << "initial_license_duration_remaining = " @@ -1343,9 +1347,7 @@ INSTANTIATE_TEST_CASE_P(Cdm, WvCdmStreamingNoPstTest, class WvCdmStreamingPstTest : public WvCdmExtendedDurationTest, public ::testing::WithParamInterface {}; -// TODO(b275651559): Re-enable test once the issue with "license remaining -// duration" is addressed. -TEST_P(WvCdmStreamingPstTest, DISABLED_UsageTest) { +TEST_P(WvCdmStreamingPstTest, UsageTest) { Unprovision(); Provision(); @@ -1400,7 +1402,11 @@ TEST_P(WvCdmStreamingPstTest, DISABLED_UsageTest) { if (initial_license_duration_remaining == kUnlimitedDurationValue) { EXPECT_EQ(license_duration_remaining, kUnlimitedDurationValue); - } else { + } else if (license_duration_remaining != kUnlimitedDurationValue) { + // Possible that the initial license duration was limited due to + // the finit rental duration; however, if the license has a soft + // rental duration, it will be considered unlimited after playback + // has begun. EXPECT_NEAR(initial_license_duration_remaining - license_duration_remaining, expected_seconds_since_license_received, kClockTolerance) << "initial_license_duration_remaining = " @@ -1427,9 +1433,7 @@ class WvCdmStreamingUsageReportTest : public WvCdmExtendedDurationTest, public ::testing::WithParamInterface {}; -// TODO(b275651559): Re-enable test once the issue with "license remaining -// duration" is addressed. -TEST_P(WvCdmStreamingUsageReportTest, DISABLED_UsageTest) { +TEST_P(WvCdmStreamingUsageReportTest, UsageTest) { Unprovision(); Provision(); @@ -1488,7 +1492,11 @@ TEST_P(WvCdmStreamingUsageReportTest, DISABLED_UsageTest) { // change over time. if (initial_license_duration_remaining == kUnlimitedDurationValue) { EXPECT_EQ(license_duration_remaining, kUnlimitedDurationValue); - } else { + } else if (license_duration_remaining != kUnlimitedDurationValue) { + // Possible that the initial license duration was limited due to + // the finit rental duration; however, if the license has a soft + // rental duration, it will be considered unlimited after playback + // has begun. EXPECT_NEAR(initial_license_duration_remaining - license_duration_remaining, expected_seconds_since_license_received, kClockTolerance) << "initial_license_duration_remaining = " @@ -1660,7 +1668,11 @@ TEST_P(WvCdmOfflineUsageReportTest, UsageTest) { if (initial_license_duration_remaining == kUnlimitedDurationValue) { EXPECT_EQ(license_duration_remaining, kUnlimitedDurationValue); - } else { + } else if (license_duration_remaining != kUnlimitedDurationValue) { + // Possible that the initial license duration was limited due to + // the finit rental duration; however, if the license has a soft + // rental duration, it will be considered unlimited after playback + // has begun. EXPECT_NEAR(initial_license_duration_remaining - license_duration_remaining, expected_seconds_since_license_received, kClockTolerance) << "initial_license_duration_remaining = "