It's good to keep the test because it's not easy to
reproduce the issue to be fixed. Part of the fix will
be to remove this skip.

Merge branch 'MDL-72843-qbank_comment-unit-testsuite-resolve' of https://github.com/catalyst/moodle-MDL-70329

Merge branch 'MDL-72924-fix-component-initialisation-for-performance-checks' of https://github.com/keevan/moodle

Ideally any new test should be using a class name matching
the file name, and to use namespace when clearly belonging to
a component (sub-namespace also allowed for better accuracy).

And, also, adding a @covers label to indicate which class
(preferred, method is also possible) is being covered with the test.

When the messagingallusers setting is disabled only contacts that are
on the same course as the user should be returned as results.

We can reduce the number of candidate users significantly by
pre-filtering them by being enrolled on the same courses as the user
performing the search.

Since we must also return the user themselves sometimes, even when they
are not enrolled on a course I used a UNION as the most efficient way of
retriving them, this meant that the order by had to reference returned
columns by number to be sufficiently cross-database compatiable.

This also implements the functionality for SCORM packages, which may
have SVG images embedded within their content which require loading
directly (and are only created by users with appropriate risk flag).

- Update the behat tests to more precisely check the state in the completion report.
- Check combination of completion conditions with passing grade

- Reverts the custom flow logic introduced in the 52206
- Expand unit tests to cover different grade based completion settings