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

Move automated backup helper Unit Tests

    Details

    • Type: Task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.1
    • Fix Version/s: 2.3.2
    • Component/s: Backup, Unit tests
    • Labels:

      Description

      Move automated backup helper Unit Tests to backup/util/helper/tests/cronhelper_test.php

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            fred Frédéric Massart added a comment -

            This MUST NOT be integrated in 2.2, nor before MDL-28531 is closed. The branches are rebased on MDL-28531, which explains the extra commits.

            Show
            fred Frédéric Massart added a comment - This MUST NOT be integrated in 2.2, nor before MDL-28531 is closed. The branches are rebased on MDL-28531 , which explains the extra commits.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            With MDL-28531 already integrated, I'm going to push this too now...

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - With MDL-28531 already integrated, I'm going to push this too now...
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated, thanks! (23 and master)

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, thanks! (23 and master)
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            CI servers will tell us if the moved tests pass ok (don't forget testing instructions always ).

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - CI servers will tell us if the moved tests pass ok (don't forget testing instructions always ).
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Tests passed, so passing.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Tests passed, so passing.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            YEAR!*

            CAF*, TOT!*

            • Your effort amazingly resulted. (unbelievable :-P)
            • Closing as fixed.
            • Tons of thanks.
            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - YEAR!* CAF*, TOT!* Your effort amazingly resulted. (unbelievable :-P) Closing as fixed. Tons of thanks.

              People

              • Assignee:
                fred Frédéric Massart
                Reporter:
                fred Frédéric Massart
                Peer reviewer:
                Eloy Lafuente (stronk7)
                Integrator:
                Eloy Lafuente (stronk7)
                Tester:
                Eloy Lafuente (stronk7)
                Participants:
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  10/Sep/12