Skip to content

test: add tests for StaticAssetsController - #3003

Merged
jelveh merged 2 commits into
mainfrom
DS/2974
May 8, 2026
Merged

test: add tests for StaticAssetsController#3003
jelveh merged 2 commits into
mainfrom
DS/2974

Conversation

@Salazareo

Copy link
Copy Markdown
Member

Covers each config-gated branch (client_libs_root, puterjs_root, gui_assets_root, builtin_apps) using real temp directories so the existsSync checks see real files instead of mocked module behavior. Asserts both registration outcomes (which routes appear, on which subdomains) and the file resolution rules for the puter.js / puter.dev.js fallback.

Closes #2974

@github-actions

github-actions Bot commented May 8, 2026

Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
馃數 Lines 31.05%
猬嗭笍 +0.08%
5457 / 17572
馃數 Statements 30.3%
猬嗭笍 +0.08%
5777 / 19062
馃數 Functions 32.94%
猬嗭笍 +0.20%
984 / 2987
馃數 Branches 19.91%
猬嗭笍 +0.06%
2794 / 14030
File CoverageNo changed files found.
Generated in workflow #62 for commit fea489f by the Vitest Coverage Report Action

Covers each config-gated branch (client_libs_root, puterjs_root,
gui_assets_root, builtin_apps) using real temp directories so the
existsSync checks see real files instead of mocked module behavior.
Asserts both registration outcomes (which routes appear, on which
subdomains) and the file resolution rules for the puter.js / puter.dev.js
fallback.

Closes #2974
@jelveh
jelveh merged commit 508d652 into main May 8, 2026
4 checks passed
@jelveh
jelveh deleted the DS/2974 branch May 8, 2026 04:45
junphine pushed a commit to junphine/puter that referenced this pull request May 21, 2026
Covers each config-gated branch (client_libs_root, puterjs_root,
gui_assets_root, builtin_apps) using real temp directories so the
existsSync checks see real files instead of mocked module behavior.
Asserts both registration outcomes (which routes appear, on which
subdomains) and the file resolution rules for the puter.js / puter.dev.js
fallback.

Closes HeyPuter#2974
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.

[Tests] Add tests for StaticAssetsController

2 participants