Use one-time login link instead of user login form in BrowserTestBase tests

Locally I have \Drupal\Tests\block_content\Functional\BlockContentListTest passing but it looks like there are lots of fails in other tests.

Well that was fun. There are three tests remaining that use the login form. One is the UserLoginTest, so we need to keep that. The other two seem to be oddities that could probably be fixed, perhaps in a followup.

It's hard to evaluate if this is overall faster than before because there is such variability in the test suite times. I did see one run that was under five minutes. In my local testing I found a saving of around 200ms per login. So that's roughly 5 mins total savings over 1500 logins.

• Remove the duplicate ::userPassResetUrl method and updated the existing user_pass_reset_url function to use current time as discussed with @catch in Slack.
• Updated UserLoginTest::testLoginCacheTagsAndDestination to explicitly submit the login form instead of relying on ::drupalLogin
• Added #3471101: Further improve user login in tests (Follow up for #3469309)

HEAD was broken. Have rebased and it's green again.

We can probably add

protected bool $useOneTimeLoginLinks = FALSE;

to the failing test.

Added MR for 11.0.x

I won't get to this for a bit, but see #17 and 21 and apply this to 11.x, it was already applied to the other branches.