From 73d8cf392b43b2e6a870a97311a3576989cce59b Mon Sep 17 00:00:00 2001 From: Mat Sutcliffe Date: Sun, 30 Dec 2018 00:27:28 +0000 Subject: [PATCH] Ref T275 testaircraftsituation: repeat test 20 times to mitigate external influences. --- .../testaircraftsituation.cpp | 27 ++++++++++++------- 1 file changed, 17 insertions(+), 10 deletions(-) diff --git a/tests/blackmisc/aviation/testaircraftsituation/testaircraftsituation.cpp b/tests/blackmisc/aviation/testaircraftsituation/testaircraftsituation.cpp index 75bf70b91..2af8b094e 100644 --- a/tests/blackmisc/aviation/testaircraftsituation/testaircraftsituation.cpp +++ b/tests/blackmisc/aviation/testaircraftsituation/testaircraftsituation.cpp @@ -229,12 +229,13 @@ namespace BlackMiscTest void CTestAircraftSituation::sortHint() { - constexpr int Max = 50000; + constexpr int Lists = 50000; + constexpr int Loops = 20; CAircraftSituationList situations = testSituations(); situations.sortAdjustedLatestFirst(); CSequence listOfLists1; CSequence listOfLists2; - for (int i = 0; i < Max; ++i) + for (int i = 0; i < Lists; ++i) { listOfLists1.push_back(situations); listOfLists2.push_back(situations); @@ -245,20 +246,26 @@ namespace BlackMiscTest QTime time; time.start(); - for (const CAircraftSituationList &s : as_const(listOfLists1)) + for (int i = 0; i < Loops; ++i) { - const CAircraftSituation s1 = s.oldestAdjustedObject(); - const CAircraftSituation s2 = s.latestAdjustedObject(); - QVERIFY(s1.getAdjustedMSecsSinceEpoch() < s2.getAdjustedMSecsSinceEpoch()); + for (const CAircraftSituationList &s : as_const(listOfLists1)) + { + const CAircraftSituation s1 = s.oldestAdjustedObject(); + const CAircraftSituation s2 = s.latestAdjustedObject(); + QVERIFY(s1.getAdjustedMSecsSinceEpoch() < s2.getAdjustedMSecsSinceEpoch()); + } } const int noHint = time.elapsed(); time.start(); - for (const CAircraftSituationList &s : as_const(listOfLists2)) + for (int i = 0; i < Loops; ++i) { - const CAircraftSituation s1 = s.oldestAdjustedObject(); - const CAircraftSituation s2 = s.latestAdjustedObject(); - QVERIFY(s1.getAdjustedMSecsSinceEpoch() < s2.getAdjustedMSecsSinceEpoch()); + for (const CAircraftSituationList &s : as_const(listOfLists2)) + { + const CAircraftSituation s1 = s.oldestAdjustedObject(); + const CAircraftSituation s2 = s.latestAdjustedObject(); + QVERIFY(s1.getAdjustedMSecsSinceEpoch() < s2.getAdjustedMSecsSinceEpoch()); + } } const int hint = time.elapsed(); const double ratio = static_cast(hint) / static_cast(noHint); // expected <1.0