From 782068ebc417e8ed097f15d697f0bd55b94438f8 Mon Sep 17 00:00:00 2001 From: Viktor Govako Date: Wed, 1 Nov 2023 22:56:05 -0300 Subject: [PATCH] [search][test] Fixed quality tests. Signed-off-by: Viktor Govako --- .../search_quality_tests/real_mwm_tests.cpp | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/search/search_quality/search_quality_tests/real_mwm_tests.cpp b/search/search_quality/search_quality_tests/real_mwm_tests.cpp index ca944849dc..7a0ad5a3f1 100644 --- a/search/search_quality/search_quality_tests/real_mwm_tests.cpp +++ b/search/search_quality/search_quality_tests/real_mwm_tests.cpp @@ -522,15 +522,19 @@ UNIT_CLASS_TEST(MwmTestsFixture, Street_BusStop) TEST_LESS(SortedByDistance(range, center), 5000.0, ()); } + /// @todo Actually, we have very fancy matching here, starting from 3rd result and below. + /// Interesting to check how it happens. { auto request = MakeRequest("Juncal train", "en"); auto const & results = request->Results(); TEST_GREATER(results.size(), kTopPoiResultsCount, ()); - // First result is a train station in other MWM, >200km away. - TEST(EqualClassifType(results[0].GetFeatureType(), classif().GetTypeByPath({"railway", "station"})), ()); - double const dist = ms::DistanceOnEarth(center, mercator::ToLatLon(results[0].GetFeatureCenter())); - TEST_GREATER(dist, 2.0E5, ()); + // First result is a "Juncal" supermarket near the train station, 24km :) + // Second result is a train station in other MWM, >200km away. + Range const range(results, 0, 2); + EqualClassifType(range, GetClassifTypes({{"shop", "supermarket"}, {"railway", "station"}})); + TEST_LESS(!SortedByDistance(range, center), 2.0E5, ()); + TEST_LESS(SortedByDistance(range, center), 3.0E5, ()); } }