Moodle
  1. Moodle
  2. MDL-33180

remove remainders of SimpleTest support in 2.4dev

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.4
    • Component/s: Unit tests
    • Labels:
    • Rank:
      41066

      Description

      • old unit test admin tool
      • lib/simpletest*
      • code coverage
      • $PHPUNIT global
      • docs
      • etc.

        Issue Links

          Activity

          Hide
          Michael de Raadt added a comment -

          Hi, Petr.

          You mentioned your intention to work on this in a recent meeting.

          Now is apparently the time to make this change if it is a deprecation. I'll leave it to you.

          Show
          Michael de Raadt added a comment - Hi, Petr. You mentioned your intention to work on this in a recent meeting. Now is apparently the time to make this change if it is a deprecation. I'll leave it to you.
          Hide
          Petr Škoda added a comment -

          yes, starting

          Show
          Petr Škoda added a comment - yes, starting
          Hide
          Sam Hemelryk added a comment -

          Changes look PERFECT thanks Petr, before this gets integrated I just want to double check with MD that he's happy with it. Assuming so it is as good as in!

          Show
          Sam Hemelryk added a comment - Changes look PERFECT thanks Petr, before this gets integrated I just want to double check with MD that he's happy with it. Assuming so it is as good as in!
          Hide
          Sam Hemelryk added a comment -

          Confirmed with MD, this has now been eradicated from master. Thanks Petr!

          Show
          Sam Hemelryk added a comment - Confirmed with MD, this has now been eradicated from master. Thanks Petr!
          Hide
          Rajesh Taneja added a comment -

          phpunit passes.

          Thanks for fixing this Petr.

          Show
          Rajesh Taneja added a comment - phpunit passes. Thanks for fixing this Petr.
          Hide
          Aparup Banerjee added a comment -

          yay, it works!

          This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week.

          Thank you all for taking the time to get us here.

          cheers!

          Show
          Aparup Banerjee added a comment - yay, it works! This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week. Thank you all for taking the time to get us here. cheers!

            People

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

              Dates

              • Created:
                Updated:
                Resolved: