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

Cleanup cohorts code and add tests

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.4
    • Component/s: Cohorts, Enrolments
    • Labels:
    • Testing Instructions:
      Hide

      1/ execute phpunit tests (please note the phpunit environment needs to be reinitialised in order to get the phpunit.xml with new suite updated)

      2/ go to cohort admin UI and try all operations there:

      • adding new cohort
      • deleting cohort
      • adding and removing members
      • searching cohorts
      Show
      1/ execute phpunit tests (please note the phpunit environment needs to be reinitialised in order to get the phpunit.xml with new suite updated) 2/ go to cohort admin UI and try all operations there: adding new cohort deleting cohort adding and removing members searching cohorts
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w38_MDL-35465_m24_cohortscleanup

      Description

      There are some coding style problems, tiny bugs and documentation issues in the cohorts related code. We also need unit tests and generators before making any changes and adding new features.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              skodak Petr Skoda added a comment -

              If tests fail in Oracle or MSSQL then please create a new issue.

              Show
              skodak Petr Skoda added a comment - If tests fail in Oracle or MSSQL then please create a new issue.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Integrated, thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
              Hide
              andyjdavis Andrew Davis added a comment -

              Seems to work fine. I did get some phpunit failures but they seem to be unrelated.

              Show
              andyjdavis Andrew Davis added a comment - Seems to work fine. I did get some phpunit failures but they seem to be unrelated.
              Hide
              poltawski Dan Poltawski added a comment -

              Congratulations, you've done it!

              Thanks, this change is now in the latest weekly release!

              Join the crowds of people tomorrow from 8am and download this Moodle release from your local apple store!

              Show
              poltawski Dan Poltawski added a comment - Congratulations, you've done it! Thanks, this change is now in the latest weekly release! Join the crowds of people tomorrow from 8am and download this Moodle release from your local apple store!

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    3/Dec/12