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

Cleanup cohorts code and add tests

    XMLWordPrintable

    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.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              skodak Petr Skoda
              Reporter:
              skodak Petr Skoda
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Andrew Davis
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Dates

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