Moodle
  1. Moodle
  2. MDL-29957

Cover datalib.php with unit tests

    Details

    • Difficulty:
      Moderate
    • Affected Branches:
      MOODLE_22_STABLE
    • Rank:
      19501

      Description

      it seems we are missing any unit test covering lib/datalib.php (it was detected when working on MDL-29912).

      So we need to implement them aimed to cover as much as possible. That would make things really easy to detect and test against all DBs, and given the DB-oriented nature of that lib, it is a win 100%

        Activity

        Hide
        Eloy Lafuente (stronk7) added a comment -

        sent to DEV backlog

        Show
        Eloy Lafuente (stronk7) added a comment - sent to DEV backlog
        Hide
        Michael de Raadt added a comment -

        Thanks for reporting this issue.

        We have detected that this issue has been inactive for over a year. It was reported as affecting versions that are no longer supported.

        If you believe that this issue is still relevant to current versions (2.5 and beyond), please comment on the issue. Issues left inactive for a further month will be closed.

        Michael d.

        TW9vZGxlDQo=

        Show
        Michael de Raadt added a comment - Thanks for reporting this issue. We have detected that this issue has been inactive for over a year. It was reported as affecting versions that are no longer supported. If you believe that this issue is still relevant to current versions (2.5 and beyond), please comment on the issue. Issues left inactive for a further month will be closed. Michael d. TW9vZGxlDQo=
        Hide
        Michael de Raadt added a comment -

        I'm closing this issue as it has been inactive for over a year has been recorded as affecting versions that are no longer supported.

        This is being done as part of a bulk annual clean-up of issues.

        If you still believe this is an issue in supported versions, please create a new issue.

        Show
        Michael de Raadt added a comment - I'm closing this issue as it has been inactive for over a year has been recorded as affecting versions that are no longer supported. This is being done as part of a bulk annual clean-up of issues. If you still believe this is an issue in supported versions, please create a new issue.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: