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

          Attachments

            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