Update xunit #76196
Update xunit #76196
Conversation
dotnet-issue-labeler
bot
added
Area-Infrastructure
untriaged
dotnet-policy-service
bot
added
the
Community
|
@jaredpar The updated versions are not present in the internal feeds. Would you consider adding them and update? My goal here is to catch up early with updates just in case there are new analyzer warnings that need to be addressed and keeping up with newer xunit versions to avoid having a future "large" jump in versions. Smaller jumps are usually a lot easier. Note that when xunit v3 is stable, it may be really good to update to it, and even switching to Microsoft.Testing.Platform instead of VSTest. |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
@Youssef1313 looks like real build ambiguities |
| @@ -65,7 +65,7 @@ | |||
| await Console.Out.WriteLineAsync($"Discovering tests in {testDescriptor}...").ConfigureAwait(false); | |||
|
|
|||
| using var xunit = new XunitFrontController(AppDomainSupport.IfAvailable, assemblyFileName, shadowCopy: false); | |||
| var configuration = ConfigReader.Load(assemblyFileName); | |||
| var configuration = ConfigReader.Load(assemblyFileName, configFileName: null); | |||
There was a problem hiding this comment.
For reference: A new overload was added that also has optional parameter, causing ambiguity.
| @@ -28,7 +29,7 @@ public static TheoryData<LanguageVersion> LangVersions() | |||
| } | |||
|
|
|||
| private sealed class CombinatorialLangVersions() | |||
| : CombinatorialValuesAttribute(LangVersions().Select(d => d.Single()).ToArray()); | |||
| : CombinatorialValuesAttribute(((IEnumerable<object[]>)LangVersions()).Select(d => d.Single()).ToArray()); | |||
There was a problem hiding this comment.
For reference: Previously TheoryData<T> was an IEnumerable<object[]>, but now it's both IEnumerable<object[]> and IEnumerable<T>, causing Linq's Select to not be resolved properly without the cast.
|
Thanks @jaredpar. Let's see now maybe. |
|
@jaredpar Are you interested in fixing https://xunit.net/xunit.analyzers/rules/xUnit2031, or do you think it should be suppressed instead? There is not much violations. |
|
I pushed a fix for now. If you prefer to disable xUnit2031, let me know and I'll revert the changes and disable the analyzer. |
|
I'm fine with a fix here. |
No description provided.