You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by ep...@apache.org on 2023/03/08 14:44:29 UTC

[airflow] 14/20: Use "all" integrations to run tests when "all" is used (#29517)

This is an automated email from the ASF dual-hosted git repository.

ephraimanierobi pushed a commit to branch v2-5-test
in repository https://gitbox.apache.org/repos/asf/airflow.git

commit cf89b09a62e173f0fbdee13d4f6bea3a65ab7401
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Tue Feb 14 00:10:12 2023 +0100

    Use "all" integrations to run tests when "all" is used (#29517)
    
    The change #29449 made a distinction between "all" and "available"
    tests and it turned out that "All" test type uses "available" tests.
    The name "all" is actually wrong now. It should be renamed to be
    "all-with-tests" or similar, but this should be done in a separate PR
    
    (cherry picked from commit 72c3817a44eea5005761ae3b621e8c39fde136ad)
---
 dev/breeze/src/airflow_breeze/params/shell_params.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dev/breeze/src/airflow_breeze/params/shell_params.py b/dev/breeze/src/airflow_breeze/params/shell_params.py
index fad96e81f5..fb7ebe0c30 100644
--- a/dev/breeze/src/airflow_breeze/params/shell_params.py
+++ b/dev/breeze/src/airflow_breeze/params/shell_params.py
@@ -23,6 +23,7 @@ from pathlib import Path
 
 from airflow_breeze.branch_defaults import AIRFLOW_BRANCH, DEFAULT_AIRFLOW_CONSTRAINTS_BRANCH
 from airflow_breeze.global_constants import (
+    ALL_INTEGRATIONS,
     ALLOWED_BACKENDS,
     ALLOWED_CONSTRAINTS_MODES_CI,
     ALLOWED_INSTALLATION_PACKAGE_FORMATS,
@@ -31,7 +32,6 @@ from airflow_breeze.global_constants import (
     ALLOWED_POSTGRES_VERSIONS,
     ALLOWED_PYTHON_MAJOR_MINOR_VERSIONS,
     APACHE_AIRFLOW_GITHUB_REPOSITORY,
-    AVAILABLE_INTEGRATIONS,
     DOCKER_DEFAULT_PLATFORM,
     MOUNT_ALL,
     MOUNT_REMOVE,
@@ -238,7 +238,7 @@ class ShellParams:
         if self.include_mypy_volume:
             compose_file_list.append(DOCKER_COMPOSE_DIR / "mypy.yml")
         if "all" in self.integration:
-            integrations = AVAILABLE_INTEGRATIONS
+            integrations = ALL_INTEGRATIONS
         else:
             integrations = self.integration
         if len(integrations) > 0: