Skip to content

Commit

Permalink
Fix CI's docs filter generation (apache#35087)
Browse files Browse the repository at this point in the history
* Fix CI's docs filter generation

* Fix test cases
  • Loading branch information
Taragolis authored Oct 20, 2023
1 parent f816237 commit fdc443d
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 11 deletions.
2 changes: 1 addition & 1 deletion dev/breeze/src/airflow_breeze/utils/selective_checks.py
Original file line number Diff line number Diff line change
Expand Up @@ -726,7 +726,7 @@ def docs_list_as_string(self) -> str | None:
packages.append("docker-stack")
if providers_affected:
for provider in providers_affected:
packages.append(f"{provider.replace('.', '-')}")
packages.append(provider.replace("-", "."))
return " ".join(packages)

@cached_property
Expand Down
20 changes: 10 additions & 10 deletions dev/breeze/tests/test_selective_checks.py
Original file line number Diff line number Diff line change
Expand Up @@ -693,9 +693,9 @@ def test_expected_output_pull_request_v2_3(
"needs-helm-tests": "true",
"run-tests": "true",
"docs-build": "true",
"docs-list-as-string": "apache-airflow helm-chart amazon apache-beam apache-cassandra "
"cncf-kubernetes common-sql facebook google hashicorp microsoft-azure "
"microsoft-mssql mysql openlineage oracle postgres "
"docs-list-as-string": "apache-airflow helm-chart amazon apache.beam apache.cassandra "
"cncf.kubernetes common.sql facebook google hashicorp microsoft.azure "
"microsoft.mssql mysql openlineage oracle postgres "
"presto salesforce samba sftp ssh trino",
"run-kubernetes-tests": "true",
"upgrade-to-newer-dependencies": "false",
Expand Down Expand Up @@ -975,19 +975,19 @@ def test_upgrade_to_newer_dependencies(
pytest.param(
("docs/apache-airflow-providers-google/docs.rst",),
{
"docs-list-as-string": "amazon apache-beam apache-cassandra "
"cncf-kubernetes common-sql facebook google hashicorp "
"microsoft-azure microsoft-mssql mysql openlineage oracle "
"docs-list-as-string": "amazon apache.beam apache.cassandra "
"cncf.kubernetes common.sql facebook google hashicorp "
"microsoft.azure microsoft.mssql mysql openlineage oracle "
"postgres presto salesforce samba sftp ssh trino",
},
id="Google provider docs changed",
),
pytest.param(
("airflow/providers/common/sql/common_sql_python.py",),
{
"docs-list-as-string": "apache-airflow amazon apache-drill apache-druid apache-hive "
"apache-impala apache-pinot common-sql databricks elasticsearch "
"exasol google jdbc microsoft-mssql mysql odbc openlineage "
"docs-list-as-string": "apache-airflow amazon apache.drill apache.druid apache.hive "
"apache.impala apache.pinot common.sql databricks elasticsearch "
"exasol google jdbc microsoft.mssql mysql odbc openlineage "
"oracle postgres presto slack snowflake sqlite trino vertica",
},
id="Common SQL provider package python files changed",
Expand Down Expand Up @@ -1026,7 +1026,7 @@ def test_upgrade_to_newer_dependencies(
),
pytest.param(
("airflow/providers/celery/file.py",),
{"docs-list-as-string": "apache-airflow celery cncf-kubernetes"},
{"docs-list-as-string": "apache-airflow celery cncf.kubernetes"},
id="Celery python files changed",
),
pytest.param(
Expand Down

0 comments on commit fdc443d

Please sign in to comment.