Skip to content

Commit

Permalink
In case a provider is not ready, also run lowest dependency tests (#4…
Browse files Browse the repository at this point in the history
  • Loading branch information
jscheffl authored Sep 18, 2024
1 parent ac9a9f5 commit 2ebd8b5
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
4 changes: 3 additions & 1 deletion dev/breeze/src/airflow_breeze/utils/selective_checks.py
Original file line number Diff line number Diff line change
Expand Up @@ -873,7 +873,9 @@ def separate_test_types_list_as_string(self) -> str | None:
current_test_types = set(self._get_test_types_to_run(split_to_individual_providers=True))
if "Providers" in current_test_types:
current_test_types.remove("Providers")
current_test_types.update({f"Providers[{provider}]" for provider in get_available_packages()})
current_test_types.update(
{f"Providers[{provider}]" for provider in get_available_packages(include_not_ready=True)}
)
if self.skip_provider_tests:
current_test_types = {
test_type for test_type in current_test_types if not test_type.startswith("Providers")
Expand Down
6 changes: 4 additions & 2 deletions dev/breeze/tests/test_selective_checks.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,9 @@

ALL_DOCS_SELECTED_FOR_BUILD = ""
ALL_PROVIDERS_AFFECTED = ""
LIST_OF_ALL_PROVIDER_TESTS = " ".join(f"Providers[{provider}]" for provider in get_available_packages())
LIST_OF_ALL_PROVIDER_TESTS = " ".join(
f"Providers[{provider}]" for provider in get_available_packages(include_not_ready=True)
)


# commit that is neutral - allows to keep pyproject.toml-changing PRS neutral for unit tests
Expand Down Expand Up @@ -1119,7 +1121,7 @@ def test_full_test_needed_when_scripts_changes(files: tuple[str, ...], expected_
"needs-mypy": "true",
"mypy-folders": "['airflow', 'providers', 'docs', 'dev']",
},
id="Everything should run including full providers when"
id="Everything should run including full providers when "
"full tests are needed even if no files are changed",
)
),
Expand Down

0 comments on commit 2ebd8b5

Please sign in to comment.