Skip to content

test: intentionally broken boombox submission [NAT-133]#57

Open
tallnato wants to merge 1 commit intomainfrom
paperclip/backelino/NAT-133
Open

test: intentionally broken boombox submission [NAT-133]#57
tallnato wants to merge 1 commit intomainfrom
paperclip/backelino/NAT-133

Conversation

@tallnato
Copy link
Copy Markdown
Member

Summary

  • Adds a test boombox entry (content/_boombox/test-broken-boombox/) with 4 intentional validation errors
  • Missing audio field in frontmatter
  • Wrong file extension (.aac instead of .mp3/.wav/.flac)
  • Missing description field in frontmatter
  • Invalid tag not in content/tags.yaml

Expected CI Behavior

Both validate-pr.yml and validate-boombox.yml should fail with these errors:

boombox/test-broken-boombox.md: missing required field "description"
boombox/test-broken-boombox.md: unknown tag "definitely-not-a-real-tag"
boombox/test-broken-boombox/broken-track.aac: invalid file type ".aac" (allowed: .mp3, .wav, .flac)
boombox/test-broken-boombox.md: missing required field "audio"

Test plan

  • Local validation confirms all 4 errors detected
  • CI Validate PR workflow fails
  • CI Validate Boombox Content workflow fails
  • Results documented in PR comments

🤖 Generated with Claude Code

Adds a test boombox entry with 4 intentional errors:
- Missing `audio` field in frontmatter
- Wrong file extension (.aac instead of .mp3/.wav/.flac)
- Missing `description` field in frontmatter
- Invalid tag not in tags.yaml

Co-Authored-By: Paperclip <noreply@paperclip.ing>
@github-actions github-actions bot added the boombox Changes to boombox content label Mar 27, 2026
@tallnato
Copy link
Copy Markdown
Member Author

CI Results (Expected Failures)

Both validation workflows failed as expected:

Check Status
Validate PR Failed
Validate Boombox Content Failed
Semantic PR Passed
Label Passed
Single Component Check Passed

Validation errors detected:

  1. boombox/test-broken-boombox.md: missing required field "description"
  2. boombox/test-broken-boombox.md: unknown tag "definitely-not-a-real-tag"
  3. boombox/test-broken-boombox/broken-track.aac: invalid file type ".aac" (allowed: .mp3, .wav, .flac)
  4. boombox/test-broken-boombox.md: missing required field "audio"

Conclusion: Validation pipeline correctly catches all 4 intentional errors.

Copy link
Copy Markdown
Member Author

@tallnato tallnato left a comment

Choose a reason for hiding this comment

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

Review: Looks Good ✓

Intentional test PR — all 4 validation errors are present and CI correctly rejects the submission.

Checklist:

  • ✅ Missing audio field in frontmatter
  • ✅ Wrong file extension (.aac instead of .mp3/.wav/.flac)
  • ✅ Missing description field in frontmatter
  • ✅ Invalid tag (definitely-not-a-real-tag)
  • Validate PR workflow failed as expected
  • Validate Boombox Content workflow failed as expected
  • ✅ Changes scoped to content/_boombox/ only — no unrelated files

No security, correctness, or scoping issues. LGTM for the stated test purpose.

Note: This PR should NOT be merged to main since it contains intentionally invalid content. Close it once test results are documented, or keep it open as a reference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

boombox Changes to boombox content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant