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

Commit ca78ddf4 authored by Tiffany Yang's avatar Tiffany Yang
Browse files

libtimeinstate: Print missing uid on test failure 

AllUidTimeInStateMonotonic and AllUidConcurrentTimesMonotonic runs have
occasionally failed on this assertion, but without knowing which uid we
expect to find in the map, it is difficult to find the root cause. Print
the uid that we failed on for easier debugging.

Flag: TEST_ONLY
Bug: 422280686
Bug: 413162440
Test: treehugger
Change-Id: Ibb8a859538d3bfed8f69861ed97bf47145bfe3a6
parent 2810984f

libs/cputimeinstate/testtimeinstate.cpp

+2 −2
Original line number Diff line number Diff line
@@ -367,7 +367,7 @@ TEST_F(TimeInStateTest, AllUidTimeInStateMonotonic) { 
    for (const auto &kv : *map1) {

        uint32_t uid = kv.first;

        auto times = kv.second;

        ASSERT_NE(map2->find(uid), map2->end());

        ASSERT_NE(map2->find(uid), map2->end()) << "no entry for uid " << uid;

        for (uint32_t policy = 0; policy < times.size(); ++policy) {

            for (uint32_t freqIdx = 0; freqIdx < times[policy].size(); ++freqIdx) {

                auto before = times[policy][freqIdx];
@@ -390,7 +390,7 @@ TEST_F(TimeInStateTest, AllUidConcurrentTimesMonotonic) { 
    for (const auto &kv : *map1) {

        uint32_t uid = kv.first;

        auto times = kv.second;

        ASSERT_NE(map2->find(uid), map2->end());

        ASSERT_NE(map2->find(uid), map2->end()) << "no entry for uid " << uid;

        for (uint32_t i = 0; i < times.active.size(); ++i) {

            auto before = times.active[i];

            auto after = (*map2)[uid].active[i];