Skip to content

Conversation

IgorChvyrov-sm
Copy link
Contributor

@IgorChvyrov-sm IgorChvyrov-sm commented Aug 6, 2025

Changes:

  1. Generated API clients using openapi-genetrator tool. (no need to review due to autogen)
  2. Generated pydantic models using openapi-generator tool. (no need to review due to autogen)
  3. Implemented adapters for API clients. (code review required)
  4. Implemented and refactored adapters for pydantic models. (code review required)
  5. Implemented async OrkesClients. (code review required)
  6. Implemented serdeser tests for pydantic models. (code review required)
  7. Implemented unit tests for async OrkesClients. (code review required)

Reason:

Adding an asynchronous client.

Copy link

codecov bot commented Aug 6, 2025

Codecov Report

❌ Patch coverage is 72.27187% with 1024 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ctor/asyncio_client/http/api/admin_resource_api.py 25.13% 143 Missing ⚠️
...ncio_client/http/api/authorization_resource_api.py 27.06% 97 Missing ⚠️
...ctor/asyncio_client/configuration/configuration.py 54.42% 67 Missing ⚠️
...conductor/asyncio_client/automator/task_handler.py 65.16% 54 Missing ⚠️
.../conductor/asyncio_client/automator/task_runner.py 80.53% 29 Missing ⚠️
...ctor/asyncio_client/adapters/api_client_adapter.py 38.23% 21 Missing ⚠️
...cio_client/adapters/models/workflow_run_adapter.py 44.44% 20 Missing ⚠️
...asyncio_client/adapters/models/workflow_adapter.py 63.33% 11 Missing ⚠️
...ctor/asyncio_client/adapters/models/any_adapter.py 72.72% 6 Missing ⚠️
...ncio_client/adapters/models/declaration_adapter.py 72.72% 6 Missing ⚠️
... and 118 more
Files with missing lines Coverage Δ
...io_client/adapters/api/application_resource_api.py 100.00% <100.00%> (ø)
..._client/adapters/api/authorization_resource_api.py 100.00% <100.00%> (ø)
.../asyncio_client/adapters/api/event_resource_api.py 100.00% <100.00%> (ø)
.../asyncio_client/adapters/api/group_resource_api.py 100.00% <100.00%> (ø)
...io_client/adapters/api/integration_resource_api.py 100.00% <100.00%> (ø)
...yncio_client/adapters/api/metadata_resource_api.py 100.00% <100.00%> (ø)
...asyncio_client/adapters/api/prompt_resource_api.py 100.00% <100.00%> (ø)
...ncio_client/adapters/api/scheduler_resource_api.py 100.00% <100.00%> (ø)
...asyncio_client/adapters/api/schema_resource_api.py 100.00% <100.00%> (ø)
...asyncio_client/adapters/api/secret_resource_api.py 100.00% <100.00%> (ø)
... and 223 more

... and 333 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@IgorChvyrov-sm IgorChvyrov-sm marked this pull request as ready for review August 11, 2025 05:59
@IgorChvyrov-sm IgorChvyrov-sm merged commit 813791b into main Aug 27, 2025
2 checks passed
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.

2 participants