diff --git a/test/gurka/CMakeLists.txt b/test/gurka/CMakeLists.txt index fd20e7806c..6e8a28733d 100644 --- a/test/gurka/CMakeLists.txt +++ b/test/gurka/CMakeLists.txt @@ -11,13 +11,10 @@ if(ENABLE_DATA_TOOLS) # "-Werror" flag # Avoid adding a filename to this list set(TESTS_WITH_WARNINGS - test_elevation.cc test_gtfs.cc test_instructions_roundabout.cc - test_landmarks.cc test_languages.cc test_locate.cc - test_match.cc test_multi_level_loki.cc test_osrm_serializer.cc test_pbf_api.cc @@ -25,8 +22,6 @@ if(ENABLE_DATA_TOOLS) test_phonemes_w_langs.cc test_recost.cc test_time_tracking.cc - test_traffic.cc - test_traffic_smoothing.cc ) ## Add executable targets diff --git a/test/gurka/test_elevation.cc b/test/gurka/test_elevation.cc index 0f7a8c7fe9..ba2916a019 100644 --- a/test/gurka/test_elevation.cc +++ b/test/gurka/test_elevation.cc @@ -220,7 +220,7 @@ TEST(Standalone, ElevationCompareToSkadi) { result.Parse(route_json.c_str()); for (size_t leg_index = 0; leg_index < waypoints.size() - 1; ++leg_index) { - auto s = + [[maybe_unused]] auto s = rapidjson::get_child_optional(result, ("/trip/legs/" + std::to_string(leg_index) + "/shape") .c_str()); diff --git a/test/gurka/test_landmarks.cc b/test/gurka/test_landmarks.cc index c4855213e2..53eece209c 100644 --- a/test/gurka/test_landmarks.cc +++ b/test/gurka/test_landmarks.cc @@ -620,7 +620,7 @@ TEST(LandmarkTest, TestLandmarksInManeuvers) { "Checking landmarks in maneuver failed: cannot find the maneuver in the expected result!"); } ASSERT_EQ(result_landmarks.size(), expected->second.size()); - for (auto i = 0; i < result_landmarks.size(); ++i) { + for (size_t i = 0; i < result_landmarks.size(); ++i) { EXPECT_EQ(result_landmarks[i], expected->second[i]); } } diff --git a/test/gurka/test_match.cc b/test/gurka/test_match.cc index 3198b862c5..3be216b114 100644 --- a/test/gurka/test_match.cc +++ b/test/gurka/test_match.cc @@ -262,8 +262,9 @@ uint32_t speed_from_edge(const valhalla::Api& api, bool compare_with_previous_ed node.cost().elapsed_cost().seconds() - node.cost().transition_cost().seconds()) / 3600.0; auto new_kmh = static_cast(km / h + .5); - if (is_valid(kmh) && compare_with_previous_edge) + if (is_valid(kmh) && compare_with_previous_edge) { EXPECT_EQ(kmh, new_kmh); + } kmh = new_kmh; } return kmh; diff --git a/test/gurka/test_traffic.cc b/test/gurka/test_traffic.cc index 613852a869..411cde3707 100644 --- a/test/gurka/test_traffic.cc +++ b/test/gurka/test_traffic.cc @@ -296,8 +296,6 @@ TEST(Traffic, CutGeoms) { uint32_t index, baldr::TrafficSpeed* current) -> void { baldr::GraphId tile_id(tile.header->tile_id); auto BD = gurka::findEdge(reader, map.nodes, "BD", "D", tile_id); - baldr::TrafficSpeed* existing = - const_cast(tile.speeds + index); current->breakpoint1 = 255; if (std::get<1>(BD) != nullptr && std::get<0>(BD).id() == index) { current->overall_encoded_speed = 0; diff --git a/test/gurka/test_traffic_smoothing.cc b/test/gurka/test_traffic_smoothing.cc index ef31753916..1f3d76fa40 100644 --- a/test/gurka/test_traffic_smoothing.cc +++ b/test/gurka/test_traffic_smoothing.cc @@ -558,7 +558,7 @@ class MapMatchWithTraffic : public ::testing::Test { traffic_speed->breakpoint1 = 255; }); - test::customize_historical_traffic(map.config, [](DirectedEdge& e) { + test::customize_historical_traffic(map.config, [](DirectedEdge&) { // speeds for every 5 min bucket of the week std::array historical; historical.fill(6);