Fix multiplebrowser suitesummary

NavneetThekkumpat commented 1 year ago

Description

Console TestSuite summary is wrong when run with multiple browser config .

After fix:

[x] The code change is covered by unit tests. I have added tests that prove my fix is effective or that my feature works
[x] I have performed end-to-end test locally.
[x] New and existing unit tests pass locally with my changes
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I used clear names for everything
[x] I have performed a self-review of my own code

github-actions[bot] commented 1 year ago

Package	Line Rate	Branch Rate	Complexity	Health
Microsoft.PowerApps.TestEngine	90%	88%	914	✔
Summary	90% (2383 / 2633)	88% (556 / 630)	914	✔

Minimum allowed line rate is 85%

NavneetThekkumpat commented 1 year ago

Ensure the changes are tested in consuming applications (PAC CLI) before merging this PR.

Sure, but we would still need to change the ConsoleEventHandler in the PAC CLI side with these changes