On 28/06/2022 19.04, Volkmar W. Pogatzki wrote:
Bug: https://bugs.gentoo.org/801694
Signed-off-by: Volkmar W. Pogatzki <[email protected]>
---
  eclass/java-utils-2.eclass | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/eclass/java-utils-2.eclass b/eclass/java-utils-2.eclass
index e5a2b611d11..cfe04d631a0 100644
--- a/eclass/java-utils-2.eclass
+++ b/eclass/java-utils-2.eclass
@@ -1901,7 +1901,9 @@ etestng() {
                ${JAVA_TEST_RUNNER_EXTRA_ARGS[@]}
        )
- [[ ! "${JAVA_TEST_RUNNER_EXTRA_ARGS[@]}" =~ "-usedefaultlisteners" ]] && args+=( -usedefaultlisteners false )
+       if [[ ! "${JAVA_TEST_RUNNER_EXTRA_ARGS[@]}" =~ "-usedefaultlisteners" 
]]; then
+               args+=( -verbose 3 -usedefaultlisteners true )
+       fi
args+=( -testclass ${tests} )

LGTM, but I would split the args value in multiple lines, which makes it clear what the argument name and the argument value is and hence makes it easier to read:

args+=(
        -verbose 3
        -usedefaultlisteners true
)

- Flow

Reply via email to