Skip to content

Add a unit test to ensure base URL with a path is handled correctly#14063

Merged
JamieMagee merged 1 commit intomainfrom
jamiemagee/api_client_path_test
Jan 30, 2026
Merged

Add a unit test to ensure base URL with a path is handled correctly#14063
JamieMagee merged 1 commit intomainfrom
jamiemagee/api_client_path_test

Conversation

@JamieMagee
Copy link
Copy Markdown
Member

What are you trying to accomplish?

This is a follow-up from #13800 and #14058 to prevent a regression

Anything you want to highlight for special attention from reviewers?

How will you know you've accomplished your goal?

Checklist

  • I have run the complete test suite to ensure all tests and linters pass.
  • I have thoroughly tested my code changes to ensure they work as expected, including adding additional tests for new functionality.
  • I have written clear and descriptive commit messages.
  • I have provided a detailed description of the changes in the pull request, including the problem it addresses, how it fixes the problem, and any relevant details about the implementation.
  • I have ensured that the code is well-documented and easy to understand.

@JamieMagee JamieMagee requested a review from a team as a code owner January 30, 2026 21:23
@JamieMagee JamieMagee merged commit f4178da into main Jan 30, 2026
195 checks passed
@JamieMagee JamieMagee deleted the jamiemagee/api_client_path_test branch January 30, 2026 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants