Skip to content

docs: Add guidance for running tests with proper permissions#152

Merged
jaypatrick merged 2 commits intomasterfrom
docs/test-permissions-guide
Jan 14, 2026
Merged

docs: Add guidance for running tests with proper permissions#152
jaypatrick merged 2 commits intomasterfrom
docs/test-permissions-guide

Conversation

@jaypatrick
Copy link
Owner

Problem

Developers running deno test directly encounter permission errors because the tests require specific Deno permissions (--allow-read, --allow-write, --allow-net, --allow-env, --unstable-kv) that are configured in the deno.json task.

Solution

Added clear documentation to the README explaining:

  • Always use deno task test instead of bare deno test
  • Listed all required permissions with explanation
  • Updated the example for running specific test files to include the necessary flags

Changes

  • Added a prominent note in the "Available tasks" section
  • Updated the example command for running specific test files
  • Helps prevent confusion and test failures due to missing permissions

Testing

Verified that the documentation is clear and the examples are correct.

Co-Authored-By: Warp agent@warp.dev

jaypatrick and others added 2 commits January 13, 2026 23:40
- Add note that deno task test should be used instead of bare deno test
- Explain required permissions (--allow-read, --allow-write, --allow-net, --allow-env, --unstable-kv)
- Update example for running specific test files to include permissions

This helps developers avoid permission errors when running tests directly.

Co-Authored-By: Warp <agent@warp.dev>
@github-actions
Copy link

Version bumped: 0.7.19 -> 0.7.20

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 14, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
adblock-compiler 24493b0 Jan 14 2026, 04:41 AM

@jaypatrick jaypatrick merged commit b310788 into master Jan 14, 2026
1 check passed
@jaypatrick jaypatrick deleted the docs/test-permissions-guide branch January 14, 2026 04:41
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