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

Create some unit tests for lib/upgradelib.php

    Details

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

      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.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            stronk7 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
            stronk7 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
            poltawski Dan Poltawski added a comment -

            Integrated to master, thanks!

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

            Testing during integration

            Show
            poltawski Dan Poltawski added a comment - Testing during integration
            Hide
            stronk7 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
            stronk7 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:
                  Fix Release Date:
                  14/May/13