osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/40588?usp=email )

Change subject: Cosmetic: testenv: tweak argparse formatting ......................................................................

Cosmetic: testenv: tweak argparse formatting

Make all "add_argument" calls consistently have one function argument in one line, instead of only having it in some places, and have all arguments in one line in others. Having one function arg in one line is enforced via ruff format when having a comma at the end of the function call.

Change-Id: Ia1804f0313a27ddf8f7197ce0b2ad3b56421b290 --- M _testenv/testenv/__init__.py 1 file changed, 31 insertions(+), 6 deletions(-)

Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve pespin: Looks good to me, approved

diff --git a/_testenv/testenv/__init__.py b/_testenv/testenv/__init__.py index 511ec58..f2f0321 100644 --- a/_testenv/testenv/__init__.py +++ b/_testenv/testenv/__init__.py @@ -73,7 +73,10 @@ sub_run = sub.add_parser("run", help="build components and run a testsuite")

group = sub_run.add_argument_group("testsuite options") - group.add_argument("testsuite", help="a directory in osmo-ttcn3-hacks.git (msc, bsc, mgw, ...)") + group.add_argument( + "testsuite", + help="a directory in osmo-ttcn3-hacks.git (msc, bsc, mgw, ...)", + ) group.add_argument( "-t", "--test", @@ -85,8 +88,17 @@ action="append", help="which testenv.cfg to use (supports * wildcards via fnmatch)", ) - group.add_argument("-i", "--io-uring", action="store_true", help="set LIBOSMO_IO_BACKEND=IO_URING") - group.add_argument("-T", "--titan-version", help="which eclipse-titan version to use") + group.add_argument( + "-i", + "--io-uring", + action="store_true", + help="set LIBOSMO_IO_BACKEND=IO_URING", + ) + group.add_argument( + "-T", + "--titan-version", + help="which eclipse-titan version to use", + )

group = sub_run.add_argument_group("source/binary options", "All components are built from source by default.") group.add_argument( @@ -147,10 +159,19 @@ " repository ending in :latest is set" " or --latest is used.", ) - group.add_argument("--latest", action="store_true", help="use latest configs") + group.add_argument( + "--latest", + action="store_true", + help="use latest configs", + )

group = sub_run.add_argument_group("podman options", "All components are run directly on the host by default.") - group.add_argument("-p", "--podman", action="store_true", help="run all components inside podman") + group.add_argument( + "-p", + "--podman", + action="store_true", + help="run all components inside podman", + ) group.add_argument( "-d", "--distro", @@ -165,7 +186,11 @@ )

group = sub_run.add_argument_group("output options") - group.add_argument("-l", "--log-dir", help="log here instead of a random dir in /tmp") + group.add_argument( + "-l", + "--log-dir", + help="log here instead of a random dir in /tmp", + ) group.add_argument( "-n", "--no-tee",