Create more informational assert collection log for FunctionalTests o…

[lateapexearlyspeed]

…f FileSystemGlobbing.

Hello, this PR is for #44281 .

I verified in my local pc with this helper method in that test project by intentionally changing expected count or updating one item value. It will give more information than before. Also seems some other runtime projects were already using Assert.Collection(). I wrap this method into a 'private static void AssertCollection(IEnumerable expected, IEnumerable actual)' and use it in specified test file.

Please review and give suggestion if need, so that I can update code correspondingly, thanks very much and hope have a try to contribute .Net !

[dnfadmin]

All CLA requirements met.

[ghost]

Tagging subscribers to this area: @maryamariyan
See info in area-owners.md if you want to be subscribed.

[danmoseley]

I'll let @maryamariyan review this one. :)

[lateapexearlyspeed]

Hi @maryamariyan Seems CI pipeline was blocked by CI issue and was mentioned in #32805; also the check 'runtime (Libraries Build windows allConfigurations x64 Debug)' was being in progress from "4d 23h ago".

Is there any way I can re-trigger CI build besides of pushing a new commit, what is action I should do now, thanks.

[danmoseley]

You can close and reopen the PR.

[lateapexearlyspeed]

Hi @danmosemsft Thanks for tip :) All checks passed now. Any more comment, can it be merged ?

[danmoseley]

@maryamariyan ?

[lateapexearlyspeed]

Hi @maryamariyan Any chance you can have a look at this PR? :)

[maryamariyan]

Thanks for the update @lateapexearlyspeed. Looking now

[maryamariyan]

The approach LGTM with some nit comments.
Thanks @lateapexearlyspeed for the contribution. We'll make sure next follow ups are faster :)

[danmoseley]

@maryamariyan is this good to merge?

[maryamariyan]

It's merged now. Thank you!