Skip to content

chore(core): cover source cdn bundle entry in unit tests - #254

Merged
coryrylan merged 1 commit into
mainfrom
topic/upkeep/coverage/bundle-source-entry
Aug 28, 2026
Merged

chore(core): cover source cdn bundle entry in unit tests#254
coryrylan merged 1 commit into
mainfrom
topic/upkeep/coverage/bundle-source-entry

Conversation

@cursor

@cursor cursor Bot commented Aug 28, 2026

Copy link
Copy Markdown

Selected task

Coverage for projects/core/src/bundle.ts.

Selector rationale: "1 uncovered lines, 0% lines and 100% branches, the smallest gap below the 90% threshold".

The existing suite imported only ../dist/bundles/index.js, so istanbul never recorded the source entry.

Measurement

Lines Branches
Before 0/1 (0%) 0/0 (100%)
After 1/1 (100%) 0/0 (100%)

projects/core unit tests: 1888 passed, including the 3 cdn bundle cases.

Public API

No public API changed. Production bundle.ts is untouched. Behavior is unchanged.

Left alone

  • Dist-bundle import and custom-element registration assertions remain so the emitted CDN artifact is still checked.
  • Phantom LFS visual PNG dirty files and generated metadata JSON were not staged.

Validation

  • projects/core: lint, test, test:types, test:axe, test:ssr, test:lighthouse (78), test:visual, test:coverage
  • root pnpm run ci
  • git diff --check on the changed file
Open in Web View Automation 

imports the source bundle entry so istanbul records the global
nve_elements.state.bundle assignment. line coverage rises from
0% to 100%. no public api or behavior change.

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
@coryrylan coryrylan added the upkeep label Aug 28, 2026 — with Cursor
@cursor
cursor Bot marked this pull request as ready for review August 28, 2026 06:23
@coryrylan
coryrylan merged commit eefd842 into main Aug 28, 2026
15 checks passed
@coryrylan
coryrylan deleted the topic/upkeep/coverage/bundle-source-entry branch August 28, 2026 13:12
@coryrylan

Copy link
Copy Markdown
Collaborator

🎉 This issue has been resolved in version 2.6.0 🎉

Changelog

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants