From: jackychen Date: Sat, 14 Nov 2015 00:02:43 +0000 (-0800) Subject: Enable resize test(down&up) by changing the bitrate. X-Git-Tag: v1.6.0~550 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=204cde580a5f6dd5e7511c932c47c068046d9671;p=libvpx Enable resize test(down&up) by changing the bitrate. Change-Id: I5a4f1f7b9de20fbfc28cb743dcd29c0eeca736f8 --- diff --git a/test/resize_test.cc b/test/resize_test.cc index 16ca8c43a..bc91fe226 100644 --- a/test/resize_test.cc +++ b/test/resize_test.cc @@ -407,7 +407,7 @@ TEST_P(ResizeRealtimeTest, TestInternalResizeDownUpChangeBitRate) { // Disable dropped frames. cfg_.rc_dropframe_thresh = 0; // Starting bitrate low. - cfg_.rc_target_bitrate = 90; + cfg_.rc_target_bitrate = 80; ASSERT_NO_FATAL_FAILURE(RunLoop(&video)); unsigned int last_w = cfg_.g_w; @@ -431,10 +431,8 @@ TEST_P(ResizeRealtimeTest, TestInternalResizeDownUpChangeBitRate) { } } - // Verify that we get at least 1 resize event in this test. - // TODO(marpan): We should be able to force 2 resize events - // with some change in encoder settings/bitrate. - ASSERT_GE(resize_count, 1) << "Resizing should occur at least once."; + // Verify that we get 2 resize events in this test. + ASSERT_EQ(resize_count, 2) << "Resizing should occur twice."; } vpx_img_fmt_t CspForFrameNumber(int frame) {