refs #863 Fixed wrong test that had been masked by CRemoteAircraftProviderDummy being too clever.

This commit is contained in:
Mathew Sutcliffe
2017-02-07 15:31:50 +00:00
parent 598733d521
commit 64f20f0141
2 changed files with 2 additions and 4 deletions

View File

@@ -188,14 +188,12 @@ namespace BlackMisc
void CRemoteAircraftProviderDummy::insertNewSituation(const CAircraftSituation &situation)
{
this->m_situations.push_front(situation);
this->m_situations.sortLatestFirst(); // like in real world, latest should be first
emit addedRemoteAircraftSituation(situation);
}
void CRemoteAircraftProviderDummy::insertNewAircraftParts(const CCallsign &callsign, const CAircraftParts &parts)
{
this->m_parts[callsign].push_front(parts);
this->m_parts[callsign].sortLatestFirst(); // like in real world, latest should be first
emit addedRemoteAircraftParts(callsign, parts);
}

View File

@@ -63,7 +63,7 @@ namespace BlackMiscTest
const qint64 ts = 1425000000000; // QDateTime::currentMSecsSinceEpoch();
const qint64 deltaT = 5000; // ms
const qint64 offset = 5000;
for (int i = 0; i < IRemoteAircraftProvider::MaxSituationsPerCallsign; i++)
for (int i = IRemoteAircraftProvider::MaxSituationsPerCallsign - 1; i >= 0; i--)
{
CAircraftSituation s(getTestSituation(cs, i, ts, deltaT, offset));
@@ -74,7 +74,7 @@ namespace BlackMiscTest
}
constexpr int partsCount = 10;
for (int i = 0; i < partsCount; i++)
for (int i = partsCount - 1; i >= 0; i--)
{
CAircraftParts p(getTestParts(i, ts, deltaT));
interpolator.addAircraftParts(p);