Motivation and Context

Throwing and logging exceptions in the runtime is a little verbose. It also creates quite a few conditional arcs for test coverage (or lack there-of). Utility method streamlines this pattern and eliminates all but one conditional arc.

throw new KernelException("message").Log(this._logger);

Note: Considered this pattern, opted on favoring the never-null object as the root of the extention method.

throw this._logger.LogException(new KernelException("message"));

Description

• Introduced shared exception extension to streamline logging and minimize conditional arcs.
• Switched to LogError override that accepts the exception object.
• Initialize logger based on Kernel.LoggingFactory (was previously always null)
• Opportunistically added parameter names to usage of Verify
• Opportunistically removed async/await for methods where not required.

Contribution Checklist

• The code builds clean without any errors or warnings
• The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
• All unit tests pass, and I have added new tests where possible
• I didn't break anyone 😄