Fix two Level 2 scoring holes and the PR template's deploy command - #1
Merged
Conversation
…rated container Two scoring holes, both reproduced against real trees: 1. Level 2 awarded 40 raw points into a 35-point cap, so dropping any single baseline check cost nothing. A solved tree with the @secure() parameter deleted printed the FAIL and still scored 35/35 and 100/100, which made the assignment's own "Starter hygiene: @secure() still present" criterion unenforceable. Rebalanced to 20 baseline + 15 extensions = 35 exactly and removed the cap. That same tree now scores 95. 2. The curated check passed on the string alone, so a fresh starter plus a comment reading `// TODO: add a container named 'curated'` collected the full extension while the grader simultaneously warned "only 1 nested container resource(s)". Now requires a second container resource AND the name. Fresh starter still scores 40/100 with Level 2 at 20/35, so the assignment chapter's published baseline claim is unchanged. Also fixes the PR template, whose "How to run" deployed with `dummySecret` (the starter declares `dbAdminPassword`) and omitted `environment`, and which described a single container although Task 2 requires raw + curated. Adds the teardown prompt to the docs/portal_confirm.md stub. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
📝 HackYourFuture auto gradeAssignment Score: 40 / 100 ❌Status: ❌ Not passed Test Details |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What I built
Two grader scoring holes and three PR-template errors, all reproduced against real trees before and after the fix.
Grader (
.hyf/test.sh)curatedcheck passed on the string alone. Now requires a second container resource and the name.PR template
dummySecret=did not exist; the starter declaresdbAdminPassword. Also addedenvironment=, with a note that it only applies after Task 1.raw+curated.Scaffold
docs/portal_confirm.mdprompts for the teardown line the assignment chapter now asks for.How to review
.hyf/test.sh, then thecuratedblock.if [[ "$l2" -gt 35 ]]) is gone and the raw sum is exactly 35.How to run
git switch fix/grader-scoring-holes bash .hyf/test.sh # fresh starter: 40/100, Level 2 20/35To reproduce the holes on
main: delete the@secure()block from a solved tree and run the grader (scores 100/100 before this PR, 95 after); or append// TODO: add a container named 'curated'to a freshmodules/storage.bicep(awards the extension before, not after).What reviewers should see (expected results)
@secure()deleted@secure()+ module deleted'curated'in a commentThe fresh-starter number is deliberately unchanged, because the assignment chapter publishes it.
Known limitations / out of scope
lassebenni/github-actions-fork@fix/allow-unsafe-pr-checkout-for-autograde, a personal fork at a non-default ref, whose upstream PR was closed without merge. Not touched here, but it is a single point of failure for cohort grading.Self-check
bash .hyf/test.shpasses on a solved tree (100/100) and fails a fresh starter (40/100)bash -n .hyf/test.shcleanrg-hyf-students