Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ac2522db authored by Gavin Corkery's avatar Gavin Corkery
Browse files

Reduce minimum time threshold on dumpstate_smoke_test 

Sometimes bugreports finish slightly before the 30 second
threshold, causing test failures. This change reduces this
threshold to 20 seconds and fixes some misleading failure
logs.

Test: atest dumpstate_smoke_test
Bug: 211004508
Bug: 211488888
Change-Id: I37e7707df3237f39d099484a26f195a85f16767a
parent 5cba3db3

cmds/dumpstate/tests/dumpstate_smoke_test.cpp

+4 −4
Original line number Diff line number Diff line
@@ -240,11 +240,11 @@ TEST_F(ZippedBugreportGenerationTest, Is1MBMBinSize) { 
    EXPECT_GE(st.st_size, 1000000 /* 1MB */);

}



TEST_F(ZippedBugreportGenerationTest, TakesBetween30And300Seconds) {

    EXPECT_GE(duration, 30s) << "Expected completion in more than 30s. Actual time "

                             << duration.count() << " s.";

TEST_F(ZippedBugreportGenerationTest, TakesBetween20And300Seconds) {

    EXPECT_GE(duration, 20s) << "Expected completion in more than 20s. Actual time "

                             << duration.count() << " ms.";

    EXPECT_LE(duration, 300s) << "Expected completion in less than 300s. Actual time "

                              << duration.count() << " s.";

                              << duration.count() << " ms.";

}



/**