From 20833734e1cebff4f2a5bb3a23e5909f4e68f3e2 Mon Sep 17 00:00:00 2001 From: Fred Gylys-Colwell Date: Thu, 20 May 2021 15:28:12 +0000 Subject: [PATCH] Add fudge to duration tests Merge from Widevine repo of http://go/wvgerrit/125046 Add some extra fudge to the termination time. This assumes that flaky tests were caused by roundoff error. Bug: 175741647 Change-Id: I9bcc86f9b9540e5985df647dd1b5c5d439556e2b --- libwvdrmengine/cdm/core/test/duration_use_case_test.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libwvdrmengine/cdm/core/test/duration_use_case_test.cpp b/libwvdrmengine/cdm/core/test/duration_use_case_test.cpp index c778eace..68d87241 100644 --- a/libwvdrmengine/cdm/core/test/duration_use_case_test.cpp +++ b/libwvdrmengine/cdm/core/test/duration_use_case_test.cpp @@ -358,8 +358,10 @@ class CdmDurationTest : public WvCdmTestBaseWithEngine, // this tests a cyan bar with a black X. This assumes that |cutoff| is before // |stop|. Both |start| and |cutoff| are system times. void TerminatePlayback(uint64_t start, uint64_t cutoff) { - ASSERT_LT(start, cutoff); - AllowPlayback(start, cutoff); + // We subtract some fudge from the cutoff to account for possible round off + // error when computing the playback window. + ASSERT_LT(start, cutoff - kFudge); + AllowPlayback(start, cutoff - kFudge); SleepUntil(cutoff + kFudge); FailDecrypt(); }