Skip to content

DO NOT MERGE: planted failing test proves the Unit Tests gate (D-0100) - #134

Closed
acamarata wants to merge 2 commits into
mainfrom
debt/d0100-planted-fail
Closed

acamarata wants to merge 2 commits into
mainfrom
debt/d0100-planted-fail

Conversation

@acamarata

Copy link
Copy Markdown
Collaborator

Throwaway. Stacked on #133 plus one planted failing test. Expected: Test / Unit Tests fails. Closed and deleted once the run is recorded.

… key prefixes (D-0100, D-0527)

test.yml ran 'pnpm test -- --coverage --ci --passWithNoTests'. pnpm forwards
the literal '--', so jest read the flags as path patterns, matched nothing and
exited 0. Run 'pnpm run test:ci' instead. jest now skips the ticket worktree
folder so a main-checkout run does not also run every worktree's suite copy.

API keys used nself_ + 2 letters as the lookup prefix and validated only the
first record with it, so a colliding key never validated. New keys get
10 random base32 chars; validation checks every record with the prefix by
hash, so legacy 2-letter keys keep working.
@github-actions

Copy link
Copy Markdown

Lighthouse Audit Results

Category Score Status
Performance 79 🟠
Accessibility 100 🟢
Best Practices 96 🟢
SEO 91 🟢

Score Guide

  • 🟢 90-100: Good
  • 🟠 50-89: Needs improvement
  • 🔴 0-49: Poor

View full Lighthouse report

@acamarata acamarata closed this Oct 11, 2026
@acamarata
acamarata deleted the debt/d0100-planted-fail branch October 11, 2026 21:25
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