Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions .github/workflows/ci-amd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -384,6 +384,8 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}

if: needs.build-info.outputs.run-tests == 'true'

tests-postgres-providers:
Expand Down Expand Up @@ -412,6 +414,7 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
if: needs.build-info.outputs.run-tests == 'true'

tests-mysql-core:
Expand Down Expand Up @@ -440,6 +443,7 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
if: needs.build-info.outputs.run-tests == 'true'

tests-mysql-providers:
Expand Down Expand Up @@ -468,6 +472,7 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
if: needs.build-info.outputs.run-tests == 'true'


Expand Down Expand Up @@ -499,6 +504,7 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
if: needs.build-info.outputs.run-tests == 'true'

tests-sqlite-providers:
Expand Down Expand Up @@ -529,6 +535,7 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
if: needs.build-info.outputs.run-tests == 'true'


Expand Down Expand Up @@ -559,6 +566,7 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
if: needs.build-info.outputs.run-tests == 'true'

tests-non-db-providers:
Expand Down Expand Up @@ -588,6 +596,7 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
if: needs.build-info.outputs.run-tests == 'true'

tests-special:
Expand Down Expand Up @@ -673,6 +682,7 @@ jobs:
monitor-delay-time-in-seconds: 120
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}

tests-with-lowest-direct-resolution-providers:
name: "Low dep tests: providers"
Expand Down Expand Up @@ -702,6 +712,7 @@ jobs:
monitor-delay-time-in-seconds: 120
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}

build-prod-images:
name: Build PROD images
Expand Down
6 changes: 6 additions & 0 deletions .github/workflows/ci-arm.yml
Original file line number Diff line number Diff line change
Expand Up @@ -312,6 +312,7 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
if: needs.build-info.outputs.run-tests == 'true'

tests-postgres-providers:
Expand Down Expand Up @@ -340,6 +341,7 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
if: needs.build-info.outputs.run-tests == 'true'

tests-sqlite-core:
Expand Down Expand Up @@ -370,6 +372,7 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
if: needs.build-info.outputs.run-tests == 'true'

tests-sqlite-providers:
Expand Down Expand Up @@ -400,6 +403,7 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
if: needs.build-info.outputs.run-tests == 'true'


Expand Down Expand Up @@ -430,6 +434,7 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
if: needs.build-info.outputs.run-tests == 'true'

tests-non-db-providers:
Expand Down Expand Up @@ -459,6 +464,7 @@ jobs:
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests }}
use-uv: ${{ needs.build-info.outputs.use-uv }}
default-branch: ${{ needs.build-info.outputs.default-branch }}
if: needs.build-info.outputs.run-tests == 'true'

build-prod-images:
Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/run-unit-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,10 @@ on: # yamllint disable-line rule:truthy
description: "Whether to use uv"
required: true
type: string
default-branch:
description: "The default branch of the repository"
required: true
type: string
permissions:
contents: read
jobs:
Expand Down Expand Up @@ -164,6 +168,7 @@ jobs:
UPGRADE_BOTO: "${{ inputs.upgrade-boto }}"
AIRFLOW_MONITOR_DELAY_TIME_IN_SECONDS: "${{inputs.monitor-delay-time-in-seconds}}"
VERBOSE: "true"
DEFAULT_BRANCH: "${{ inputs.default-branch }}"
if: inputs.test-group == 'core' || inputs.skip-providers-tests != 'true'
steps:
- name: "Cleanup repo"
Expand Down
10 changes: 9 additions & 1 deletion .github/workflows/special-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -110,6 +110,7 @@ jobs:
debug-resources: ${{ inputs.debug-resources }}
skip-providers-tests: ${{ inputs.skip-providers-tests }}
use-uv: ${{ inputs.use-uv }}
default-branch: ${{ inputs.default-branch }}

tests-min-sqlalchemy-providers:
name: "Min SQLAlchemy test: providers"
Expand All @@ -134,6 +135,7 @@ jobs:
debug-resources: ${{ inputs.debug-resources }}
skip-providers-tests: ${{ inputs.skip-providers-tests }}
use-uv: ${{ inputs.use-uv }}
default-branch: ${{ inputs.default-branch }}

tests-boto-core:
name: "Latest Boto test: core"
Expand All @@ -159,6 +161,7 @@ jobs:
debug-resources: ${{ inputs.debug-resources }}
skip-providers-tests: ${{ inputs.skip-providers-tests }}
use-uv: ${{ inputs.use-uv }}
default-branch: ${{ inputs.default-branch }}

