[Tests] Remove inline awaits

[stacey-gammon]

We should make this part of our style guide as well, but inline awaits are confusing.

e.g. for the following statement is a() being awaited on, or b()?
await a().b()

To avoid this confusion we should just stop inlining awaits. The above usage is prolific in our tests when a() returns a promise, and I suspect causing some test instability.