Skip to content

chore: clarify fs routes testing - #3907

Open
ryux1 wants to merge 1 commit into
freshframework:mainfrom
ryux1:docs/fs-routes-testing
Open

ryux1 wants to merge 1 commit into
freshframework:mainfrom
ryux1:docs/fs-routes-testing

Conversation

@ryux1

@ryux1 ryux1 commented Sep 17, 2026

Copy link
Copy Markdown

What this PR does

Adds an info note to the route-testing guide that:

  • .fsRoutes() consumes route data from Fresh's build cache rather than scanning routes/ during a direct unit test
  • unit tests should register imported route handlers explicitly
  • tests that need filesystem discovery, layouts, or islands should use the documented full Vite build path

This addresses the confusing all-404 behavior discussed in Q&A #3906.

Validation

  • deno fmt --check docs/latest/testing/index.md
  • deno task check:docs

@ryux1

ryux1 commented Oct 8, 2026

Copy link
Copy Markdown
Author

Apologies for the delayed follow-up. I rechecked this against the current discussion and the change remains a focused documentation clarification: direct .fsRoutes() tests consume the build cache, while filesystem discovery requires the documented Vite build path. If this is still useful for the testing guide, it is ready for review; if the documentation direction has changed, please let me know and I will adjust the scope.

This branch has not been deployed

No deployments
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