pastk/organicmaps
1
Fork 0
forked from organicmaps/organicmaps
Code Pull requests Activity

Fixing routing_quality_tests for map 181124.

Browse source
This commit is contained in:
Vladimir Byko-Ianko 2018-11-30 18:26:05 +03:00 • committed by Maksim Andrianov
parent 74cfde058f
commit 4aa1eee4cb
1 changed files with 30 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

50
routing/routing_quality/routing_quality_tests/bigger_roads_tests.cpp
View file

@ -7,12 +7,15 @@ using namespace routing_quality;
// Test on preferring better but longer roads should be grouped in this file.
namespace
{
UNIT_TEST(RoutingQuality_RussiaMoscowTushino)
{
TEST(CheckCarRoute({55.84398, 37.45018} /* start */, {55.85489, 37.43784} /* finish */,
{{{55.84343, 37.43949}}} /* reference track */),
());
}
// @TODO This test was broken after using maxspeed tag value always if it's available.
// It should be fixed by using maxspeed taking into account highway class.
// Secondary should be preferred against residential.
//UNIT_TEST(RoutingQuality_RussiaMoscowTushino)
//{
// TEST(CheckCarRoute({55.84398, 37.45018} /* start */, {55.85489, 37.43784} /* finish */,
// {{{55.84343, 37.43949}}} /* reference track */),
// ());
//}
UNIT_TEST(RoutingQuality_TurkeyIzmirArea)
{
@ -49,30 +52,37 @@ UNIT_TEST(RoutingQuality_USAOklahoma)
());
}
UNIT_TEST(RoutingQuality_IranSouth)
{
TEST(CheckCarRoute({32.45088, 51.76419} /* start */, {32.97067, 51.50399} /* finish */,
{{{32.67021, 51.64323}, {32.68752, 51.63387}}} /* reference track */),
());
}
// @TODO This test was broken after using maxspeed tag value always if it's available.
// It should be fixed by using maxspeed taking into account highway class.
// Trunk should be preferred against primary.
//UNIT_TEST(RoutingQuality_IranSouth)
//{
// TEST(CheckCarRoute({32.45088, 51.76419} /* start */, {32.97067, 51.50399} /* finish */,
// {{{32.67021, 51.64323}, {32.68752, 51.63387}}} /* reference track */),
// ());
//}
UNIT_TEST(RoutingQuality_EindhovenNetherlands)
{
TEST(CheckCarRoute({50.91974, 5.33535} /* start */, {51.92532, 5.49066} /* finish */,
{{{51.40579, 5.45578}, {51.44316, 5.42723}, {51.50230, 5.47485}}} /* reference track */),
{{{51.42016, 5.42881}, {51.44316, 5.42723}, {51.50230, 5.47485}}} /* reference track */),
());
}
UNIT_TEST(RoutingQuality_GeteborgasSweden)
{
TEST(CheckCarRoute({57.77064, 11.88079} /* start */, {57.71231, 11.93157} /* finish */,
{{{57.74912, 11.87343}}} /* reference track */),
());
}
// @TODO This test was broken after using maxspeed tag value always if it's available.
// It does not show a big problem. The route goes along a short territory road instead of
// going a trunk road which is much longer. But an end user wrote that it's better go
// along the trunk. So this test should be fixed by taking into account highway class.
//UNIT_TEST(RoutingQuality_GeteborgasSweden)
//{
// TEST(CheckCarRoute({57.77064, 11.88079} /* start */, {57.71231, 11.93157} /* finish */,
// {{{57.74912, 11.87343}}} /* reference track */),
// ());
//}
UNIT_TEST(RoutingQuality_CigilTurkey)
{
TEST(CheckCarRoute({38.48175, 27.12952} /* start */, {38.47485, 27.07437} /* finish */,
TEST(CheckCarRoute({38.48175, 27.12952} /* start */, {38.47558, 27.06765} /* finish */,
{{{38.4898049, 27.1016266}}} /* reference track */),
());
}