test(python-sdk): drop build API path encoding integration tests - #1709
test(python-sdk): drop build API path encoding integration tests#1709mishushakov wants to merge 1 commit into
Conversation
Remove test_build_api_path_encoding.py; encode_path_param is already covered by tests/shared/api/test_encode_path_param.py. Co-authored-by: Mish Ushakov <mishushakov@users.noreply.github.com>
|
Package ArtifactsBuilt from f416baa. Download artifacts from this workflow run. JS SDK ( npm install ./e2b-2.41.1-cursor-drop-build-api-path-encoding-tests-3bae.0.tgzCLI ( npm install ./e2b-cli-2.16.3-cursor-drop-build-api-path-encoding-tests-3bae.0.tgzPython SDK ( pip install ./e2b-2.41.0+cursor.drop.build.api.path.encoding.tests.3bae-py3-none-any.whl |
|
/sdk claim |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Claimed into #1713.
The commit (65c96289) was applied unmodified, so the new branch is byte-identical to this PR's head, and the original commit authorship and Co-authored-by trailer are preserved. The branch was already current with main, so no merge was needed. The contents were not reviewed or changed, and the description was carried over verbatim.
One thing I could not do: please close this PR manually in favour of #1713. I have no tool for closing a pull request (the gh CLI available to me is read-only), and Closes #1709 would not work either since GitHub only auto-closes issues, not pull requests, from a PR body.
Sent by Cursor Automation: /claim Claim SDK PR by SDK Factory
Supersedes #1709 — claimed via `/sdk claim` by @mishushakov. This is a clone of #1709: the original commit (`65c96289`) is applied unmodified, so the tree here is byte-identical to that PR's head and the original commit authorship and `Co-authored-by` trailer are preserved. The branch was already current with `main` (1 commit ahead, 0 behind), so no merge was needed. The contents were not reviewed or changed. Please close #1709 in favour of this PR. The original description follows verbatim. --- ## Summary Removes `tests/shared/template/test_build_api_path_encoding.py`. Path encoding is already covered by `tests/shared/api/test_encode_path_param.py`. ## Verification ```bash cd packages/python-sdk uv run pytest tests/shared/api/test_encode_path_param.py tests/shared -q # 175 passed, 1 skipped ``` [Slack Thread](https://e2b-team.slack.com/archives/D0962B9UKEE/p1786973222264879?thread_ts=1786973222.264879&cid=D0962B9UKEE) <div><a href="https://cursor.com/agents/bc-57d27899-5769-437a-a242-7956988cdb1f?cursor_ref=pr_footer&cursor_cta=open_in_web"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/open-in-web-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/open-in-web-light.png"><img alt="Open in Web" width="114" height="28" src="https://cursor.com/assets/images/open-in-web-dark.png"></picture></a> <a href="https://cursor.com/automations/3b1a5376-9bd3-11f1-ba66-0e7d0216e441"><picture><source media="(prefers-color-scheme: dark)" srcset="https://cursor.com/assets/images/view-automation-dark.png"><source media="(prefers-color-scheme: light)" srcset="https://cursor.com/assets/images/view-automation-light.png"><img alt="View Automation" width="141" height="28" src="https://cursor.com/assets/images/view-automation-dark.png"></picture></a> </div> Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Mish Ushakov <mishushakov@users.noreply.github.com>
![https://cursor.com/assets/images/open-in-web-dark.png"><source](https://cursor.com/assets/images/open-in-web-dark.png"><source){kind=link}
![https://cursor.com/assets/images/open-in-web-light.png"><img](https://cursor.com/assets/images/open-in-web-light.png"><img){kind=link}
![https://cursor.com/assets/images/open-in-web-dark.png"></picture></a> <a](https://cursor.com/assets/images/open-in-web-dark.png"></picture></a> <a){kind=link}
![https://cursor.com/assets/images/view-automation-dark.png"><source](https://cursor.com/assets/images/view-automation-dark.png"><source){kind=link}
![https://cursor.com/assets/images/view-automation-light.png"><img](https://cursor.com/assets/images/view-automation-light.png"><img){kind=link}
![https://cursor.com/assets/images/view-automation-dark.png"></picture></a> </div](https://cursor.com/assets/images/view-automation-dark.png"></picture></a> </div){kind=link}
2 participants
Summary
Removes
tests/shared/template/test_build_api_path_encoding.py. Path encoding is already covered bytests/shared/api/test_encode_path_param.py.Verification
Slack Thread