Skip to content

Conversation

@jroachgolf84
Copy link
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.

@boring-cyborg boring-cyborg bot added area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch labels Nov 13, 2025
@jroachgolf84
Copy link
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
Member

potiuk commented Nov 13, 2025

Both issues are already fixed in #58231

@potiuk potiuk closed this Nov 13, 2025
@potiuk
Copy link
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

Labels

area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants