Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-49531

Parallel behat runner with extraopts or tags broken

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      Run the behat parallel executor with extra options or tags:

      php admin/tool/behat/cli/init.php --parallel=4
      php admin/tool/behat/cli/run.php -f progress --out php://stdout --tags ~@javascript

      ...the behat tests should run and complete successfully, instead of failing to start.

      Show
      Run the behat parallel executor with extra options or tags: php admin/tool/behat/cli/init.php --parallel=4 php admin/tool/behat/cli/run.php -f progress --out php://stdout --tags ~@javascript ...the behat tests should run and complete successfully, instead of failing to start.
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_29_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      behat_parallel_extraopts

      Description

      The parallel behat runner script does not correctly handle the 'extraopts' passing; extra options / tags will not get a space between the additional CLI parameters, resulting in incorrect commands.

      Example:
      php admin/tool/behat/cli/run.php -v --tags '~@javascript' -f junit --out ../behat_results.xml -f progress --out php://stdout
      Results in:
      vendor/bin/behat -config /dev/shm/moodledata/behat1/behat/behat.yml -v '@javascript' -f 'junit' --out '../behat_results.xml' -f 'progress' --out 'php://stdout'-tags="'@javascript'"

      I will provide a small patch for this.

        Attachments

          Activity

            People

            • Assignee:
              tlevi Tony Levi
              Reporter:
              tlevi Tony Levi
              Peer reviewer:
              Rajesh Taneja
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Eloy Lafuente (stronk7)
              Participants:
              Component watchers:
              Andrew Nicols, Mathew May, Michael Hawkins, Shamim Rezaie, Simey Lameze
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                11/May/15