Skip to content

test: require queued jog observation and verify controller reset position#79

Merged
ianbruene merged 1 commit into
masterfrom
codex/tighten-mockgrbl-integration-tests
Jul 18, 2026
Merged

test: require queued jog observation and verify controller reset position#79
ianbruene merged 1 commit into
masterfrom
codex/tighten-mockgrbl-integration-tests

Conversation

@ianbruene

Copy link
Copy Markdown
Owner

Motivation

  • Stabilize the two newest Linux-only mockgrbl_e2e integration tests by making their assertions stronger and observable rather than timing-dependent.
  • Ensure the queued absolute-jog sequencing test actually observes queued commands while motion is active instead of only checking final positions.
  • Ensure the reset-during-jog test verifies DDGo observed the mock's materialized post-reset X position, not just that the mock ended up idle.

Description

  • Modified internal/integration/mockgrbl/mockgrbl_e2e_test.go to use a longer queued jog target and feed (queuedJogTarget = -3.0, queuedJogFeed = 120.0) so queueing is observable in CI.
  • Added an explicit observation step that waits for a mock state with `State ==

Codex Task

@ianbruene
ianbruene merged commit aca0a83 into master Jul 18, 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