This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch v2-8-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/v2-8-test by this push:
new 655f3326ad Fix failing static/unit tests.
655f3326ad is described below
commit 655f3326adf5fc32164f96fd45bc2e28866a1fe8
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Thu Feb 8 08:41:25 2024 +0100
Fix failing static/unit tests.
---
dev/breeze/src/airflow_breeze/utils/selective_checks.py | 2 ++
dev/breeze/tests/test_packages.py | 4 ++--
dev/breeze/tests/test_selective_checks.py | 12 ++----------
3 files changed, 6 insertions(+), 12 deletions(-)
diff --git a/dev/breeze/src/airflow_breeze/utils/selective_checks.py
b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
index dfc1d47e61..394774a4b1 100644
--- a/dev/breeze/src/airflow_breeze/utils/selective_checks.py
+++ b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
@@ -14,6 +14,7 @@
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.
+
from __future__ import annotations
import json
@@ -874,6 +875,7 @@ class SelectiveChecks:
"check-airflow-provider-compatibility",
"check-extra-packages-references",
"check-provider-yaml-valid",
+ "validate-operators-init",
"lint-helm-chart",
)
)
diff --git a/dev/breeze/tests/test_packages.py
b/dev/breeze/tests/test_packages.py
index d0e214156f..31b36a6113 100644
--- a/dev/breeze/tests/test_packages.py
+++ b/dev/breeze/tests/test_packages.py
@@ -114,12 +114,12 @@ def test_get_removed_providers():
def test_get_suspended_provider_ids():
# Modify it every time we suspend/resume provider
- assert ["daskexecutor", "plexus"] == get_suspended_provider_ids()
+ assert ["apache.sqoop", "daskexecutor", "plexus"] ==
get_suspended_provider_ids()
def test_get_suspended_provider_folders():
# Modify it every time we suspend/resume provider
- assert ["daskexecutor", "plexus"] == get_suspended_provider_folders()
+ assert ["apache/sqoop", "daskexecutor", "plexus"] ==
get_suspended_provider_folders()
@pytest.mark.parametrize(
diff --git a/dev/breeze/tests/test_selective_checks.py
b/dev/breeze/tests/test_selective_checks.py
index e6532aa078..7fc620c1f7 100644
--- a/dev/breeze/tests/test_selective_checks.py
+++ b/dev/breeze/tests/test_selective_checks.py
@@ -698,7 +698,7 @@ def test_expected_output_pull_request_main(
"docs-build": "true",
"docs-list-as-string": "apache-airflow docker-stack",
"full-tests-needed": "true",
- "skip-pre-commits":
"check-airflow-provider-compatibility,check-extra-packages-references,check-provider-yaml-valid,identity,lint-helm-chart,mypy-core,mypy-dev,mypy-docs,mypy-providers",
+ "skip-pre-commits":
"check-airflow-provider-compatibility,check-extra-packages-references,check-provider-yaml-valid,identity,lint-helm-chart,mypy-core,mypy-dev,mypy-docs,mypy-providers,validate-operators-init",
"skip-provider-tests": "true",
"upgrade-to-newer-dependencies": "false",
"parallel-test-types-list-as-string": "API Always
BranchExternalPython "
@@ -744,9 +744,6 @@ def test_expected_output_full_tests_needed(
"docs-build": "false",
"docs-list-as-string": None,
"full-tests-needed": "false",
- "skip-pre-commits":
"check-airflow-provider-compatibility,check-extra-packages-references,"
- "check-provider-yaml-valid,flynt,identity,lint-helm-chart,"
-
"mypy-core,mypy-dev,mypy-docs,mypy-providers,ts-compile-format-lint-www",
"upgrade-to-newer-dependencies": "false",
"skip-provider-tests": "true",
"parallel-test-types-list-as-string": None,
@@ -773,9 +770,6 @@ def test_expected_output_full_tests_needed(
"docs-build": "true",
"docs-list-as-string": "apache-airflow docker-stack",
"full-tests-needed": "false",
- "skip-pre-commits":
"check-airflow-provider-compatibility,check-extra-packages-references,"
- "check-provider-yaml-valid,identity,lint-helm-chart,"
-
"mypy-core,mypy-dev,mypy-docs,mypy-providers,ts-compile-format-lint-www",
"run-kubernetes-tests": "true",
"upgrade-to-newer-dependencies": "false",
"skip-provider-tests": "true",
@@ -806,7 +800,6 @@ def test_expected_output_full_tests_needed(
"docs-build": "true",
"docs-list-as-string": "apache-airflow docker-stack",
"full-tests-needed": "false",
- "skip-pre-commits":
"check-airflow-provider-compatibility,check-extra-packages-references,check-provider-yaml-valid,identity,lint-helm-chart,mypy-core,mypy-dev,mypy-docs,mypy-providers,ts-compile-format-lint-www",
"run-kubernetes-tests": "true",
"upgrade-to-newer-dependencies": "false",
"skip-provider-tests": "true",
@@ -835,7 +828,6 @@ def test_expected_output_full_tests_needed(
"run-kubernetes-tests": "false",
"upgrade-to-newer-dependencies": "false",
"skip-provider-tests": "true",
- "skip-pre-commits":
"check-airflow-provider-compatibility,check-extra-packages-references,check-provider-yaml-valid,identity,lint-helm-chart,mypy-core,mypy-dev,mypy-docs,mypy-providers,ts-compile-format-lint-www",
"parallel-test-types-list-as-string": "API Always
BranchExternalPython BranchPythonVenv "
"CLI Core ExternalPython Operators Other PlainAsserts
PythonVenv Serialization WWW",
"needs-mypy": "true",
@@ -1066,7 +1058,7 @@ def test_expected_output_pull_request_target(
"needs-helm-tests": "false",
"run-tests": "true",
"docs-build": "true",
- "skip-pre-commits":
"check-airflow-provider-compatibility,check-extra-packages-references,check-provider-yaml-valid,identity,lint-helm-chart,mypy-core,mypy-dev,mypy-docs,mypy-providers",
+ "skip-pre-commits":
"check-airflow-provider-compatibility,check-extra-packages-references,check-provider-yaml-valid,identity,lint-helm-chart,mypy-core,mypy-dev,mypy-docs,mypy-providers,validate-operators-init",
"docs-list-as-string": "apache-airflow docker-stack",
"upgrade-to-newer-dependencies": "true",
"parallel-test-types-list-as-string": "API Always
BranchExternalPython BranchPythonVenv "
