[Don Jayamanne (DonJayamanne)]

For #4276

• Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
• Title summarizes what is changing
• [n/a] Has a news entry file (remember to thank yourself!)
• [n/a] Has sufficient logging.
• [n/a] Has telemetry for enhancements.
• [n/a] Unit tests & system/integration tests are added/updated
• [n/a] Test plan is updated as appropriate
• [n/a] package-lock.json has been regenerated by running npm install (if dependencies have changed)

[Derek Keeler (d3r3kk)]

Please have a look at my comments, I think they make sense but if you have some reason to push ahead I'm ok with it.

[codecov]

Codecov Report

Merging #4319 into master will increase coverage by 22%.
The diff coverage is 72%.

@@           Coverage Diff            @@
##           master   #4319     +/-   ##
========================================
+ Coverage      57%     79%    +22%     
========================================
  Files         348     422     +74     
  Lines       14566   19402   +4836     
  Branches     2252    3122    +870     
========================================
+ Hits         8263   15252   +6989     
+ Misses       5774    4147   -1627     
+ Partials      529       3    -526

Flag	Coverage Δ
#Linux	68% <72%> (?)
#Windows	54% <70%> (?)
#macOS	53% <70%> (?)

[codecov]

Codecov Report

Merging #4319 into master will increase coverage by 23%.
The diff coverage is 72%.

@@           Coverage Diff            @@
##           master   #4319     +/-   ##
========================================
+ Coverage      57%     79%    +23%     
========================================
  Files         348     422     +74     
  Lines       14566   19402   +4836     
  Branches     2252    3122    +870     
========================================
+ Hits         8263   15280   +7017     
+ Misses       5774    4119   -1655     
+ Partials      529       3    -526

Flag	Coverage Δ
#Linux	68% <72%> (?)
#Windows	69% <72%> (?)
#macOS	68% <72%> (?)