Updated icons and tooltip in test explorer indicating status of test files/suites

[karrtikr]

For #4583 #4689

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

[codecov]

Codecov Report

Merging #4650 into master will decrease coverage by 1%.
The diff coverage is 68%.

@@           Coverage Diff           @@
##           master   #4650    +/-   ##
=======================================
- Coverage      77%     77%   -<1%     
=======================================
  Files         447     447            
  Lines       21345   21377    +32     
  Branches     3493    3506    +13     
=======================================
+ Hits        16346   16369    +23     
- Misses       4995    5004     +9     
  Partials        4       4

Flag	Coverage Δ
#Linux	66% <66%> (+1%)	⬆️
#Windows	66% <66%> (+1%)	⬆️
#macOS	66% <66%> (+1%)	⬆️

[ericsnowcurrently]

Why are the .svg files modified? Are those the ones from @luabud?

[DonJayamanne]

Please add tests for how we calculate pass count.

[karrtikr]

Are those the ones from @luabud?

Actually No. But we are gonna the change the icons eventually anyways ( those don't match with one described in issues). Should i still revert it?

addressed

[karrtikr]

@DonJayamanne @ericsnowcurrently
Implemented based on the comment of Dan:

If any test fails, the parent should be failure, otherwise pass if all tests that ran passed, and not run if no tests ran.