support null for name resolver and more relaxed logging. #35
Annotations
5 warnings
src/PetroGlyph.Games.EawFoc/test/PlayableObjectTest.cs#L56
Non-nullable property 'Game' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
src/PetroGlyph.Games.EawFoc/test/ModServices/CompositeModNameResolverTest.cs#L17
Possible null reference return.
|
src/PetroGlyph.Games.EawFoc/test/GameServices/SteamWorkshopWebpageDownloaderTest.cs#L10
Support for 'async void' unit tests is being removed from xUnit.net v3. To simplify upgrading, convert the test to 'async Task' instead. (https://xunit.net/xunit.analyzers/rules/xUnit1048)
|
src/PetroGlyph.Games.EawFoc/test/FileBasedLanguageFinderTest.cs#L40
Comparing an instance of System.Collections.Generic.List<EawModinfo.Spec.ILanguageInfo> with an instance of System.Collections.Generic.ISet<EawModinfo.Spec.ILanguageInfo> has undefined results, because the order of items in the set is not predictable. Create a stable order for the set (i.e., by using OrderBy from Linq). (https://xunit.net/xunit.analyzers/rules/xUnit2027)
|
src/PetroGlyph.Games.EawFoc/test/GameServices/GameLanguageFinderTest.cs#L63
Do not use Assert.Equal() to check for collection size. Use Assert.Single instead. (https://xunit.net/xunit.analyzers/rules/xUnit2013)
|
This job succeeded
Loading