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

          Attachments

            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