Details

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

      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()

        Issue Links

          Activity

          Petr Škoda created issue -
          Petr Škoda 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 ]
          Petr Škoda made changes -
          Status Open [ 1 ] Development in progress [ 3 ]
          Petr Škoda made changes -
          Link This issue will be resolved by MDL-31857 [ MDL-31857 ]
          Petr Škoda 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
          Petr Škoda 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
          Petr Škoda made changes -
          Link This issue is blocked by MDL-32251 [ MDL-32251 ]
          Hide
          Petr Škoda added a comment -

          to integrators: please integrate after MDL-32251

          Show
          Petr Škoda added a comment - to integrators: please integrate after MDL-32251
          Petr Škoda 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
          Sam Hemelryk made changes -
          Currently in integration Yes [ 10041 ]
          Petr Škoda 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
          Eloy Lafuente (stronk7) made changes -
          Status Waiting for integration review [ 10010 ] Integration review in progress [ 10004 ]
          Integrator stronk7
          Hide
          Petr Škoda added a comment -

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

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

          I have converted all backup tests and improved performance too.

          Show
          Petr Škoda added a comment - I have converted all backup tests and improved performance too.
          Petr Škoda 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
          Petr Škoda 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
          Petr Škoda 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
          Petr Škoda 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
          Petr Škoda 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
          Petr Škoda 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
          Petr Škoda 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
          Petr Škoda 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
          Petr Škoda 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()
          Petr Škoda 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()
          Petr Škoda 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()
          Petr Škoda 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()
          Petr Škoda 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
          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
          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
          Petr Škoda 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
          Petr Škoda added a comment -

          rebased on top of current integration master

          Show
          Petr Škoda added a comment - rebased on top of current integration master
          Petr Škoda made changes -
          Link This issue blocks MDL-32323 [ MDL-32323 ]
          Hide
          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
          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
          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
          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
          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
          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
          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
          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
          Eloy Lafuente (stronk7) added a comment -

          Integrated, thanks!

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

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

          Show
          Eloy Lafuente (stronk7) added a comment - And passing, as far as both simpletest and phpunit are completing without problems.
          Eloy Lafuente (stronk7) made changes -
          Status Testing in progress [ 10011 ] Tested [ 10006 ]
          Hide
          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
          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!
          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: