Skip to content

Update path for breeze testing task-sdk-integration-tests ...#58265

Closed
jroachgolf84 wants to merge 1 commit intoapache:mainfrom
jroachgolf84:patch-4
Closed

Update path for breeze testing task-sdk-integration-tests ...#58265
jroachgolf84 wants to merge 1 commit intoapache:mainfrom
jroachgolf84:patch-4

Conversation

@jroachgolf84
Copy link
Copy Markdown
Collaborator

Description

Updating the docs to properly reflect the path that should be passed when running specific Task SDK integration tests.

Per the previous docs, a command like (breeze testing task-sdk-integration-tests task_sdk_tests/test_variable_operations.py) this should execute tests in the test_variable_operations.py file. However, running this command resulted in the exception: ERROR: file or directory not found: task_sdk_tests/test_variable_operations.py.

Adding tests/ in front of task_sdk_tests/... solved this issue. The updated command looks something like this:

breeze testing task-sdk-integration-tests tests/task_sdk_tests/test_variable_operations.py

The appropriate changes in the documentation were made to reflect this.

@jroachgolf84
Copy link
Copy Markdown
Collaborator Author

jroachgolf84 commented Nov 13, 2025

I may need to wait and evaluate this a bit further. It seems that event specifying a certain fail (like I did in the command above) still runs all the integration tests in the task-sdk-integration-tests/tests/task_sdk_tests directory. Here's a snippet of the proof. In the output below, Asset Event tests are being executed, even though they shouldn't.

ERROR tests/task_sdk_tests/test_asset_event_operations.py::test_asset_event_get - python_on_whales.exceptions.DockerException: The command executed was `/usr/local/bin/docker compose --file /private/var/folders/lk/16q03xkd4fbc40xg6g738rxr0000gn/T/pytest-of-jroachgolf84/pytest-12/airflow-task-sdk-test0/docker-compose.yaml up --detach --wait`.
It returned with code 1
The content of stdout can be found above the stacktrace (it wasn't captured).
The content of stderr can be found above the stacktrace (it wasn't captured).
ERROR tests/task_sdk_tests/test_asset_event_operations.py::test_asset_event_get_not_found - python_on_whales.exceptions.DockerException: The command executed was `/usr/local/bin/docker compose --file /private/var/folders/lk/16q03xkd4fbc40xg6g738rxr0000gn/T/pytest-of-jroachgolf84/pytest-12/airflow-task-sdk-test0/docker-compose.yaml up --detach --wait`.

cc: @potiuk

@potiuk
Copy link
Copy Markdown
Member

potiuk commented Nov 13, 2025

Both issues are already fixed in #58231

@potiuk potiuk closed this Nov 13, 2025
@potiuk
Copy link
Copy Markdown
Member

potiuk commented Nov 13, 2025

Particularly the second one (which I noticed yesterday as well) was fixed by:

    all_tests = [test_path.as_posix()]
    # Always with color and -s to see print outputs as they come
    pytest_args = ["--color=yes", "-s"]
    if not any(pytest_arg.startswith("tests/") for pytest_arg in extra_pytest_args):
        # Only add all tests when no tests were specified on the command line
        pytest_args.extend(all_tests)
    else:
        pytest_args.extend(extra_pytest_args)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants