Details

    • Type: Improvement
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.3
    • Component/s: Unit tests
    • Labels:

      Description

      Includes:

      • constants refactoring
      • new half-finished geenrator
      • reworked db table init
      • support for $CFG->debug = -1
      • functional DB tests
      • fixed $DB->get_indexes() to not throw exceptions when table does not exist
      • fix handling of user passwords in test db
      • add debug info to exception messages
      • removed unnecessary PHP debug errors from mathslib
      • fixed @error suppression in get_string
      • fixed PHPUnit error handler setup
      • added timezone info to default install

      ====================

      remaining tests:

      • grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
      • completionlib - using mocks heavily
      • /question/* (Tim?)
      • webservices - seems to do nothing
      • /mod/data/simpletest/* needs to be rewritten to use data generator instead of stored CSV records that include links ids

      TODO:

      • discuss problem in navigationlib_test.php method test_find_expandable()

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              skodak Petr Skoda created issue -
              skodak Petr Skoda made changes -
              Field Original Value New Value
              Fix Version/s DEV backlog [ 10464 ]
              Priority Minor [ 4 ] Critical [ 2 ]
              Labels triaged
              Assignee Eloy Lafuente (stronk7) [ stronk7 ] Petr Škoda (skodak) [ skodak ]
              skodak Petr Skoda made changes -
              Status Open [ 1 ] Development in progress [ 3 ]
              skodak Petr Skoda made changes -
              Link This issue will be resolved by MDL-31857 [ MDL-31857 ]
              skodak Petr Skoda made changes -
              Description advanced_testcase implementation:
              * generator for courses, users, etc.


              Note to self:
              * do not forget to add new textlib tests
              advanced_testcase implementation:
              * generator for courses, users, etc.


              Note to self:
              * do not forget to add new textlib tests
              * add test if PHPUnit package installed - class_exists in admin/tool/phpunit/cli/util.php
              skodak Petr Skoda made changes -
              Status Development in progress [ 3 ] Waiting for integration review [ 10010 ]
              Pull Master Diff URL https://github.com/skodak/moodle/compare/caee6e6c...w14_MDL-32149_m23_phpunit2
              Pull Master Branch w14_MDL-32149_m23_phpunit2
              Pull from Repository git://github.com/skodak/moodle.git
              Fix Version/s 2.3 [ 10657 ]
              Fix Version/s DEV backlog [ 10464 ]
              Testing Instructions 1/ run original SimpleTests
              2/ run new PHPUnit tests
              skodak Petr Skoda made changes -
              Link This issue is blocked by MDL-32251 [ MDL-32251 ]
              Hide
              skodak Petr Skoda added a comment -

              to integrators: please integrate after MDL-32251

              Show
              skodak Petr Skoda added a comment - to integrators: please integrate after MDL-32251
              skodak Petr Skoda made changes -
              Description advanced_testcase implementation:
              * generator for courses, users, etc.


              Note to self:
              * do not forget to add new textlib tests
              * add test if PHPUnit package installed - class_exists in admin/tool/phpunit/cli/util.php
              Includes:
              * new advanced_testcase
              * half-finished data generator
              * constants refactoring
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install
              samhemelryk Sam Hemelryk made changes -
              Currently in integration Yes [ 10041 ]
              skodak Petr Skoda made changes -
              Description advanced_testcase implementation:
              * generator for courses, users, etc.


              Note to self:
              * do not forget to add new textlib tests
              * add test if PHPUnit package installed - class_exists in admin/tool/phpunit/cli/util.php
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Waiting for integration review [ 10010 ] Integration review in progress [ 10004 ]
              Integrator stronk7
              Hide
              skodak Petr Skoda added a comment -

              I have added one more commit fixing the globals reset and improved $USER handling

              Show
              skodak Petr Skoda added a comment - I have added one more commit fixing the globals reset and improved $USER handling
              Hide
              skodak Petr Skoda added a comment -

              I have converted all backup tests and improved performance too.

              Show
              skodak Petr Skoda added a comment - I have converted all backup tests and improved performance too.
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be completely messed up from MDL-24330, it might be best to revert it completely
              * portfolio tests
              * repository test
              * completion and conditional lib test (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/*_test.php (Tim?)
              * course test
              * /filter/
              * /mod/
              * ratings
              * webservices
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be completely messed up from MDL-24330, it might be best to revert it completely
              * portfolio tests
              * repository test
              * completion and conditional lib test (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/*_test.php (Tim?)
              * course test
              * /filter/
              * /mod/
              * ratings
              * webservices
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be completely messed up from MDL-24330, it might be best to revert it completely
              * portfolio tests
              * repository test
              * completion and conditional lib test (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/*_test.php (Tim?)
              * /filter/
              * /mod/
              * ratings
              * webservices
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be completely messed up from MDL-24330, it might be best to revert it completely
              * portfolio tests
              * repository test
              * completion and conditional lib test (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/*_test.php (Tim?)
              * /filter/
              * /mod/
              * ratings
              * webservices
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be completely messed up from MDL-24330, it might be best to revert it completely
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/*_test.php (Tim?)
              * /mod/
              * ratings
              * webservices
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be completely messed up from MDL-24330, it might be best to revert it completely
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/*_test.php (Tim?)
              * /mod/
              * ratings
              * webservices
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be completely messed up from MDL-24330, it might be best to revert it completely
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/*_test.php (Tim?)
              * /mod/
              * webservices
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be completely messed up from MDL-24330, it might be best to revert it completely
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/*_test.php (Tim?)
              * /mod/
              * webservices
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be completely messed up from MDL-24330, it might be best to revert it completely
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/* and /mod/quit/*(Tim?)
              * /mod/wiki/* (David?)
              * webservices
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be completely messed up from MDL-24330, it might be best to revert it completely
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/* and /mod/quit/*(Tim?)
              * /mod/wiki/* (David?)
              * webservices
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/* and /mod/quit/*(Tim?)
              * /mod/wiki/* (David?)
              * webservices
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/* and /mod/quit/*(Tim?)
              * /mod/wiki/* (David?)
              * webservices
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/* and /mod/quiz/*(Tim?)
              * /mod/wiki/* (David?)
              * webservices
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/* and /mod/quiz/*(Tim?)
              * /mod/wiki/* (David?)
              * webservices
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/* and /mod/quiz/*(Tim?)
              * webservices
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * navigation lib tests (SamH?)
              * /question/* and /mod/quiz/*(Tim?)
              * webservices
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * /question/* and /mod/quiz/*(Tim?)
              * webservices

              TODO:
              * discuss problem in navigationlib_test.php method test_find_expandable()
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * /question/* and /mod/quiz/*(Tim?)
              * webservices

              TODO:
              * discuss problem in navigationlib_test.php method test_find_expandable()
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * /question/* and /mod/quiz/*(Tim?)
              * webservices (seems to do nothing)

              TODO:
              * discuss problem in navigationlib_test.php method test_find_expandable()
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * portfolio tests
              * repository test
              * completion and conditional lib tests (using mocks heavily)
              * /question/* and /mod/quiz/*(Tim?)
              * webservices (seems to do nothing)

              TODO:
              * discuss problem in navigationlib_test.php method test_find_expandable()
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * completion and conditional lib tests (using mocks heavily)
              * /question/* and /mod/quiz/*(Tim?)
              * webservices (seems to do nothing)

              TODO:
              * discuss problem in navigationlib_test.php method test_find_expandable()
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * completion and conditional lib tests (using mocks heavily)
              * /question/* and /mod/quiz/*(Tim?)
              * webservices (seems to do nothing)

              TODO:
              * discuss problem in navigationlib_test.php method test_find_expandable()
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * completion and conditional lib tests (using mocks heavily)
              * /question/* (Tim?)
              * webservices (seems to do nothing)

              TODO:
              * discuss problem in navigationlib_test.php method test_find_expandable()
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * completion and conditional lib tests (using mocks heavily)
              * /question/* (Tim?)
              * webservices (seems to do nothing)

              TODO:
              * discuss problem in navigationlib_test.php method test_find_expandable()
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * completionlib - using mocks heavily
              * /question/* (Tim?)
              * webservices - seems to do nothing

              TODO:
              * discuss problem in navigationlib_test.php method test_find_expandable()
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Note to avoid forgetting it. Today MDL-17327 has been integrated and it has added a bunch of simpletest tests. Those will need to migrate those too.

              We'll need to delete already migrated tests ASAP, ensuring we are not forgetting anything like those new ones in mod/data.

              Ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Note to avoid forgetting it. Today MDL-17327 has been integrated and it has added a bunch of simpletest tests. Those will need to migrate those too. We'll need to delete already migrated tests ASAP, ensuring we are not forgetting anything like those new ones in mod/data. Ciao
              skodak Petr Skoda made changes -
              Description Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * completionlib - using mocks heavily
              * /question/* (Tim?)
              * webservices - seems to do nothing

              TODO:
              * discuss problem in navigationlib_test.php method test_find_expandable()
              Includes:
              * constants refactoring
              * new half-finished geenrator
              * reworked db table init
              * support for $CFG->debug = -1
              * functional DB tests
              * fixed $DB->get_indexes() to not throw exceptions when table does not exist
              * fix handling of user passwords in test db
              * add debug info to exception messages
              * removed unnecessary PHP debug errors from mathslib
              * fixed @error suppression in get_string
              * fixed PHPUnit error handler setup
              * added timezone info to default install

              ====================

              remaining tests:
              * grade tests seem to be kind of broken from MDL-24330, it might be best to revert it completely (do not ask me why it passes at present, I tried conversion and failed pretty badly)
              * completionlib - using mocks heavily
              * /question/* (Tim?)
              * webservices - seems to do nothing
              * /mod/data/simpletest/* needs to be rewritten to use data generator instead of stored CSV records that include links ids

              TODO:
              * discuss problem in navigationlib_test.php method test_find_expandable()
              Hide
              skodak Petr Skoda added a comment -

              rebased on top of current integration master

              Show
              skodak Petr Skoda added a comment - rebased on top of current integration master
              skodak Petr Skoda made changes -
              Link This issue blocks MDL-32323 [ MDL-32323 ]
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Found 2 portfolio leftovers preventing simpletest to complete:

              • boxnet/simpletest/testportfoliopluginboxnet.php
              • download/simpletest/testportfolioplugindownload.php

              Added commit on top deleting them.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Found 2 portfolio leftovers preventing simpletest to complete: boxnet/simpletest/testportfoliopluginboxnet.php download/simpletest/testportfolioplugindownload.php Added commit on top deleting them.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Added another commit about some expectError() uses in lib/simpletest/testmathslib.php leading to failures. It seems it was borked in previous simpletest version and not it was working but leading to failures.

              I've executed the tests both with E_ALL | E_STRICT, E_NOTICE and 0 and the tests passed always. So, cleaning those 3 expectError()

              Ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Added another commit about some expectError() uses in lib/simpletest/testmathslib.php leading to failures. It seems it was borked in previous simpletest version and not it was working but leading to failures. I've executed the tests both with E_ALL | E_STRICT, E_NOTICE and 0 and the tests passed always. So, cleaning those 3 expectError() Ciao
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Simpletest: 196/203 test cases complete: 7737 passes, 0 fails and 0 exceptions.
              PHPUnit: OK (579 tests, 17162 assertions).

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Simpletest: 196/203 test cases complete: 7737 passes, 0 fails and 0 exceptions. PHPUnit: OK (579 tests, 17162 assertions).
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Adding Tim, so he can start swimming in the amazing world of Moodle supporting phpunit and lots of cases already available.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Adding Tim, so he can start swimming in the amazing world of Moodle supporting phpunit and lots of cases already available.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Integrated, thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Integration review in progress [ 10004 ] Waiting for testing [ 10005 ]
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Waiting for testing [ 10005 ] Testing in progress [ 10011 ]
              Tester stronk7
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              And passing, as far as both simpletest and phpunit are completing without problems.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - And passing, as far as both simpletest and phpunit are completing without problems.
              stronk7 Eloy Lafuente (stronk7) made changes -
              Status Testing in progress [ 10011 ] Tested [ 10006 ]
              Hide
              nebgor Aparup Banerjee added a comment -

              The code here has been spread to upstream moodle repositories and mirrors for anyone to use .

              Closing, have a good weekend!

              Show
              nebgor Aparup Banerjee added a comment - The code here has been spread to upstream moodle repositories and mirrors for anyone to use . Closing, have a good weekend!
              nebgor Aparup Banerjee made changes -
              Status Tested [ 10006 ] Closed [ 6 ]
              Resolution Fixed [ 1 ]
              Currently in integration Yes [ 10041 ]
              Integration date 05/Apr/12

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    25/Jun/12