Replace done w/ async part 4

[mdrichardson]

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

Covers

(Alphabetically such that botbuilder comes before botbuilder-)

• botbuilder-dialogs

Excludes

None

Description

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

Specific Changes

• done => async/await
• 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 683873092

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 5 unchanged lines in 3 files lost coverage.
• Overall coverage decreased (-0.03%) to 85.094%

---

Files with Coverage Reduction	New Missed Lines	%
libraries/botbuilder-lg/src/templateExtensions.ts	1	84.48%
libraries/adaptive-expressions/src/triggerTrees/node.ts	2	95.26%
libraries/adaptive-expressions/src/triggerTrees/trigger.ts	2	61.63%

Totals
Change from base Build 683871407:	-0.03%
Covered Lines:	18771
Relevant Lines:	21006

---

💛 - Coveralls

[mdrichardson]

Note: This test wasn't deleted; it's the one above. The diff just looks weird.