Skip to content

fix: test regressions + v3.0.0-beta.14 release#955

Open
lane711 wants to merge 1 commit into
mainfrom
lane711/sonicjs-release-engineer
Open

fix: test regressions + v3.0.0-beta.14 release#955
lane711 wants to merge 1 commit into
mainfrom
lane711/sonicjs-release-engineer

Conversation

@lane711

@lane711 lane711 commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Remove duplicate media_asset registration in document-types-seed.ts (lines 208–227 overwrote internal: true from lines 114–142, causing media_asset to appear in content type dropdowns)
  • Fix version count assertions in integration tests: publish() with versioning-off deletes old published row when no longer current draft — final count is 1, not 2
  • Fix swapped args in on-cron-tick.test.ts: calls used (ctx, event) but function signature is (event, ctx)
  • Bump version to 3.0.0-beta.14 (skipping beta.13, which was published from a diverged branch)

Test plan

  • All 3 previously failing test groups now pass
  • Full suite: 1565 passed, 0 failed, 247 skipped
  • @sonicjs-cms/core@3.0.0-beta.14 published with latest and beta dist-tags
  • create-sonicjs@3.0.0-beta.14 published with latest and beta dist-tags

🤖 Generated with Claude Code

… v3.0.0-beta.14

- document-types-seed: remove duplicate media_asset registration (lines 208–227)
  that overwrote internal:true, causing media_asset to appear in content dropdowns
- admin-content-docbacked + api-content-crud tests: update COUNT assertion from 2→1;
  versioning-off publish() deletes old published row so only v2 survives
- on-cron-tick tests: fix swapped (ctx, event) → (event, ctx) argument order
- Bump version to 3.0.0-beta.14 (skip beta.13, published from diverged branch)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@lane711 lane711 force-pushed the lane711/sonicjs-release-engineer branch from 94684a7 to 9c55a4d Compare June 26, 2026 16:08
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