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

Disable xdebug for travis

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 3.0.1
    • Fix Version/s: 3.0.2
    • Component/s: Unit tests
    • Labels:
    • Testing Instructions:
      Hide
      1. Sync with travis
      2. Push a branch with this commit, and another one without this commit
      3. Race them
      4. Confirm that things still work with this commit, and that they are hopefully faster too
      Show
      Sync with travis Push a branch with this commit, and another one without this commit Race them Confirm that things still work with this commit, and that they are hopefully faster too
    • Affected Branches:
      MOODLE_30_STABLE
    • Fixed Branches:
      MOODLE_30_STABLE
    • Pull Master Branch:
      MDL-52595-master

      Description

      As others have found out (e.g. Composer), php can be as much as 3x slower when run with xdebug (https://getcomposer.org/doc/articles/troubleshooting.md#xdebug-impact-on-composer).

      Tests on travis show that we can halve our test execution time by disabling xdebug before we run them: https://travis-ci.org/andrewnicols/moodle/builds/98444235

      At the moment we do not need xdebug for phpunit or behat. They are only required for code coverage analysis.

      In the case of php >= 56, we can also swap it out for phpdbg -qrr if we do want it. phpdbg should be faster than xdebug.

        Attachments

          Activity

            People

            • Assignee:
              dobedobedoh Andrew Nicols
              Reporter:
              dobedobedoh Andrew Nicols
              Peer reviewer:
              Rajesh Taneja
              Integrator:
              Dan Poltawski
              Tester:
              Dan Poltawski
              Participants:
              Component watchers:
              Andrew Nicols, Mathew May, Michael Hawkins, Shamim Rezaie, Simey Lameze
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                11/Jan/16