Replace done w/ async part 2

[mdrichardson]

Partial fix for #3226. This will be done in multiple PRs.

Covers

(Alphabetically such that botbuilder comes before botbuilder-)

• botbuilder-ai through botbuilder-applicationinsights, inclusive

Excludes

• Many tests in TelemetryWaterfallTest.test.js

Description

Replaces the use of done() in tests with proper async/await.

Specific Changes

• done => async/await
• Remove done completely when used in synchronous tests. For example, it was in some beforeEach() hooks, but it's not needed; nock doesn't even use it there
• Remove some of the try/catches that are no longer needed since we don't need done(err)
• Avoid arrow function in mocha tests
• A lot of no-op stuff auto-fixes with our prettier/eslint settings

Notes (mostly for me)

• Regex for finding done usage: (done =>)|\(done\)
• Search in: ./libraries/**/**/tests/**/*.js
• For adding await to adapter.send() calls: (?<!(await)|(const))\sadapter

[mdrichardson]

Note: This is moved into afterEach(), below

[coveralls]

Pull Request Test Coverage Report for Build 676716494

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.04%) to 85.084%

---

Totals
Change from base Build 665654550:	0.04%
Covered Lines:	18786
Relevant Lines:	21032

---

💛 - Coveralls

[joshgummersall]

One quick comment, otherwise it looks good.