kirylkaveryn/organicmaps
1
Fork 0
forked from organicmaps/organicmaps-tmp
Code Pull requests Activity

[tracking] Wrong packet tests.

Browse source
This commit is contained in:
Vladimir Byko-Ianko 2020-05-27 17:49:36 +03:00 • committed by mpimenov
parent 928e42f90f
commit 30e835a851
2 changed files with 20 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
tracking/protocol.cpp
View file

@ -121,7 +121,7 @@ Protocol::DataElementsVec Protocol::DecodeDataPacket(PacketType type, vector<uin
}
catch (Reader::SizeException const & ex)
{
LOG(LERROR, ("Wrong packet. SizeException. Msg:", ex.Msg(), ". What:", ex.what()));
LOG(LWARNING, ("Wrong packet. SizeException. Msg:", ex.Msg(), ". What:", ex.what()));
return {};
}
}

19
tracking/tracking_tests/protocol_test.cpp
View file

@ -108,3 +108,22 @@ UNIT_TEST(Protocol_DecodeDataPacket)
DecodeDataPacketVersionTest(points, Protocol::PacketType::DataV0);
DecodeDataPacketVersionTest(points, Protocol::PacketType::DataV1);
}
UNIT_TEST(Protocol_DecodeWrongDataPacket)
{
vector<vector<uint8_t>> payloads = {
vector<uint8_t>{},
vector<uint8_t>{0x25},
vector<uint8_t>{0x0},
vector<uint8_t>{0x0, 0x0, 0x23, 0xFF},
vector<uint8_t>{0xFF, 0x1, 0x23, 0xFF, 0x1, 0x0, 0x27, 0x63, 0x32, 0x9, 0xFF},
};
for (auto const packetType : {Protocol::PacketType::DataV0, Protocol::PacketType::DataV1})
{
for (auto const & payload : payloads)
{
auto result = Protocol::DecodeDataPacket(packetType, payload);
TEST(result.empty(), (packetType, payload));
}
}
}