test(integration): enable Cloudflare integration tests in CI #88
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add CLOUDFLARE_AUTH environment variable to CI workflow - Update cloudflare.integration.js to use process.env.CLOUDFLARE_AUTH - Update account ID and test domain for current Cloudflare account - Remove .skip from Cloudflare integration test Fixes #87 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Lars Trieloff <lars@trieloff.net>
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
- Add enableSubdomain() method to CloudflareDeployer - Call enableSubdomain() after worker deployment - Use POST method to enable subdomain (not PUT) - Increase retry404 from 0 to 5 for propagation delays - Requires Workers Scripts:Edit permission on API token This fixes the issue where deployed workers weren't accessible on workers.dev due to subdomain not being enabled. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Lars Trieloff <lars@trieloff.net>
|
This PR will trigger a patch release when merged. |
- Add POST /subdomain mock to both CloudflareDeployer tests - Fixes unit test failures from enableSubdomain() addition 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Lars Trieloff <lars@trieloff.net>
- Update test assertion from rockerduck to minivelos - Matches updated account configuration 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Lars Trieloff <lars@trieloff.net>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
CLOUDFLARE_AUTHsecret to CI workflow environmentChanges
.github/workflows/main.yaml:36- AddCLOUDFLARE_AUTHenvironment variabletest/cloudflare.integration.js:38- Remove.skipto enable testtest/cloudflare.integration.js:50-52- Update account ID, test domain, and add auth parameterTest Plan
Fixes #87
🤖 Generated with Claude Code