Analyzer to convert Assert.Collection to Assert.Single

[bradwilson]

While updating the xUnit.net source, I noticed that I did this a lot:

Assert.Collection(
    collection,
    item => // assertion(s) for the single item
);

This could/should be converted to this:

var item = Assert.Single(collection);
// assertion(s) for the single item

[netchkin]

Can you please include an example for // assertion(s) for the single item in Assert.Collection example?

[adison88]

@bradwilson I see most of these usages in unit tests in xUnit.net source. I can pick this one up to update the source. Please let me know if there're any other places which needs an update.

[adison88]

Hello @bradwilson , created #2230 for review.

[vb2ae]

@bradwilson did pr #2230 fix the issue? Wondering if this ticket should be closed

[bradwilson]

It did. Thanks for pointing it out.

[bradwilson]

Oops, I jumped the gun. That PR fixed the usage in xUnit.net, but did not create the analyzer.

[adison88]

@bradwilson what other changes are needed to close this ticket?

[bradwilson]

@bradwilson what other changes are needed to close this ticket?

This issue is to create a new analyzer and fixer.

[etherfield]

@adison88, do you still want to do this? I can help with the analyzer and fixer if you don't mind.

[adison88]

@etherfield go ahead.

[bradwilson]

Available in 1.6.0-pre.8.