-
Sub-task
-
Resolution: Fixed
-
Minor
-
3.11.10, 4.0.4, 4.1
-
MOODLE_311_STABLE, MOODLE_400_STABLE, MOODLE_401_STABLE
-
MOODLE_311_STABLE, MOODLE_400_STABLE
-
This corresponds to the progress of point B of MDL-71049.
In this case applied to more testcases, they are a bunch, so no worth listing them like in previous issues that were for testcases with many repetitions.
I'll be creating a few (5-6) issues like this to advance with the remaining cases (80-100 files on each) to keep it more or less manageable (review, rebase...).
Changes include:
- Add or fix namespaces to be correct component ones. With sub-namespaces when the tests belong to a known API.
- Fixing incorrect use \xxx statements with leading backslash.
- Remove file phpdoc blocks.
- Remove MOODLE_INTERNAL if not needed.
- Changing code to point to global scope when needed (when referencing to classes belonging to other components) or adding new use xxx statements (for classes corresponding to the same component/area).
- Change testcase class names to match file name (_test.
Goals:
- Verify that there aren't remaining files and classes to be processed.
- Verify that all the files can be run individually.
- Verify that complete runs continue passing ok.
- Apply the changes to both 311 (where PHPUnit was introduced) and master.
Notes:
- The changes should be 99.99% contained within phpunit /tests/ files. If something out from there is detected, it will be handled in separate commit, with justifications.