NotificationsYou must be signed in to change notification settings
Fork928
Star10.1k

chore: add test for enterprise server cli#12353

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

johnstcn merged 2 commits intomainfromcj/enterprise-server-test

Feb 29, 2024

Merged

chore: add test for enterprise server cli#12353

johnstcn merged 2 commits intomainfromcj/enterprise-server-test

Feb 29, 2024

Conversation

Copy link

Member

johnstcn commentedFeb 28, 2024

To validate that this exercises the code path, add a panic intoenterprise/cli/server.go and run the test:

=== RUN   TestServer=== PAUSE TestServer=== CONT  TestServer    clitest.go:75: invoking command: coder --global-config /tmp/TestServer19218759/001 server --in-memory --http-address :40317 --access-url http://example.com    t.go:99: 2024-02-28 17:00:39.503 [info]  cli: stdout: Started HTTP listener at http://[::]:40317    t.go:99: 2024-02-28 17:00:39.503 [info]  cli: stdout: View the Web UI: http://example.com    t.go:99: 2024-02-28 17:00:39.503 [warn]  cli: telemetry disabled, unable to notify of security issues. Read more: https://coder.com/docs/v2/latest/admin/telemetry    t.go:99: 2024-02-28 17:00:39.504 [info]  cli: stderr: 2024-02-28 17:00:39.503 [warn]  cli: telemetry disabled, unable to notify of security issues. Read more: https://coder.com/docs/v2/latest/admin/telemetrypanic: got here [recovered]        panic: panic recovered for coder server: got hereFAIL    github.com/coder/coder/v2/enterprise/cli        0.098sFAIL