Moodle
  1. Moodle
  2. MDL-39005

Create some unit tests for lib/upgradelib.php

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Unit tests
    • Labels:
    • Rank:
      49114

      Description

      While working on MDL-22153, I detected that we had 0 coverage for the stuff present @ lib/upgradelib.php.

      This is about to test some of the classes & methods there, not aiming for complete coverage but to have the suite covering some points and ready for more to be added.

        Issue Links

          Activity

          Hide
          Eloy Lafuente (stronk7) added a comment -

          Sending straight to integration, not many tests (just 1), but the one I needed to have MDL-22153 changes covered.

          Show
          Eloy Lafuente (stronk7) added a comment - Sending straight to integration, not many tests (just 1), but the one I needed to have MDL-22153 changes covered.
          Hide
          Dan Poltawski added a comment -

          Integrated to master, thanks!

          Show
          Dan Poltawski added a comment - Integrated to master, thanks!
          Hide
          Dan Poltawski added a comment -

          Testing during integration

          Show
          Dan Poltawski added a comment - Testing during integration
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Your awesome contributions are now part of Moodle, your fav LMS out there.

          Closing this as fixed.

          Many thanks for all the hard work, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Your awesome contributions are now part of Moodle, your fav LMS out there. Closing this as fixed. Many thanks for all the hard work, ciao

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: