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 "