| commit | 7c64a411d76a168c038839aa0d21241ae9a19cb9 | |
| author | Andrew Nicols <andrew@nicols.co.uk> | |
| Wed, 21 Dec 2022 01:41:48 +0000 (21 09:41 +0800) | ||
| committer | Andrew Nicols <andrew@nicols.co.uk> | |
| Tue, 23 May 2023 02:55:42 +0000 (23 10:55 +0800) | ||
| tree | 4f902fb31be9ee0d1358e785a9be3477745e253f | treesnapshot (tar.gz zip) |
| parent | 4781b41631664d3c5e7e94cfcc279b2af81e4dd1 | commitdiff |
MDL-76733 mod_assign: Tidy up availability behat tests
This feature had lots of small issues and it made sense to fix it whilst
investigating a query:
* most of the steps do not require JavaScript
* it uses the UI to set an admin setting, for every scenario:
** only 3-4 of the scenarios actually test that setting
** it is very slow to do it his way when we have a generator step we can use
* we create two assignments in the Background, but each test only uses one of them
* we create the assignments in the Background with a generator, but
update them to modify various settings in each Scenario using the UI
when we should just create one assignment for each test and set it up
correctly for that Scenario
This feature had lots of small issues and it made sense to fix it whilst
investigating a query:
* most of the steps do not require JavaScript
* it uses the UI to set an admin setting, for every scenario:
** only 3-4 of the scenarios actually test that setting
** it is very slow to do it his way when we have a generator step we can use
* we create two assignments in the Background, but each test only uses one of them
* we create the assignments in the Background with a generator, but
update them to modify various settings in each Scenario using the UI
when we should just create one assignment for each test and set it up
correctly for that Scenario
| mod/assign/tests/behat/set_availability.feature | diffblobblamehistory |