Skip to content

Comments

test: add e2e completion coverage and precedence docs#51

Merged
leodido merged 2 commits intomainfrom
codex/complete-e2e-coverage
Feb 23, 2026
Merged

test: add e2e completion coverage and precedence docs#51
leodido merged 2 commits intomainfrom
codex/complete-e2e-coverage

Conversation

@leodido
Copy link
Owner

@leodido leodido commented Feb 23, 2026

Summary

  • add an end-to-end Cobra __complete integration test to exercise auto-registered Complete<FieldName> hooks through the real shell-completion path
  • add a test covering precedence when manually registering completion after structcli.Define() auto-registration
  • add targeted tests for completion-hook error/guard paths to improve coverage
  • document completion registration precedence in README

Validation

  • go test ./...
  • go test ./... (from examples/full)
  • coverage: 87.4% total statements

@leodido leodido merged commit bf464f3 into main Feb 23, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant