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
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