pass behat args via step env

Author: swissspidy

.github/workflows/testing.yml

@@ -326,21 +326,9 @@ jobs:
           WP_CLI_TEST_DBTYPE: ${{ matrix.dbtype || 'mysql' }}
           WP_CLI_TEST_DBSOCKET: '${{ steps.setup-mysql.outputs.base-dir }}/tmp/mysql.sock'
           WP_CLI_TEST_COVERAGE: ${{ matrix.coverage }}
+          BEHAT_ARGS: ${{ matrix.coverage && env.RUNNER_DEBUG && '--debug --format=pretty' || matrix.coverage && '--debug' || env.RUNNER_DEBUG && '--format=pretty' || '' }}
         run: |
-          ARGS=""
-
-          if [[ $WP_CLI_TEST_COVERAGE == 'true' ]]; then
-            # The flag was only added in v3.17.0
-            if composer behat -- --help 2>/dev/null | grep xdebug; then
-              ARGS+=" --xdebug"
-            fi
-          fi
-
-          if [[ $RUNNER_DEBUG == '1' ]]; then
-            ARGS+=" --format=pretty"
-          fi
-
-          composer behat -- $ARGS || composer behat-rerun -- $ARGS
+          composer behat -- $BEHAT_ARGS || composer behat-rerun -- $BEHAT_ARGS
 
       - name: Retrieve list of coverage files
         id: coverage_files