`execute_async` API is unclear

[danth]

Description

The parameters to execute_async are binary_path and args, implying that it works like subprocess.run and executes the command directly.

However some integration tests pass a full Bash script as the binary_path, for example:

itf/test/integration/test_async_exec.py

Line 156 in 53a7ba3

proc = target.execute_async("echo line1; echo line2; echo line3")

The implementation is based around this.

Analysis results

No response

Solution

Either change the implementation to run the process directly, or change the API to be more clear.

Error Occurrence Rate

Reproducible

How to reproduce

No response

Supporting Information

No response

Classification

Minor

First Affected Release

not released (main)

Last Affected Release

not released (main)

Expected Fixed Release

before release (main)

Category

• Safety Relevant
• Security Relevant