Skip to content

Commit

Permalink
remove un used python_version param argument (#43609)
Browse files Browse the repository at this point in the history
  • Loading branch information
gopidesupavan authored Nov 2, 2024
1 parent 59ea748 commit a8921ae
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 8 deletions.
8 changes: 2 additions & 6 deletions dev/breeze/src/airflow_breeze/utils/run_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -309,9 +309,7 @@ def generate_args_for_pytest(
):
result_log_file, warnings_file, coverage_file = test_paths(test_type, backend, helm_test_package)
if skip_db_tests and parallel_test_types_list:
args = convert_parallel_types_to_folders(
parallel_test_types_list, skip_provider_tests, python_version=python_version
)
args = convert_parallel_types_to_folders(parallel_test_types_list, skip_provider_tests)
else:
args = convert_test_type_to_pytest_args(
test_type=test_type,
Expand Down Expand Up @@ -394,9 +392,7 @@ def generate_args_for_pytest(
return args


def convert_parallel_types_to_folders(
parallel_test_types_list: list[str], skip_provider_tests: bool, python_version: str
):
def convert_parallel_types_to_folders(parallel_test_types_list: list[str], skip_provider_tests: bool):
args = []
for _test_type in parallel_test_types_list:
args.extend(
Expand Down
2 changes: 0 additions & 2 deletions dev/breeze/tests/test_pytest_args_for_test_types.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@

import pytest

from airflow_breeze.global_constants import DEFAULT_PYTHON_MAJOR_MINOR_VERSION
from airflow_breeze.utils.run_tests import convert_parallel_types_to_folders, convert_test_type_to_pytest_args


Expand Down Expand Up @@ -347,7 +346,6 @@ def test_folders_for_parallel_test_types(
convert_parallel_types_to_folders(
parallel_test_types_list=parallel_test_types.split(" "),
skip_provider_tests=skip_provider_tests,
python_version=DEFAULT_PYTHON_MAJOR_MINOR_VERSION,
)
== folders
)

0 comments on commit a8921ae

Please sign in to comment.