Skip to content

Commit

Permalink
Merge branch 'fix_test' into 'master'
Browse files Browse the repository at this point in the history
Reduce precision for random point position comparison (#8260)

Closes #8260

See merge request OpenMW/openmw!4483
  • Loading branch information
Assumeru committed Dec 19, 2024
2 parents 82a8e63 + 054d5ec commit 356c530
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion apps/components_tests/detournavigator/navigator.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,11 @@ MATCHER_P3(Vec3fEq, x, y, z, "")
return std::abs(arg.x() - x) < 1e-3 && std::abs(arg.y() - y) < 1e-3 && std::abs(arg.z() - z) < 1e-3;
}

MATCHER_P4(Vec3fEq, x, y, z, precision, "")
{
return std::abs(arg.x() - x) < precision && std::abs(arg.y() - y) < precision && std::abs(arg.z() - z) < precision;
}

namespace
{
using namespace testing;
Expand Down Expand Up @@ -605,7 +610,7 @@ namespace
const auto result = findRandomPointAroundCircle(
*mNavigator, mAgentBounds, mStart, 100.0, Flag_walk, []() { return Misc::Rng::rollClosedProbability(); });

ASSERT_THAT(result, Optional(Vec3fEq(70.35845947265625, 335.592041015625, -2.6667339801788330078125)))
ASSERT_THAT(result, Optional(Vec3fEq(70.35845947265625, 335.592041015625, -2.6667339801788330078125, 1)))
<< (result ? *result : osg::Vec3f());

const auto distance = (*result - mStart).length();
Expand Down

0 comments on commit 356c530

Please sign in to comment.