tests-boto-providers:
name: "Latest Boto test: providers"
Expand All @@ -184,7 +187,7 @@ jobs:
debug-resources: ${{ inputs.debug-resources }}
skip-providers-tests: ${{ inputs.skip-providers-tests }}
use-uv: ${{ inputs.use-uv }}

default-branch: ${{ inputs.default-branch }}

tests-pendulum-2-core:
name: "Pendulum2 test: core"
Expand All @@ -210,6 +213,7 @@ jobs:
debug-resources: ${{ inputs.debug-resources }}
skip-providers-tests: ${{ inputs.skip-providers-tests }}
use-uv: ${{ inputs.use-uv }}
default-branch: ${{ inputs.default-branch }}

tests-pendulum-2-providers:
name: "Pendulum2 test: providers"
Expand All @@ -235,6 +239,7 @@ jobs:
debug-resources: ${{ inputs.debug-resources }}
skip-providers-tests: ${{ inputs.skip-providers-tests }}
use-uv: ${{ inputs.use-uv }}
default-branch: ${{ inputs.default-branch }}

tests-quarantined-core:
name: "Quarantined test: core"
Expand All @@ -259,6 +264,7 @@ jobs:
debug-resources: ${{ inputs.debug-resources }}
skip-providers-tests: ${{ inputs.skip-providers-tests }}
use-uv: ${{ inputs.use-uv }}
default-branch: ${{ inputs.default-branch }}

tests-quarantined-providers:
name: "Quarantined test: providers"
Expand All @@ -283,6 +289,7 @@ jobs:
debug-resources: ${{ inputs.debug-resources }}
skip-providers-tests: ${{ inputs.skip-providers-tests }}
use-uv: ${{ inputs.use-uv }}
default-branch: ${{ inputs.default-branch }}

tests-system-core:
name: "System test: ${{ matrix.test-group }}"
Expand All @@ -307,3 +314,4 @@ jobs:
debug-resources: ${{ inputs.debug-resources }}
skip-providers-tests: ${{ inputs.skip-providers-tests }}
use-uv: ${{ inputs.use-uv }}
default-branch: ${{ inputs.default-branch }}
3 changes: 2 additions & 1 deletion airflow-core/tests/unit/always/test_providers_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@
ProvidersManager,
)

from tests_common.test_utils.markers import skip_if_force_lowest_dependencies_marker
from tests_common.test_utils.markers import skip_if_force_lowest_dependencies_marker, skip_if_not_on_main
from tests_common.test_utils.paths import AIRFLOW_ROOT_PATH


Expand Down Expand Up @@ -230,6 +230,7 @@ def test_hooks(self):
raise AssertionError("There are warnings generated during hook imports. Please fix them")
assert [w.message for w in warning_records if "hook-class-names" in str(w.message)] == []

@skip_if_not_on_main
@pytest.mark.execution_timeout(150)
def test_hook_values(self):
provider_dependencies = json.loads(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@
from airflow.utils.operator_resources import Resources

from tests_common.test_utils.config import conf_vars
from tests_common.test_utils.markers import skip_if_force_lowest_dependencies_marker
from tests_common.test_utils.markers import skip_if_force_lowest_dependencies_marker, skip_if_not_on_main
from tests_common.test_utils.mock_operators import (
AirflowLink,
AirflowLink2,
Expand Down Expand Up @@ -477,12 +477,14 @@ def setup_test_cases(self):

# Skip that test if latest botocore is used - it reads all example dags and in case latest botocore
# is upgraded to latest, usually aiobotocore can't be installed and some of the system tests will fail with
# import errors.
# import errors. Also skip if not running on main branch - some of the example dags might fail due to
# outdated imports in past branches
@pytest.mark.skipif(
os.environ.get("UPGRADE_BOTO", "") == "true",
reason="This test is skipped when latest botocore is installed",
)
@skip_if_force_lowest_dependencies_marker
@skip_if_not_on_main
@pytest.mark.db_test
def test_serialization(self):
"""Serialization and deserialization should work for every DAG and Operator."""
Expand Down
6 changes: 6 additions & 0 deletions devel-common/src/tests_common/test_utils/markers.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,3 +25,9 @@
os.environ.get("FORCE_LOWEST_DEPENDENCIES", "") == "true",
reason="When lowest dependencies are set only some providers are loaded",
)


skip_if_not_on_main = pytest.mark.skipif(
os.environ.get("DEFAULT_BRANCH", "main") != "main",
reason="This test is only run on main branch in CI",
)