test(agent): split server exec tests by concern - #23225
Conversation
Preserve all 37 existing tests while separating message, websocket I/O, resize queue, and handler coverage. Keep the websocket fixture in a shared test helper. Fixes kubestellar#23218 Signed-off-by: eunwoo song <presentsong@naver.com>
✅ Deploy Preview for kubestellarconsole canceled.Built without sensitive environment variables
|
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
👋 Welcome to the KubeStellar community! 💖 Thanks and congrats 🎉 for opening your first PR here! We're excited to have you contributing. Before merge, please ensure:
📬 If you're using KubeStellar in your organization, please add your name to our Adopters list. 🙏 It really helps the project gain momentum and credibility — a small contribution back with a big impact. Resources:
A maintainer will review your PR soon. Hope you have a great time here! 🌟 ~~~~~~~~~~ 🌟 📬 If you like KubeStellar, please ⭐ star ⭐ our repo to support it! 🙏 It really helps the project gain momentum and credibility — a small contribution back with a big impact. |
📌 Fixes
Fixes #23218
📝 Summary of Changes
Changes Made
Checklist
Screenshots or Logs
go test ./pkg/agent/ -run 'AgentExec|AgentWS|AgentTerminal|HandleExec|ExecConstants' -count=1go test ./pkg/agent/ -count=1gofmtandgit diff --check👀 Reviewer Notes
The test-function inventory is unchanged: 37 before and after the split.