fix: export no-auth test helpers from main module #30
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
The no-auth test helpers (test_agent_no_auth, test_agent_a2a_no_auth, and their config constants) were added in PR #29 and exist in the adcp.testing module, but were not exported from the main adcp module. This meant users had to use: from adcp.testing import test_agent_no_auth Instead of the expected: from adcp import test_agent_no_auth This commit adds the missing exports to match the other test helpers that are already exported. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
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
Export the no-auth test helpers (
test_agent_no_auth,test_agent_a2a_no_auth) and their config constants from the mainadcpmodule. These helpers exist inadcp.testingbut weren't exposed at the top level, forcing users to use a less discoverable import path.Changes
Added 4 exports to
src/adcp/__init__.py:test_agent_no_authandtest_agent_a2a_no_auth(client instances)TEST_AGENT_MCP_NO_AUTH_CONFIGandTEST_AGENT_A2A_NO_AUTH_CONFIG(config objects)Users can now use:
from adcp import test_agent_no_authinstead offrom adcp.testing import test_agent_no_auth🤖 Generated with Claude Code