commit 62f8095bddc360dc4be879a13f7ff94280389fe0
author Wan-Teh Chang <wtc@google.com> Fri Jan 20 14:14:30 2023 -0800
committer Wan-Teh Chang <wtc@google.com> Fri Jan 20 14:30:40 2023 -0800
tree 12be9afd71a353d4e5b7c22046d90c2500fcf0d2
parent e185493001aedb59cf42761bb9459f192915e7ff

Disable ProgressiveTest.DimensionChange

This is better than skip the test unconditionally.

tests/gtest/avifprogressivetest.cc

diff --git a/tests/gtest/avifprogressivetest.cc b/tests/gtest/avifprogressivetest.cc
index 861f09b..4ae55a6 100644
--- a/tests/gtest/avifprogressivetest.cc
+++ b/tests/gtest/avifprogressivetest.cc
@@ -81,17 +81,16 @@
   TestDecode(kImageSize, kImageSize);
 }
 
-TEST_F(ProgressiveTest, DimensionChange) {
+// TODO(wtc): Investigate why the following assertion in libaom failed:
+//   av1/encoder/mcomp.c:1717: av1_full_pixel_search: Assertion
+//   `ms_params->ms_buffers.ref->stride == ms_params->search_sites->stride'
+//   failed.
+// ms_params->ms_buffers.ref->stride is 832, and
+// ms_params->search_sites->stride is 0.
+TEST_F(ProgressiveTest, DISABLED_DimensionChange) {
   if (avifLibYUVVersion() == 0) {
     GTEST_SKIP() << "libyuv not available, skip test.";
   }
-  // TODO(wtc): Investigate why the following assertion in libaom failed:
-  //   av1/encoder/mcomp.c:1717: av1_full_pixel_search: Assertion
-  //   `ms_params->ms_buffers.ref->stride == ms_params->search_sites->stride'
-  //   failed.
-  // ms_params->ms_buffers.ref->stride is 832, and
-  // ms_params->search_sites->stride is 0.
-  GTEST_SKIP() << "Skip test because of an assertion failure in libaom";
 
   encoder_->extraLayerCount = 1;
   encoder_->minQuantizer = 0;