From afead891d1ea019980872cfa4ca9c9f9c0e4fbaf Mon Sep 17 00:00:00 2001
From: Yuri D'Elia <wavexx@thregr.org>
Date: Wed, 11 May 2022 00:06:23 +0200
Subject: [PATCH] motion: Fix motion::triple_move step count limit

Do not assume any step can be merged while moving three axes at the same
time.

Use the worst case scenario involving indepedent parameters for each
axis.
---
 tests/unit/modules/motion/test_motion.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/unit/modules/motion/test_motion.cpp b/tests/unit/modules/motion/test_motion.cpp
index 60bb1e8..3317243 100644
--- a/tests/unit/modules/motion/test_motion.cpp
+++ b/tests/unit/modules/motion/test_motion.cpp
@@ -216,7 +216,7 @@ TEST_CASE("motion::triple_move", "[motion]") {
     motion.PlanMoveTo(Pulley, p, 1);
 
     // perform the move with a maximum step limit
-    REQUIRE(stepUntilDone(std::max({ i, s, p }) + 1) != -1);
+    REQUIRE(stepUntilDone(i + s + p) != -1);
 
     // check queue status
     REQUIRE(motion.QueueEmpty());