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

Specify phpunit dependency as require-dev instead of require

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4.1, 2.5
    • Fix Version/s: 2.4.1
    • Component/s: Unit tests
    • Labels:
    • Testing Instructions:
      Hide

      1/ delete dirroot/vendor/
      2/ delete dirrott/composer.lock
      3/ try php composer.phar install --dev
      4/ verify phpunit is installed into vendor dir
      5/ verify --dev was added to the info in the phpunit admin tool

      Show
      1/ delete dirroot/vendor/ 2/ delete dirrott/composer.lock 3/ try php composer.phar install --dev 4/ verify phpunit is installed into vendor dir 5/ verify --dev was added to the info in the phpunit admin tool
    • Affected Branches:
      MOODLE_24_STABLE, MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w49_MDL-36936_m25_composerdev

      Description

      In 2.4 phpunit is specified as a 'require' package in the composer installer (composer.json), the 'require-dev' attribute, which is more appropriate for dev/test purposes can be used instead of it. Would be useful if in future Moodle intends to use composer to manage other libraries dependencies (used by Moodle core). The only difference using this attribute instead of 'require' is that composer has to be executed with --dev option (http://getcomposer.org/doc/04-schema.md#require-dev)

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              skodak Petr Skoda
              Reporter:
              dmonllao David Monllaó
              Integrator:
              Sam Hemelryk
              Tester:
              David Monllaó
              Participants:
              Component watchers:
              Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                14/Jan/13