bobu-putheeckal opened a new pull request, #67415: URL: https://github.com/apache/airflow/pull/67415
Adds a focused regression test for the public pools list endpoint when a pool has unlimited slots (`slots == -1`). Current `main` already sanitizes `Pool.open_slots()` from `inf` to `-1`; this test covers the issue-reported `GET /pools` path so the behavior stays guarded. Fixes #65377 Tests run: - `uv run --no-dev --with ruff ruff check airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_pools.py` - `AIRFLOW_SOURCES="$PWD" uv run --no-dev --with pytest --with pytest-mock --with time-machine --with 'apache-airflow-core @ ./airflow-core' --with 'apache-airflow-task-sdk @ ./task-sdk' --with 'apache-airflow-devel-common[no-doc] @ ./devel-common' --with 'apache-airflow-providers-git @ ./providers/git' --with 'apache-airflow-shared-secrets-masker @ ./shared/secrets_masker' pytest airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_pools.py::TestGetPools`\n\nBlocked checks:\n- Plain `uv run pytest airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_pools.py::TestGetPools::test_should_respond_200_with_unlimited_slots` could not build `jpype1` because this machine has no Java runtime; reran the focused tests with a trimmed dependency set above. -- 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]
