commit 29722be8461a1eaf99743bdd6e99a0ba269e79e8
author Wan-Teh Chang <wtc@google.com> Thu Aug 27 09:18:08 2020 -0700
committer Wan-Teh Chang <wtc@google.com> Mon Sep 28 18:45:24 2020 +0000
tree b5a1f337db42139b2642100b7aec00090957ebba
parent 372c3a3747cd47f4ef9b2b3944ee5c80e18ff046

Do not mark StillPicturePresenceTest as Large

It takes about 10 seconds to run a debug build of
StillPicturePresenceTestLarge, so it doesn't seem necessary to mark this
test as Large. We can improve test coverage of the verification build of
CLs by not marking this test as Large.

Change-Id: I5ed9257a51f8714186276ec1528721797165074d

test/still_picture_test.cc

diff --git a/test/still_picture_test.cc b/test/still_picture_test.cc
index 39ad2ba..797972d 100644
--- a/test/still_picture_test.cc
+++ b/test/still_picture_test.cc
@@ -18,16 +18,16 @@
 namespace {
 
 // This class is used to test the presence of still picture feature.
-class StillPicturePresenceTestLarge
+class StillPicturePresenceTest
     : public ::libaom_test::CodecTestWith2Params<libaom_test::TestMode, int>,
       public ::libaom_test::EncoderTest {
  protected:
-  StillPicturePresenceTestLarge()
+  StillPicturePresenceTest()
       : EncoderTest(GET_PARAM(0)), encoding_mode_(GET_PARAM(1)),
         enable_full_header_(GET_PARAM(2)) {
     still_picture_coding_violated_ = false;
   }
-  virtual ~StillPicturePresenceTestLarge() {}
+  virtual ~StillPicturePresenceTest() {}
 
   virtual void SetUp() {
     InitializeConfig();
@@ -82,7 +82,7 @@
 };
 
 // TODO(crbug.com/aomedia/2813): Fix and re-enable the test.
-TEST_P(StillPicturePresenceTestLarge, DISABLED_StillPictureEncodePresenceTest) {
+TEST_P(StillPicturePresenceTest, DISABLED_StillPictureEncodePresenceTest) {
   libaom_test::I420VideoSource video("hantro_collage_w352h288.yuv", 352, 288,
                                      cfg_.g_timebase.den, cfg_.g_timebase.num,
                                      0, 1);
@@ -90,7 +90,7 @@
   ASSERT_EQ(still_picture_coding_violated_, false);
 }
 
-AV1_INSTANTIATE_TEST_SUITE(StillPicturePresenceTestLarge,
+AV1_INSTANTIATE_TEST_SUITE(StillPicturePresenceTest,
                            ::testing::Values(::libaom_test::kOnePassGood,
                                              ::libaom_test::kTwoPassGood),
                            ::testing::Values(1, 0));