Full integration test - #167
Closed
kevihan-te wants to merge 7 commits into
Closed
Conversation
Move the integration mock server into test-only sdk_test_support helpers and expose ApiException.exception_class_for_http_status for generated integration error assertions. Co-authored-by: Cursor <cursoragent@cursor.com>
Sync OAS-driven integration test artifacts (mock manifest, test base, conftest, and per-operation integration tests) across 21 packages for CP-2453 full rollout evaluation. Co-authored-by: Cursor <cursoragent@cursor.com>
kevihan-te
requested review from
brumarqu-te,
goncarva-te,
jabrowne-te,
mpragosa-te,
phpinhei-te,
rodrirod-te,
sfreitas-te and
stulowes-te
as code owners
July 28, 2026 11:07
kevihan-te
changed the title
Success response bodies now use valid JSON quotes instead of HTML entities, and optional expand enum params are omitted when OpenAPI examples are invalid. Co-authored-by: Cursor <cursoragent@cursor.com>
Treat Z and +00:00 as equivalent when comparing serialized SDK request bodies to OAS examples, fixing alert suppression window integration tests. Co-authored-by: Cursor <cursoragent@cursor.com>
Recursively ignore read-only fields in nested request objects and regenerate dashboards/streaming integration tests for list responses. Co-authored-by: Cursor <cursoragent@cursor.com>
Regenerate endpoint-agents and event-detection integration tests with Python booleans, sync get_events ongoing param support, and compare responses using only fields present in constructed models. Co-authored-by: Cursor <cursoragent@cursor.com>
☂️ Python Coverage
Overall Coverage
New Files
Modified Files
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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
MockApiServer, exception status mapping) inthousandeyes-sdk-coremock_manifest.py,integration_test_utils.py,conftest.py, and per-operationtest_*_integration.pymodulesScope
Excluded:
thousandeyes-sdk-bgp-monitors(missing OAS spec in generator)Status
Do not merge — This PR is only for preview the final test result
Related generator PR: https://github.com/thousandeyes/thousandeyes-sdk-generator/pull/79
Test plan
pip install -e thousandeyes-sdk-core+ target packages)pytest thousandeyes-sdk-core/test/test_mock_server.pyMade with Cursor