Skip to content

test(client): add DebugConfig tests for GET /debug/config - #150

Merged
ferhimedamine merged 1 commit into
mainfrom
test/debug-config
Aug 2, 2026
Merged

test(client): add DebugConfig tests for GET /debug/config#150
ferhimedamine merged 1 commit into
mainfrom
test/debug-config

Conversation

@ferhimedamine

Copy link
Copy Markdown
Contributor

Summary

  • TestDebugConfig: verifies DebugConfig() calls GET /debug/config and deserializes the env-var map correctly
  • TestDebugConfig_Forbidden: verifies 403 error propagates when caller lacks Admin scope

Context

DebugConfig() was added to the Go SDK but had no test coverage. This mirrors the async parity tests added to dakera-py in PR#191 (merged). Both SDKs now have parity tests for the GET /debug/config endpoint.

Pattern follows TestAuthorizationHeader / TestUserAgentHeaderhttptest.NewServer capturing request details.

DAK-7477

🤖 Generated with Claude Code

Covers two behaviors:
- DebugConfig calls GET /debug/config and deserializes the env-var map
- DebugConfig propagates 403 when caller lacks Admin scope

Mirrors the async parity tests added to dakera-py in PR#191.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
@ferhimedamine ferhimedamine added the auto-merge Auto-merge when CI passes label Aug 2, 2026
@ferhimedamine
ferhimedamine merged commit 1fece17 into main Aug 2, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Auto-merge when CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant