test: minor login test cleanup

[rossiam]

I want to do at least a little more clean-up of these tests but have limited this PR because of the indentation changes due to the removal of the top-level describe.

• remove superfluous top-level describe
• mock delay function from util to wait 10ms

Checklist

• I have read the CONTRIBUTING document
• My code follows the code style of this project (npm run lint produces no warnings/errors)

[changeset-bot]

⚠️ No Changeset found

Latest commit: b0dd11e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[Sitlintac]

Doesn't this need to be dynamically imported in order to be mocked?

[rossiam]

I think you're confusing this with the import for the file under test, which we do have to dynamically import after setting up the mocks. The only reason I'm even including this and its ilk is so I can use them with typeof to get the types of the mock correct. (In this case, the const delayMock: jest.Mock<typeof delay> = jest.fn() line.)