Skip to content

add local test command - #191

Merged
theodore-s-beers merged 1 commit into
mainfrom
tsb_localtest
Jun 11, 2026
Merged

add local test command#191
theodore-s-beers merged 1 commit into
mainfrom
tsb_localtest

Conversation

@theodore-s-beers

Copy link
Copy Markdown
Member

No description provided.

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new bootdev local-test command to run a local cli.yaml manifest through the existing CLI/HTTP check runner and render the results, plus introduces local-side grading logic to simulate submission outcomes.

Changes:

  • Add local-test Cobra command that loads a local manifest, runs checks, and simulates grading.
  • Add YAML tags to client.CLIData and related structs to support unmarshalling local manifests.
  • Introduce checks/local.go for evaluating results and producing a LessonSubmissionEvent, with tests covering the new behavior.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
FileDescription
go.modPromotes YAML v3 dependency to direct for local manifest parsing.
cmd/localtest.goAdds bootdev local-test command, manifest loading, OS gating, and local grading flow.
cmd/localtest_test.goTests manifest loading from a lesson directory (cli.yaml).
client/lessons.goAdds yaml: struct tags to support manifest unmarshalling into existing lesson data types.
checks/local.goAdds local result evaluation + event creation used by local-test.
checks/local_test.goTests local grading logic for CLI/HTTP results and jq evaluation.
checks/jq_test.goUpdates fixture strings in jq tests (no behavior change).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment threadcmd/localtest.go
Comment threadchecks/local.go
@theodore-s-beers
theodore-s-beers merged commit e02ffef into mainJun 11, 2026
1 check passed
@theodore-s-beers
theodore-s-beers deleted the tsb_localtest branch June 11, 2026 14:13
Sign up for freeto 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.

3 participants

@theodore-s-beers@squashd