Improving and Adding Logs for Workflow

[RyanLettieri]

Description

Added in logging statements for dapr workflow and optional parameter for setting the log level.

Note that the failures are due to every individual case of a large switch-case statement not being tested

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #946

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

• Code compiles correctly
• Created/updated tests
• Extended the documentation

[mukundansundar]

@RyanLettieri Can you look into the coverage checks that are failing in this PR?

[RyanLettieri]

@RyanLettieri Can you look into the coverage checks that are failing in this PR?

The coverage checks are due to the switch case statement for the log levels. We log what the log level was set to in the switch case statements, I.e. "Severe", "Warning", etc. I didn't believe it made much sense to create test cases for each individual log level but if needed I certainly can.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (40617f1) 76.23% compared to head (0cf104e) 76.29%.

Additional details and impacted files

@@             Coverage Diff              @@
##             master     #978      +/-   ##
============================================
+ Coverage     76.23%   76.29%   +0.06%     
- Complexity     1462     1463       +1     
============================================
  Files           138      138              
  Lines          4489     4497       +8     
  Branches        524      524              
============================================
+ Hits           3422     3431       +9     
  Misses          784      784              
+ Partials        283      282       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.