is that actually a good change? aren't you relying on this promise to always be async in order for this test to actually test something? it's probably a safe bet, but in principle, you are relying on implementation, no?