Improve test coverage for bot.ts

[arunr-inji]

Current State

bot.ts has 0% branch coverage and 41.5% line coverage. The existing tests only check basic token validation but don't test the rest of the functionality.

This issue is to improve test coverage for bot.ts. This is coming off of discussion here

[arunr-inji]

@KnorpelSenf let me know if there's anything specific you'd like to see here. I can pick this issue up!

[KnorpelSenf]

I don't think there should be any surprises here.

The hard part here is probably to have good coverage of all the different ways in which things can happen. There is a bunch of promise handling, and the order of operations (when does which promise resolve) leads to vastly different code paths. This is probably best implemented using Promise.withResolvers which lets you resolve promises explicitly.

Otherwise, things are straightforward.

[arunr-inji]

@KnorpelSenf here's a first attempt at improving coverage for bot.ts: #860

Let me know what you think!

[KnorpelSenf]

@arunr-inji thank you! Review is there