From 357ee293f3e9456d5165960abb79a6588de11242 Mon Sep 17 00:00:00 2001 From: Richard Fine Date: Mon, 31 Dec 2018 12:06:19 +0000 Subject: [PATCH] Remove default value for SimplePathfindingScenario steps Remove the default value as it's more consistent to just always have the test case specify it explicitly. --- test/tests/Pathfinding.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/test/tests/Pathfinding.cpp b/test/tests/Pathfinding.cpp index d1abec1e0f..1065c0f5bb 100644 --- a/test/tests/Pathfinding.cpp +++ b/test/tests/Pathfinding.cpp @@ -159,7 +159,7 @@ struct SimplePathfindingScenario TileCoordsXYZ goal; uint32_t steps; - SimplePathfindingScenario(const char* _name, const TileCoordsXYZ& _start, const TileCoordsXYZ& _goal, int _steps = 10000) + SimplePathfindingScenario(const char* _name, const TileCoordsXYZ& _start, const TileCoordsXYZ& _goal, int _steps) : name(_name) , start(_start) , goal(_goal) @@ -231,7 +231,7 @@ TEST_P(ImpossiblePathfindingTest, CannotFindPathFromStartToGoal) INSTANTIATE_TEST_CASE_P( ForScenario, ImpossiblePathfindingTest, ::testing::Values( - SimplePathfindingScenario("PathWithGap", { 6, 9, 14 }, { 10, 9, 14 }), - SimplePathfindingScenario("PathWithFences", { 6, 7, 14 }, { 10, 7, 14 }), - SimplePathfindingScenario("PathWithCliff", { 10, 5, 14 }, { 12, 5, 14 })), + SimplePathfindingScenario("PathWithGap", { 6, 9, 14 }, { 10, 9, 14 }, 10000), + SimplePathfindingScenario("PathWithFences", { 6, 7, 14 }, { 10, 7, 14 }, 10000), + SimplePathfindingScenario("PathWithCliff", { 10, 5, 14 }, { 12, 5, 14 }, 10000)), SimplePathfindingScenario::ToName);