feat(tests): add e2e test for code-server #643
Merged
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
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds an end-to-end integration test for the code-server module that validates the complete installation and health check workflow. The test runs the installation script in a containerized environment and verifies that code-server is properly installed, executable, and responding to health checks.
Key changes:
- Added e2e test that installs code-server in an Ubuntu container and validates the installation
- Test verifies both the binary installation and the running service health endpoint
- Includes proper resource cleanup using try-finally pattern
matifali
approved these changes
Jan 12, 2026
6 tasks
matifali
added a commit
that referenced
this pull request
Jan 20, 2026
## Description Updates AGENTS.md to be more concise (~36 lines) while adding missing commands and a PR review checklist based on recent PR feedback. ## Changes - **Commands section**: Added `bun run tftest`, `bun run tstest`, single test commands, and version-bump script - **Structure section**: Added note that URLs must be relative (from #639) - **Code Style section**: Added `tf` vs `hcl` guidance and relative icon paths - **New PR Review Checklist**: Based on patterns from recent PRs including: - Version bumping requirements (#661, #617) - Breaking changes documentation (#636) - Graceful error handling in scripts (#658) - Diagnostic logging for tests (#643) - **AI attribution requirement**: PRs should note model/tool used ## Type of Change - [ ] New module - [ ] New template - [ ] Bug fix - [ ] Feature/enhancement - [x] Documentation - [ ] Other --- Generated with [Amp](https://ampcode.com/threads/T-019bcb7e-2e92-76f2-a1aa-2023ecdb0763) using Claude Sonnet 4
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Description
Adds e2e test for code-server that actually installs and health checks code-server
Type of Change
Testing & Validation
bun test)bun fmt)Related Issues