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
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
if AIRFLOW_V_3_0_PLUS:
from airflow.sdk import BaseOperator
else:
from airflow.models.baseoperator import BaseOperator
from airflow.models.baseoperator import BaseOperator # type: ignore[no-redef]

from airflow.models.taskinstance import TaskInstanceState
from airflow.providers.openlineage.extractors.base import (
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
if AIRFLOW_V_3_0_PLUS:
from airflow.sdk import BaseOperator
else:
from airflow.models.baseoperator import BaseOperator
from airflow.models.baseoperator import BaseOperator # type: ignore[no-redef]

from airflow.providers.openlineage.extractors.base import OperatorLineage
from airflow.providers.openlineage.plugins.adapter import OpenLineageAdapter
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,16 +26,6 @@
from uuid6 import uuid7

from airflow import DAG
from airflow.utils import timezone

from tests_common.test_utils.version_compat import AIRFLOW_V_3_0_PLUS

if AIRFLOW_V_3_0_PLUS:
from airflow.sdk import BaseOperator, task
else:
from airflow.decorators import task
from airflow.models.baseoperator import BaseOperator

from airflow.models.dagrun import DagRun
from airflow.models.taskinstance import TaskInstance, TaskInstanceState
from airflow.providers.common.compat.assets import Asset
Expand All @@ -60,6 +50,7 @@
from airflow.serialization.serialized_objects import SerializedBaseOperator
from airflow.timetables.events import EventsTimetable
from airflow.timetables.trigger import CronTriggerTimetable
from airflow.utils import timezone
from airflow.utils.state import DagRunState
from airflow.utils.task_group import TaskGroup
from airflow.utils.types import DagRunType
Expand All @@ -68,6 +59,12 @@
from tests_common.test_utils.mock_operators import MockOperator
from tests_common.test_utils.version_compat import AIRFLOW_V_3_0_PLUS, AIRFLOW_V_3_1_PLUS

if AIRFLOW_V_3_0_PLUS:
from airflow.sdk import BaseOperator, task
else:
from airflow.decorators import task # type: ignore[no-redef]
from airflow.models.baseoperator import BaseOperator # type: ignore[no-redef]

BASH_OPERATOR_PATH = "airflow.providers.standard.operators.bash"
PYTHON_OPERATOR_PATH = "airflow.providers.standard.operators.python"

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
if AIRFLOW_V_3_1_PLUS:
from airflow.sdk import BaseOperator
else:
from airflow.models.baseoperator import BaseOperator
from airflow.models.baseoperator import BaseOperator # type: ignore[no-redef]

if AIRFLOW_V_3_0_PLUS:
from airflow.sdk.execution_time.context import context_to_airflow_vars
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,12 +27,12 @@
if AIRFLOW_V_3_1_PLUS:
from airflow.sdk import BaseOperator
else:
from airflow.models.baseoperator import BaseOperator
from airflow.models.baseoperator import BaseOperator # type: ignore[no-redef]

if AIRFLOW_V_3_0_PLUS:
from airflow.providers.standard.utils.skipmixin import SkipMixin
else:
from airflow.models.skipmixin import SkipMixin
from airflow.models.skipmixin import SkipMixin # type: ignore[no-redef]

if TYPE_CHECKING:
from airflow.sdk.definitions.context import Context
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
if AIRFLOW_V_3_1_PLUS:
from airflow.sdk import BaseOperator
else:
from airflow.models.baseoperator import BaseOperator
from airflow.models.baseoperator import BaseOperator # type: ignore[no-redef]

if TYPE_CHECKING:
from airflow.sdk.definitions.context import Context
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,13 +60,13 @@
if AIRFLOW_V_3_1_PLUS:
from airflow.sdk import BaseOperator
else:
from airflow.models.baseoperator import BaseOperator
from airflow.models.baseoperator import BaseOperator # type: ignore[no-redef]

if AIRFLOW_V_3_0_PLUS:
from airflow.providers.standard.operators.branch import BaseBranchOperator
from airflow.providers.standard.utils.skipmixin import SkipMixin
else:
from airflow.models.skipmixin import SkipMixin
from airflow.models.skipmixin import SkipMixin # type: ignore[no-redef]
from airflow.operators.branch import BaseBranchOperator # type: ignore[no-redef]


Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
if AIRFLOW_V_3_0_PLUS:
from airflow.sdk import BaseOperator
else:
from airflow.models.baseoperator import BaseOperator
from airflow.models.baseoperator import BaseOperator # type: ignore[no-redef]

if TYPE_CHECKING:
from airflow.sdk.definitions.context import Context
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -58,8 +58,8 @@
from airflow.sdk import BaseOperator
from airflow.sdk.definitions.context import Context
else:
from airflow.models.baseoperator import BaseOperator
from airflow.utils.context import Context
from airflow.models.baseoperator import BaseOperator # type: ignore[no-redef]
from airflow.utils.context import Context # type: ignore[no-redef]


if AIRFLOW_V_3_0_PLUS:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,8 @@ def _ensure_tasks(nodes: Iterable[DAGNode]) -> Sequence[Operator]:
from airflow.sdk import BaseOperator
from airflow.sdk.definitions.mappedoperator import MappedOperator
else:
from airflow.models.baseoperator import BaseOperator
from airflow.models.mappedoperator import MappedOperator
from airflow.models.baseoperator import BaseOperator # type: ignore[no-redef]
from airflow.models.mappedoperator import MappedOperator # type: ignore[no-redef]

return [n for n in nodes if isinstance(n, BaseOperator | MappedOperator)]

Expand Down
12 changes: 5 additions & 7 deletions providers/standard/tests/unit/standard/operators/test_python.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,13 +47,6 @@
AirflowProviderDeprecationWarning,
DeserializingResultError,
)

from tests_common.test_utils.version_compat import AIRFLOW_V_3_0_PLUS

if AIRFLOW_V_3_0_PLUS:
from airflow.sdk import BaseOperator
else:
from airflow.models.baseoperator import BaseOperator
from airflow.models.taskinstance import TaskInstance, clear_task_instances, set_current_context
from airflow.providers.standard.operators.empty import EmptyOperator
from airflow.providers.standard.operators.python import (
Expand All @@ -78,6 +71,11 @@
from tests_common.test_utils.db import clear_db_runs
from tests_common.test_utils.version_compat import AIRFLOW_V_3_0_1, AIRFLOW_V_3_0_PLUS

if AIRFLOW_V_3_0_PLUS:
from airflow.sdk import BaseOperator
else:
from airflow.models.baseoperator import BaseOperator # type: ignore[no-redef]

if TYPE_CHECKING:
from airflow.models.dag import DAG
from airflow.models.dagrun import DagRun
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
if AIRFLOW_V_3_0_PLUS:
from airflow.sdk import BaseOperator
else:
from airflow.models.baseoperator import BaseOperator
from airflow.models.baseoperator import BaseOperator # type: ignore[no-redef]
from airflow.models.dag import DAG
from airflow.models.serialized_dag import SerializedDagModel
from airflow.models.xcom_arg import XComArg
Expand Down