Skip to content

TPT-4772: Align integration test API configuration - #758

Open
zliang-akamai wants to merge 1 commit into
devfrom
TPT-4772-align-integration-api-env
Open

zliang-akamai wants to merge 1 commit into
devfrom
TPT-4772-align-integration-api-env

Conversation

@zliang-akamai

Copy link
Copy Markdown
Member

📝 Description

Align integration test configuration with linodego and the Terraform provider by replacing the versioned LINODE_API_URL with an unversioned LINODE_URL and separate LINODE_API_VERSION. The test default remains v4beta; the production SDK client is unchanged. Document the configuration migration and cover the URL and version combinations with offline tests.

✔️ How to Test

python -m pytest --noconftest test/integration/linode_client/test_linode_client.py -k test_api_url_environment -q

@zliang-akamai
zliang-akamai requested review from a team as code owners November 11, 2026 18:24
@zliang-akamai
zliang-akamai requested review from ezilber-akamai and mawilk90 and removed request for a team November 11, 2026 18:24
@zliang-akamai zliang-akamai added the testing for updates to the testing suite in the changelog. label Nov 11, 2026
@zliang-akamai
zliang-akamai requested review from a team and psnoch-akamai and a balanced review from Copilot and removed request for a team November 11, 2026 18:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No active review issues were identified for the integration-test API configuration migration.

No open issues 🎉

What changed in this PR

Aligns integration-test API configuration with linodego and the Terraform provider while leaving the production SDK client unchanged.

Changes:

  • Replaces LINODE_API_URL with unversioned LINODE_URL and separate LINODE_API_VERSION, retaining the v4beta default.
  • Adds offline tests for defaults, overrides, trailing slashes, and ignored legacy configuration.
  • Documents the configuration migration.
File Description
test/​integration/​linode_client/​test_linode_client.py Adds parameterized API URL configuration tests.
test/​integration/​conftest.py Builds the API URL from separate base URL and version variables.
README.rst Documents defaults, usage, and migration from LINODE_API_URL.

Review effort: Balanced

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing for updates to the testing suite in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants