Uh oh!
There was an error while loading. Please reload this page.
test: add slim smoke tests and verify test suite for aignostics-sdk split [PYSDK-141] - #659
Closed
ari-nz wants to merge 1 commit into
Closed
test: add slim smoke tests and verify test suite for aignostics-sdk split [PYSDK-141]#659ari-nz wants to merge 1 commit into
ari-nz wants to merge 1 commit into
Conversation
…plit [PYSDK-141]
- Add `slim` pytest marker to pyproject.toml for aignostics-sdk package tests
- Create tests/aignostics_sdk/__init__.py and smoke_test.py
- 5 smoke tests verify importability of aignostics_sdk.platform.Client,
aignostics_sdk.utils.{BaseService,Health}, aignx.codegen.exceptions.ApiException,
__project_name__ backward-compat constant, and __version__ availability
- 1 xfail test documents missing aignostics_sdk.cli module (pending PYSDK-137)
- No stale `from aignostics.platform` / `from aignostics.utils` imports in tests/
- Collection: 904 tests collected (0 collection errors)
- Pre-existing failures in base branch (stale patch() paths from PYSDK-136
import rewrite) are not introduced by this PR; they will be tracked separatelyContributor
There was a problem hiding this comment.
Pull request overview
Adds initial smoke-test coverage for the new aignostics-sdk slim distribution path and registers a dedicated pytest marker so these tests can be selected under strict marker validation.
Changes:
- Adds the
slimpytest marker. - Adds smoke tests for core
aignostics_sdkand bundledaignximports, constants, version availability, and pending CLI behavior. - Adds a package marker file for the new
tests/aignostics_sdktest area.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
pyproject.toml | Registers the new slim pytest marker. |
tests/aignostics_sdk/__init__.py | Adds the test package docstring for slim SDK tests. |
tests/aignostics_sdk/smoke_test.py | Adds slim SDK smoke tests for imports, constants, version, and pending CLI entry behavior. |
| assert ApiException is not None | ||
| @pytest.mark.unit |
Comment on lines
+64
to
+65
| result = subprocess.run( | ||
| [sys.executable, "-m", "aignostics_sdk.cli", "--help"], |
This was referenced May 28, 2026
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 freeto 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.
Release Train — PYSDK-133
Summary
slimpytest marker inpyproject.tomlfor tests targeting theaignostics-sdkslim package distributiontests/aignostics_sdk/__init__.pyandtests/aignostics_sdk/smoke_test.pywith 6 tests (5 passing, 1 xfail)from aignostics.platform/from aignostics.utilsimports in test filesSmoke test coverage
test_platform_client_importableaignostics_sdk.platform.Clientresolvestest_utils_importableaignostics_sdk.utils.{BaseService,Health}resolvetest_aignx_codegen_importableaignx.codegen.exceptions.ApiExceptionresolvestest_project_name_preserved__project_name__ == "aignostics"for backward compattest_version_available__version__is non-emptytest_slim_cli_entry_pointaignostics_sdk.climodule pending PYSDK-137Known pre-existing failures (not introduced here)
The base branch
feat/PYSDK-136/import-rewritealready has 116 failing unit tests and 93 errors from stalepatch("aignostics.platform._authentication...")and similar mock paths that were not updated when PYSDK-136 rewrote thefrom aignostics.*imports. These will be addressed in a follow-up fix to PYSDK-136 and are not in scope for this PR.Deferred work
aignostics_sdk.climodule carve-out —test_slim_cli_entry_pointwill flip from xfail to green once that landsopenslide,nicegui, etc.) are still present inaignostics-sdk; the "heavy deps not present" assertions from the original task spec are deferred until PYSDK-138 mergesThis PR was created by Claude (claude-sonnet-4-6) on behalf of Ari Angelo (ari@aignostics.com).