Change tests to fail on logged errors unless they are expected

[cramforce]

A recent change no longer async throws these errors, but relying on that to fail the test was always brittle.

It would be great if we could add a device to expect error logs and fail tests if they don't do so.

[rsimha]

In #14336, a stub was checked in for console.error that would throw an error when called from within a test.

It turns out that a more elegant way of doing this is via a mock. Coming up via #14432.