Add error detection when EmitEventAsync fails in internal RunAsync

[sw-joelmut]

minor

Description

This PR adds error detection when the internal RunAsync method fails using AdaptiveDialogs, otherwise, the original Exception gets lost.

e.g.: original exception containing circular reference errors, it's captured by the dialogContext.EmitEventAsync to send the error as a message, then, it proceeds to fail, and the error thrown in the stack is the circular reference error, and the original exception gets lost.

Specific Changes

• Added try/catch around the EmitEventAsync when used in the InternalRunAsync method. If there is an error, it will be captured and thrown as an AggregateException containing the original and new error.
• Added unit test to validate this behavior.
• Updated unit test failing with similar behavior.

Testing

The following image shows the pipeline working and how the error is thrown.

[coveralls]

Pull Request Test Coverage Report for Build 378002

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 6 unchanged lines in 3 files lost coverage.
• Overall coverage remained the same at 78.438%

---

Files with Coverage Reduction	New Missed Lines	%
/libraries/AdaptiveExpressions/BuiltinFunctions/GetNextViableTime.cs	1	90.91%
/libraries/AdaptiveExpressions/BuiltinFunctions/GetPreviousViableTime.cs	1	90.91%
/libraries/Microsoft.Bot.Builder.Dialogs/DialogExtensions.cs	4	94.74%
<!--	Total:	6		-->

Totals
Change from base Build 377857:	0.0%
Covered Lines:	26127
Relevant Lines:	33309

---

💛 - Coveralls

[BruceHaley]

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll