Skip to content

test: add unit tests for prompt_score scoring logic - #62

Closed
TerminalGravity wants to merge 1 commit into
mainfrom
test/prompt-score-unit-tests
Closed

test: add unit tests for prompt_score scoring logic#62
TerminalGravity wants to merge 1 commit into
mainfrom
test/prompt-score-unit-tests

Conversation

@TerminalGravity

Copy link
Copy Markdown
Collaborator

Adds 18 unit tests covering the scorePrompt function:

  • All 4 scoring dimensions (specificity, scope, actionability, done condition)
  • Grade boundary thresholds (A+ through F)
  • Feedback generation for strong and weak prompts
  • Exports scorePrompt for direct testability

All 61 tests passing.

@TerminalGravity TerminalGravity left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Good coverage of all four scoring dimensions. Note that PR #66 supersedes the scope scoring bug fix portion, but these tests are still valuable as standalone prompt_score coverage. If #66 lands first, this may need a rebase to resolve the export conflict. CI green.

@TerminalGravity
TerminalGravity force-pushed the test/prompt-score-unit-tests branch 2 times, most recently from a8f0ea3 to 33b8f6a Compare March 4, 2026 18:44
- Export scorePrompt function for direct testing
- Cover all 4 scoring dimensions (specificity, scope, actionability, done condition)
- Test grading thresholds, edge cases (empty string, long prompts, case insensitivity)
- First tool-level test file (tests/tools/)
@TerminalGravity
TerminalGravity force-pushed the test/prompt-score-unit-tests branch from 33b8f6a to d7dbfe2 Compare March 4, 2026 20:15
@TerminalGravity

Copy link
Copy Markdown
Collaborator Author

Closing — superseded by newer PRs.

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