Skip to content

Commit

Permalink
Enable some more tests on Mono based on dotnet#34582
Browse files Browse the repository at this point in the history
  • Loading branch information
kouvel committed Mar 3, 2022
1 parent 7c0d7d5 commit 759e11a
Show file tree
Hide file tree
Showing 36 changed files with 4 additions and 108 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ private static bool GetLinqExpressionsBuiltWithIsInterpretingOnly()
// Drawing is not supported on non windows platforms in .NET 7.0+.
public static bool IsDrawingSupported => IsWindows && IsNotWindowsNanoServer && IsNotWindowsServerCore;

public static bool IsAsyncFileIOSupported => !IsBrowser && !(IsWindows && IsMonoRuntime); // https://github.com/dotnet/runtime/issues/34582
public static bool IsAsyncFileIOSupported => !IsBrowser;

public static bool IsLineNumbersSupported => !IsNativeAot;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ private void SetSecret(string id, string key, string value)
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
[ActiveIssue("https://github.com/dotnet/runtime/issues/60584", TestPlatforms.iOS | TestPlatforms.tvOS)]
public void AddUserSecrets_FindsAssemblyAttribute()
{
Expand All @@ -68,7 +67,6 @@ public void AddUserSecrets_FindsAssemblyAttribute()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
[ActiveIssue("https://github.com/dotnet/runtime/issues/60584", TestPlatforms.iOS | TestPlatforms.tvOS)]
public void AddUserSecrets_FindsAssemblyAttributeFromType()
{
Expand Down Expand Up @@ -123,7 +121,6 @@ public void AddUserSecrets_DoesThrowsIfNotOptionalAndSecretDoesNotExist()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
[ActiveIssue("https://github.com/dotnet/runtime/issues/60584", TestPlatforms.iOS | TestPlatforms.tvOS)]
public void AddUserSecrets_With_SecretsId_Passed_Explicitly()
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,6 @@ public class ArrayTests : IDisposable
";

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
[ActiveIssue("https://github.com/dotnet/runtime/issues/60583", TestPlatforms.iOS | TestPlatforms.tvOS)]
public void DifferentConfigSources_Merged_KeysAreSorted()
{
Expand Down Expand Up @@ -77,7 +76,6 @@ public void DifferentConfigSources_Merged_KeysAreSorted()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
[ActiveIssue("https://github.com/dotnet/runtime/issues/60583", TestPlatforms.iOS | TestPlatforms.tvOS)]
public void DifferentConfigSources_Merged_WithOverwrites()
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,6 @@ public void MissingFileDoesNotIncludesAbsolutePathIfWithNoPhysicalPath()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
public void LoadAndCombineKeyValuePairsFromDifferentConfigurationProviders()
{
WriteTestFiles();
Expand Down Expand Up @@ -247,7 +246,6 @@ public void LoadAndCombineKeyValuePairsFromDifferentConfigurationProviders()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
public void LoadAndCombineKeyValuePairsFromDifferentConfigurationProvidersWithAbsolutePath()
{
WriteTestFiles();
Expand Down Expand Up @@ -288,7 +286,6 @@ public void LoadAndCombineKeyValuePairsFromDifferentConfigurationProvidersWithAb
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
public void CanOverrideValuesWithNewConfigurationProvider()
{
WriteTestFiles();
Expand Down Expand Up @@ -361,7 +358,6 @@ public IConfigurationProvider Build(IConfigurationBuilder builder)
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
[ActiveIssue("https://github.com/dotnet/runtime/issues/60583", TestPlatforms.iOS | TestPlatforms.tvOS)]
public void OnLoadErrorWillBeCalledOnJsonParseError()
{
Expand Down Expand Up @@ -391,7 +387,6 @@ public void OnLoadErrorWillBeCalledOnJsonParseError()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
public void OnLoadErrorWillBeCalledOnXmlParseError()
{
_fileSystem.WriteFile("error.xml", @"gobblygook");
Expand Down Expand Up @@ -419,7 +414,6 @@ public void OnLoadErrorWillBeCalledOnXmlParseError()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
public void OnLoadErrorWillBeCalledOnIniLoadError()
{
_fileSystem.WriteFile("error.ini", @"IniKey1=IniValue1
Expand Down Expand Up @@ -448,7 +442,6 @@ public void OnLoadErrorWillBeCalledOnIniLoadError()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
public void OnLoadErrorCanIgnoreErrors()
{
_fileSystem.WriteFile("error.json", @"{""JsonKey1"": ");
Expand Down Expand Up @@ -821,7 +814,6 @@ await WaitForChange(
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
[ActiveIssue("https://github.com/dotnet/runtime/issues/50866", TestPlatforms.Android)]
public void LoadIncorrectJsonFile_ThrowException()
{
Expand All @@ -839,7 +831,6 @@ public void LoadIncorrectJsonFile_ThrowException()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
public void SetBasePathCalledMultipleTimesForEachSourceLastOneWins()
{
var builder = new ConfigurationBuilder();
Expand Down Expand Up @@ -873,7 +864,6 @@ public void SetBasePathCalledMultipleTimesForEachSourceLastOneWins()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
[ActiveIssue("https://github.com/dotnet/runtime/issues/60583", TestPlatforms.iOS | TestPlatforms.tvOS)]
public void GetDefaultBasePathForSources()
{
Expand Down Expand Up @@ -904,7 +894,6 @@ public void GetDefaultBasePathForSources()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
[SkipOnPlatform(TestPlatforms.Browser | TestPlatforms.iOS | TestPlatforms.tvOS, "System.IO.FileSystem.Watcher is not supported on Browser/iOS/tvOS")]
public void CanEnumerateProviders()
{
Expand Down Expand Up @@ -952,7 +941,6 @@ await WaitForChange(
}

[ConditionalFact(typeof(PlatformDetection), nameof(PlatformDetection.IsThreadingSupported))]
[ActiveIssue("https://github.com/dotnet/runtime/issues/34582", TestPlatforms.Windows, TargetFrameworkMonikers.Netcoreapp, TestRuntimes.Mono)]
public void BindingDoesNotThrowIfReloadedDuringBinding()
{
WriteTestFiles();
Expand Down
Loading

0 comments on commit 759e11a

Please sign in to comment.