Skip to content

test: expand mockgrbl motion integration coverage#78

Merged
ianbruene merged 1 commit into
masterfrom
codex/add-integration-tests-for-jog-sequencing-and-reset
Jul 17, 2026
Merged

test: expand mockgrbl motion integration coverage#78
ianbruene merged 1 commit into
masterfrom
codex/add-integration-tests-for-jog-sequencing-and-reset

Conversation

@ianbruene

Copy link
Copy Markdown
Owner

Motivation

  • Extend the Linux-only mockgrbl_e2e integration suite to cover multi-motion queuing and realtime reset behavior that were previously untested.

Description

  • Add TestDDGoQueuedAbsoluteJogSequencingAgainstMock to internal/integration/mockgrbl/mockgrbl_e2e_test.go, which queues three absolute machine-coordinate jogs (X, Y, Z) via controller.JogTo and asserts the mock planner executes them to final (-1, -1, -1) using both mock /state and the controller snapshot.
  • Add TestDDGoRealtimeResetDuringJogAgainstMock to the same test file, which starts a long absolute jog, waits for active motion, issues a realtime soft reset via controller.Action(grbl.ActionSoftReset), and asserts the mock returns to Idle with active/queued motion cleared and a sane materialized X position.
  • Both tests reuse the existing mock harness, real serial transport, bounded polling helpers (waitForMockState, waitForControllerState, etc.), tolerances (posTol) for motion assertions, and avoid changing production code or CI workflows.

Testing

  • Ran go test -tags 'serial mockgrbl_e2e' ./internal/integration/mockgrbl, which passed.
  • Ran go test ./..., which passed.
  • Ran go test -tags serial ./internal/transport ./internal/ports ./internal/app ./internal/grbl, which passed.

Codex Task

@ianbruene
ianbruene merged commit cb83d49 into master Jul 17, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant