tests: stepper: drv84xx: deleting outdated/duplicated tests

Removing tests which are already covered in stepper api test
suite or are outdated and have no relevance in accordance to
the recent changes in the api

Signed-off-by: Jilay Pandya <[email protected]>

Author: jilaypandya

tests/drivers/stepper/drv84xx/api/src/main.c

@@ -87,66 +87,6 @@ ZTEST_F(drv84xx_api, test_actual_position_set)
 	zassert_equal(pos, 100u, "Actual position should be %u but is %u", 100u, pos);
 }
 
-ZTEST_F(drv84xx_api, test_move_to_positive_direction_movement)
-{
-	int32_t pos = 50;
-
-	(void)stepper_enable(fixture->dev);
-	(void)stepper_set_microstep_interval(fixture->dev, 20000000);
-	(void)stepper_set_event_callback(fixture->dev, fixture->callback, NULL);
-	(void)stepper_move_to(fixture->dev, pos);
-	(void)k_poll(&stepper_event, 1, K_SECONDS(5));
-	unsigned int signaled;
-	int result;
-
-	k_poll_signal_check(&stepper_signal, &signaled, &result);
-	zassert_equal(signaled, 1, "No event detected");
-	zassert_equal(result, STEPPER_EVENT_STEPS_COMPLETED,
-		      "Event was not STEPPER_EVENT_STEPS_COMPLETED event");
-	(void)stepper_get_actual_position(fixture->dev, &pos);
-	zassert_equal(pos, 50u, "Target position should be %d but is %d", 50u, pos);
-}
-
-ZTEST_F(drv84xx_api, test_move_to_negative_direction_movement)
-{
-	int32_t pos = -50;
-
-	(void)stepper_enable(fixture->dev);
-	(void)stepper_set_microstep_interval(fixture->dev, 20000000);
-	(void)stepper_set_event_callback(fixture->dev, fixture->callback, NULL);
-	(void)stepper_move_to(fixture->dev, pos);
-	(void)k_poll(&stepper_event, 1, K_SECONDS(5));
-	unsigned int signaled;
-	int result;
-
-	k_poll_signal_check(&stepper_signal, &signaled, &result);
-	zassert_equal(signaled, 1, "No event detected");
-	zassert_equal(result, STEPPER_EVENT_STEPS_COMPLETED,
-		      "Event was not STEPPER_EVENT_STEPS_COMPLETED event");
-	(void)stepper_get_actual_position(fixture->dev, &pos);
-	zassert_equal(pos, -50, "Target position should be %d but is %d", -50, pos);
-}
-
-ZTEST_F(drv84xx_api, test_move_to_identical_current_and_target_position)
-{
-	int32_t pos = 0;
-
-	(void)stepper_enable(fixture->dev);
-	(void)stepper_set_microstep_interval(fixture->dev, 20000000);
-	(void)stepper_set_event_callback(fixture->dev, fixture->callback, NULL);
-	(void)stepper_move_to(fixture->dev, pos);
-	(void)k_poll(&stepper_event, 1, K_SECONDS(5));
-	unsigned int signaled;
-	int result;
-
-	k_poll_signal_check(&stepper_signal, &signaled, &result);
-	zassert_equal(signaled, 1, "No event detected");
-	zassert_equal(result, STEPPER_EVENT_STEPS_COMPLETED,
-		      "Event was not STEPPER_EVENT_STEPS_COMPLETED event");
-	(void)stepper_get_actual_position(fixture->dev, &pos);
-	zassert_equal(pos, 0, "Target position should not have changed from %d but is %d", 0, pos);
-}
-
 ZTEST_F(drv84xx_api, test_move_to_is_moving_true_while_moving)
 {
 	int32_t pos = 50;
@@ -181,26 +121,6 @@ ZTEST_F(drv84xx_api, test_move_to_is_moving_false_when_completed)
 	zassert_false(moving, "Driver should not be in state is_moving after finishing");
 }
 
-ZTEST_F(drv84xx_api, test_move_by_zero_steps_no_movement)
-{
-	int32_t steps = 0;
-
-	(void)stepper_enable(fixture->dev);
-	(void)stepper_set_microstep_interval(fixture->dev, 20000000);
-	(void)stepper_set_event_callback(fixture->dev, fixture->callback, NULL);
-	(void)stepper_move_by(fixture->dev, steps);
-	(void)k_poll(&stepper_event, 1, K_SECONDS(5));
-	unsigned int signaled;
-	int result;
-
-	k_poll_signal_check(&stepper_signal, &signaled, &result);
-	zassert_equal(signaled, 1, "No event detected");
-	zassert_equal(result, STEPPER_EVENT_STEPS_COMPLETED,
-		      "Event was not STEPPER_EVENT_STEPS_COMPLETED event");
-	(void)stepper_get_actual_position(fixture->dev, &steps);
-	zassert_equal(steps, 0, "Target position should be %d but is %d", 0, steps);
-}
-
 ZTEST_F(drv84xx_api, test_move_by_is_moving_true_while_moving)
 {
 	int32_t steps = 50;

tests/drivers/stepper/stepper_api/src/main.c

@@ -99,8 +99,9 @@ ZTEST_F(stepper, test_get_micro_step_res)
 {
 	enum stepper_micro_step_resolution res;
 	(void)stepper_get_micro_step_res(fixture->dev, &res);
-	zassert_equal(res, DT_PROP(DT_ALIAS(stepper), micro_step_res),
-		      "Micro step resolution not set correctly");
+	zassert_equal(DT_PROP(DT_ALIAS(stepper), micro_step_res), res,
+		      "Micro step resolution not set correctly, expected: %d, got: %d ",
+		      DT_PROP(DT_ALIAS(stepper), micro_step_res), res);
 }
 
 ZTEST_F(stepper, test_set_micro_step_interval_invalid_zero)