Skip to content

Guard provider-dependent tests in airflow-core for standalone core testing#63572

Closed
ShubhamGondane wants to merge 4 commits into
apache:mainfrom
ShubhamGondane:fix/60770-guard-provider-imports-in-core-tests
Closed

Guard provider-dependent tests in airflow-core for standalone core testing#63572
ShubhamGondane wants to merge 4 commits into
apache:mainfrom
ShubhamGondane:fix/60770-guard-provider-imports-in-core-tests

Conversation

@ShubhamGondane

Copy link
Copy Markdown
Contributor

Adds pytest.importorskip guards to core tests that import optional provider packages so they skip gracefully when providers are not installed. Removes 6 provider packages from airflow-core dev dependencies. This enables cd airflow-core && uv sync && pytest to work standalone. The pytest.importorskip pattern is already established in the codebase (test_executor_loader.py, test_connection.py). These tests continue to run in CI and in full workspace environments (uv sync --all-packages).

The remaining provider-dependent tests all test core functionality (executor loading, serialization, CLI parsing) using provider classes as test inputs. Previous PRs (#48113, #49178, #54635) already moved truly provider-specific tests. These belong in core and are guarded rather than moved.

closes: #60770


Was generative AI tooling used to co-author this PR?
  • Yes - Claude Code (claude-opus-4-6)

Generated-by: Claude Code (claude-opus-4-6) following the guidelines

@ShubhamGondane
ShubhamGondane marked this pull request as draft March 14, 2026 00:44
@ShubhamGondane
ShubhamGondane force-pushed the fix/60770-guard-provider-imports-in-core-tests branch from 19f679d to 471806a Compare March 17, 2026 00:16
@potiuk

potiuk commented Apr 22, 2026

Copy link
Copy Markdown
Member

@ShubhamGondane This draft PR has had no activity for 5 weeks. Closing to keep the queue clean.

You are welcome to reopen and continue when you're ready. If you'd like to pick it back up, please rebase onto the current main branch first.


Note: This comment was drafted by an AI-assisted triage tool and may contain mistakes. Once you have addressed the points above, an Apache Airflow maintainer — a real person — will take the next look at your PR. We use this two-stage triage process so that our maintainers' limited time is spent where it matters most: the conversation with you.

@potiuk potiuk closed this Apr 22, 2026
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.

Cleanup: Move remaining provider tests from airflow-core to support standalone uv sync

2 participants