commit ddf1c3a68267bd1ec9890ff3971ee199f70ea2c5
author Urvang Joshi <urvang@google.com> Thu Aug 12 12:37:33 2021 -0700
committer Debargha Mukherjee <debargha@google.com> Mon Aug 30 20:53:58 2021 -0700
tree 0c90298b7bb084a0219f22c9d632994aaaa7e53d
parent e5cf5bbfed910ca48126e8bdfc3df3cf139776b0

Mark some subgop unit tests as 'Large'.

These tests are very slow, so we mark them as Large. That way, they are
only run nightly, and NOT per-commit.
- SubGopSwitchingTest and
- SubGopPSNRCheckTest

e.g. AV1/SubGopSwitchingTest.SubGopSwitching/0 takes 3350331 ms = 56 min
and  AV1/SubGopPSNRCheckTest.SubGopPSNRCheck/6 takes 1358767 ms = 23 min

Related: reduce number of frames encoded in SubGopSwitchingTest to
roughly halve the time for that (for nightly tests).

Change-Id: I1a84734dc172eebd297207726f4876c7a1e24f80

test/subgop_test.cc

diff --git a/test/subgop_test.cc b/test/subgop_test.cc
index 3afafb5..731784f 100644
--- a/test/subgop_test.cc
+++ b/test/subgop_test.cc
@@ -703,17 +703,17 @@
             << " lag_in_frames:" << test_arg.lag_in_frames << " }";
 }
 
-class SubGopPSNRCheckTest
+class SubGopPSNRCheckTestLarge
     : public ::libaom_test::CodecTestWith2Params<SubgopPsnrTestParams,
                                                  aom_rc_mode>,
       public ::libaom_test::EncoderTest {
  protected:
-  SubGopPSNRCheckTest()
+  SubGopPSNRCheckTestLarge()
       : EncoderTest(GET_PARAM(0)), test_params_(GET_PARAM(1)),
         rc_end_usage_(GET_PARAM(2)) {
     Reset();
   }
-  virtual ~SubGopPSNRCheckTest() {}
+  virtual ~SubGopPSNRCheckTestLarge() {}
 
   void Reset() {
     frame_num_ = 0;
@@ -758,7 +758,7 @@
   unsigned int frame_num_;
 };
 
-TEST_P(SubGopPSNRCheckTest, SubGopPSNRCheck) {
+TEST_P(SubGopPSNRCheckTestLarge, SubGopPSNRCheck) {
   std::unique_ptr<libaom_test::VideoSource> video;
   const unsigned int kFrames = 100;
   const double psnr_diff_thresh = 0.3;
@@ -786,7 +786,7 @@
 }
 
 // TODO(any) : Enable AOM_CBR after fix
-AV1_INSTANTIATE_TEST_SUITE(SubGopPSNRCheckTest,
+AV1_INSTANTIATE_TEST_SUITE(SubGopPSNRCheckTestLarge,
                            ::testing::ValuesIn(SubGopPsnrTestVectors),
                            ::testing::Values(AOM_Q, AOM_VBR,
                                              AOM_CQ /*, AOM_CBR*/));
@@ -833,12 +833,12 @@
 };
 
 using libaom_test::ACMRandom;
-class SubGopSwitchingTest
+class SubGopSwitchingTestLarge
     : public ::libaom_test::CodecTestWith2Params<SubGopSwitchTestParams,
                                                  aom_rc_mode>,
       public ::libaom_test::EncoderTest {
  protected:
-  SubGopSwitchingTest()
+  SubGopSwitchingTestLarge()
       : EncoderTest(GET_PARAM(0)), test_params_(GET_PARAM(1)),
         rc_end_usage_(GET_PARAM(2)) {
     last_subgop_str_ = NULL;
@@ -846,7 +846,7 @@
     rnd_.Reset(ACMRandom::DeterministicSeed());
     ResetSubgop();
   }
-  virtual ~SubGopSwitchingTest() {}
+  virtual ~SubGopSwitchingTestLarge() {}
 
   virtual void SetUp() {
     InitializeConfig();
@@ -954,9 +954,9 @@
   const char *last_subgop_str_;
 };
 
-TEST_P(SubGopSwitchingTest, SubGopSwitching) {
+TEST_P(SubGopSwitchingTestLarge, SubGopSwitching) {
   std::unique_ptr<libaom_test::VideoSource> video;
-  const unsigned int kFrames = 200;
+  const unsigned int kFrames = 100;
 
   if (is_extension_y4m(test_params_.input_file)) {
     video.reset(
@@ -973,7 +973,7 @@
   EXPECT_TRUE(num_subgop_cfg_used_);
 }
 
-AV1_INSTANTIATE_TEST_SUITE(SubGopSwitchingTest,
+AV1_INSTANTIATE_TEST_SUITE(SubGopSwitchingTestLarge,
                            ::testing::ValuesIn(SubgopSwitchTestVectors),
                            ::testing::Values(AOM_Q, AOM_VBR, AOM_CQ, AOM_CBR));
 }  // namespace