As a way of improving the speed of behat tests we could reduce the amount of time spent waiting after a redirect() call with a message.

The delay parameter is used so that a message can be read by a human before the redirect. The automated tests don't need this delay to read and in fact we're just wasting time.

Of course this is pretty horrible anyway and we should be moving away from this for humans too because it makes Moodle feel slow. But thats not a job for today (and there are other issues open for it).

In my test test on @mod_forum the test running time was reduced by 10% by forcing the delay to 1s under behat.

before:

21 scenarios (21 passed)
785 steps (785 passed)
19m53.102s

after:

21 scenarios (21 passed)
785 steps (785 passed)
17m47.896s