Replace done w/ async in tests part 1

[mdrichardson]

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

Covers

(Alphabetically such that botbuilder comes before botbuilder-)

• adaptive-expressions-ie11 through botbuilder, inclusive

Excludes

• Many tests in channelServiceRoutes.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

[coveralls]

Pull Request Test Coverage Report for Build 676755689

• 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

[mdrichardson]

@joshgummersall @stevengum I opted not to make changes to channelServiceRoutes.test.js because the conversion isn't as straight-forward. I could leave it as-is, or implement something like this. Preference?

Edit: Josh and I discussed offline and decided it's best to leave as-is.

[joshgummersall]

@mdrichardson, one thing that might be helpful is to add https://www.npmjs.com/package/eslint-plugin-mocha for our test files. One reason is that it enforces using function () {} instead of () => {}. We have a section in the root eslint configuration that does some test-specific overrides (see here). Hopefully, we can add the mocha plugin to this configuration. Can you give that a shot?