sidshas03 commented on PR #55702:
URL: https://github.com/apache/airflow/pull/55702#issuecomment-3310148143
**What I updated in this PR:**
* **airflow-ctl:** fixed `assets create` and `assets create-event` wiring.
* **Tests (ctl):** replaced `httpx`/transport mocks with
`pytest.monkeypatch` so the lane doesn’t depend on extra packages; assertions
verify:
* `create()` → `POST /assets` with `{name, uri}`
* `create_event()` → `POST /assets/events` with `{asset_id, extra}`
(defaults to `{}`)
* **OpenAPI & client:** regenerated OpenAPI spec and refreshed the Python
client so new endpoints are present.
* **Datamodels:** ensured `CreateAssetBody` is present and `group` is
treated as a required `str`.
* **Static/typing:** cleaned imports and Py3.10-friendly typing; local
Breeze runs show ctl tests and python-client tests passing.
Latest commit: **b1782f7**.
Once CI is approved and running, I’ll watch the lanes and address anything
that pops up. Thanks! 🙏
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
