-
Notifications
You must be signed in to change notification settings - Fork 83
feat(integration-tests): Install CLP Python projects to enable their use in integration tests and fix new linting issues: #1549
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 18 commits
4af1365
11f4522
857744d
d302360
8418328
7774d07
f595d5d
b1bbe3f
a14fb77
8c2ba26
6f2f7e0
791ff0a
fdcf80e
2972779
ab7b791
d8d9baa
c9986d4
b563ec0
3ff0499
71dfbef
f27df5e
f4364d3
5729abc
f7823e5
1309d3b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -31,6 +31,10 @@ dev = [ | |
| "ruff>=0.11.12", | ||
| ] | ||
|
|
||
| [tool.hatch.build.targets.wheel] | ||
| packages = ["clp_mcp_server"] | ||
| include = ["clp_mcp_server/py.typed"] | ||
|
|
||
|
||
| [tool.hatch.metadata] | ||
| allow-direct-references = true | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| """Smoke test ensuring CLP native Python projects import without errors.""" | ||
Bill-hbrhbr marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
|
|
||
| from clp_mcp_server.constants import QueryJobType | ||
| from clp_package_utils.general import JobType | ||
| from clp_py_utils.clp_config import StorageEngine | ||
| from job_orchestration.scheduler.constants import CompressionJobStatus | ||
|
|
||
|
|
||
| def test_clp_native_py_project_enum_classes() -> None: | ||
| """ | ||
| Verifies that the following CLP native Python projects import successfully: | ||
| - `clp-mcp-server` | ||
| - `clp-package-utils` | ||
| - `clp-py-utils` | ||
| - `job-orchestration` | ||
| by testing conversions between their representative enum classes and literal values. | ||
Bill-hbrhbr marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| """ | ||
| assert QueryJobType.SEARCH_OR_AGGREGATION == QueryJobType(0) | ||
| assert JobType.COMPRESSION == JobType("compression") | ||
| assert StorageEngine.CLP == StorageEngine("clp") | ||
| assert CompressionJobStatus.PENDING == CompressionJobStatus(0) | ||
Large diffs are not rendered by default.
Uh oh!
There was an error while loading. Please reload this page.