Add Standard Code Test assessment JSON guide#469
Conversation
WalkthroughThis PR adds a new documentation guide for authoring "Standard Code Test" assessment JSON files in Codio. The guide covers storage location ( ChangesAssessment documentation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@coderabbitai review |
✅ Action performedReview finished.
|
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@source/instructors/authoring/guides/standard_code_test_assessment_json.rst`:
- Around line 430-431: Replace the verbose learningObjectives value (key
"learningObjectives") with a concise plain-language sentence that removes the
unexplained acronym "SWBAT" (e.g., "Students will calculate simple interest
using basic arithmetic"), and while you're editing the adjacent "guidance" value
keep it brief and consistent in tone (e.g., "Implements simple interest
calculation with basic arithmetic and formatting") so both fields read clearly
and match the guide's concise style.
- Around line 67-74: The guide currently shows inconsistent embed token spacing
— `{Check It!|assessment}` vs elsewhere `{Text |assessment}` — so pick one
canonical token format (recommend no space before the pipe, e.g. `{Check
It!|assessment}`) and update this file's example text and the surrounding
guidance to explicitly state the chosen canonical form and that authors must use
it exactly (mention the exact token pattern and that there should be no space
before the `|`), ensuring any other occurrences in related docs are aligned to
the same format.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 8bb6575c-f273-4f95-8352-9df518f6f669
📒 Files selected for processing (2)
source/instructors/authoring/guides/intro.rstsource/instructors/authoring/guides/standard_code_test_assessment_json.rst
|
|
||
| .. _standard-code-test-assessment-json: | ||
|
|
||
| Standard Code Test assessment JSON |
There was a problem hiding this comment.
Assessment should be capitalized
| In the assessment JSON, Standard Code Tests use the assessment type | ||
| ``code-output-compare``. | ||
|
|
||
| File location and naming |
There was a problem hiding this comment.
Change to "File Location and Naming"
|
|
||
| {Check It!|assessment}(code-output-compare-1075712907) | ||
|
|
||
| Use ``{Check It!|assessment}`` exactly as shown, with no spaces before the closing brace. |
There was a problem hiding this comment.
Maybe add this in an important box?
| } | ||
| } | ||
|
|
||
| Field reference |
There was a problem hiding this comment.
Capitalize r in Reference
| Field reference | ||
| --------------- | ||
|
|
||
| Top-level fields |
There was a problem hiding this comment.
Change to "Top-Level Fields"
| } | ||
| } | ||
|
|
||
| Standard input |
| } | ||
| } | ||
|
|
||
| Matching a substring |
| Set ``matchSubstring`` to ``false`` when the learner's output should match the | ||
| expected output more strictly, subject to the other comparison options. | ||
|
|
||
| Assessment files and opened files |
There was a problem hiding this comment.
Change to "Assessment Files and Opened Files"
| ] | ||
| } | ||
|
|
||
| Guide embedding example |
There was a problem hiding this comment.
Change to "Guide Embedding Example"
|
|
||
| {Check It!|assessment}(code-output-compare-165558969) | ||
|
|
||
| Authoring notes |
Summary by CodeRabbit