Skip to content

test(go): expand coverage for repl and config#5

Merged
halukerenozlu merged 1 commit into
masterfrom
codex-improve-repl-coverage
Jun 1, 2026
Merged

test(go): expand coverage for repl and config#5
halukerenozlu merged 1 commit into
masterfrom
codex-improve-repl-coverage

Conversation

@halukerenozlu

Copy link
Copy Markdown
Owner

What does this PR do?

Expands test coverage for REPL command behavior, config secret-preserving saves, and modal widget interactions. The change is test-only and does not modify runtime behavior.

Related issues

N/A

Changes

  • Add CommandRegistry.All() ordering coverage
  • Add REPL tests for skill listing/details, /use, model switching, and active provider/model helpers
  • Add config tests for SaveFilePreservingSecrets
  • Add modal widget tests for close/no-match behavior, cursor bounds, input values, and item rendering

How to test

Steps a reviewer can follow to verify the change manually:

  1. Run go test ./... -count=1
  2. Run go test -coverprofile .\coverage.out ./...
  3. Run go tool cover -func .\coverage.out and confirm total coverage is around 68.7%

Checklist

  • Tests pass (go test ./... -count=1)
  • No vet warnings (go vet ./...)
  • Build succeeds (make build)
  • Conventional commit message (e.g. feat(go/agent): ...)
  • Docs updated if behaviour changed
  • No breaking changes (or described below)

Co-Authored-By: OpenAI Codex GPT 5.5 <noreply@openai.com>
@halukerenozlu halukerenozlu merged commit 70619c4 into master Jun 1, 2026
2 checks passed
@halukerenozlu halukerenozlu deleted the codex-improve-repl-coverage branch June 1, 2026 18:57
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