Skip to content

Commit

Permalink
Put failing tests behind ActiveIssue (dotnet#852)
Browse files Browse the repository at this point in the history
  • Loading branch information
MichalStrehovsky authored Mar 25, 2021
1 parent 4564ad6 commit 09d75d0
Show file tree
Hide file tree
Showing 8 changed files with 18 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -222,7 +222,7 @@ public static string[] GetNames(Type enumType)
return enumType.GetEnumNames();

if (!enumType.IsEnum)
throw new ArgumentException(SR.Arg_MustBeEnum);
throw new ArgumentException(SR.Arg_MustBeEnum, nameof(enumType));

string[] ret = GetEnumInfo(enumType).Names;

Expand Down
2 changes: 2 additions & 0 deletions src/libraries/System.Runtime/tests/System/AttributeTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,8 @@ class ChildAttributeWithField : ParentAttribute
public int Field = 0;
}


[ActiveIssue("https://github.com/dotnet/runtimelab/issues/803" /* NativeAot */)]
[Fact]
[StringValue("\uDFFF")]
public static void StringArgument_InvalidCodeUnits_FallbackUsed()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ public void Ctor()


[Fact]
[ActiveIssue("https://github.com/dotnet/runtimelab/issues/851" /* NativeAot */)]
public void MethodHidden_ExceptionStackTrace()
{
string stacktrace = null;
Expand Down Expand Up @@ -49,6 +50,7 @@ public void MethodHidden_ExceptionStackTrace()


[Fact]
[ActiveIssue("https://github.com/dotnet/runtimelab/issues/851" /* NativeAot */)]
public void MethodHidden_EnvironmentStackTrace()
{
string stacktrace = GetStackTraceMethodA();
Expand All @@ -69,6 +71,7 @@ public void MethodHidden_EnvironmentStackTrace()


[Fact]
[ActiveIssue("https://github.com/dotnet/runtimelab/issues/851" /* NativeAot */)]
public void ConstructorHidden_EnvironmentStackTrace()
{
Assert.Contains(nameof(NotHiddenConstructor), new NotHiddenConstructor().StackTrace);
Expand All @@ -94,6 +97,7 @@ private class HiddenConstructor


[Fact]
[ActiveIssue("https://github.com/dotnet/runtimelab/issues/851" /* NativeAot */)]
public void ClassHidden_EnvironmentStackTrace()
{
string stacktrace = HiddenClass.GetStackTraceMethodA();
Expand All @@ -117,6 +121,7 @@ internal class HiddenClass


[Fact]
[ActiveIssue("https://github.com/dotnet/runtimelab/issues/851" /* NativeAot */)]
public void StructHidden_EnvironmentStackTrace()
{
string stacktrace = new HiddenStruct().GetStackTraceMethodA();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ namespace System.Tests
{
public static partial class PseudoCustomAttributeTests
{
[ActiveIssue("https://github.com/dotnet/runtimelab/issues/830" /* NativeAot */)]
[Theory]
[MemberData(nameof(TestData_AttributeExists))]
[MemberData(nameof(TestData_AttributeDoesNotExist))]
Expand All @@ -38,6 +39,7 @@ public static void IsDefined(object target, Type attributeType, Attribute attrib
}
}

[ActiveIssue("https://github.com/dotnet/runtimelab/issues/830" /* NativeAot */)]
[Theory]
[MemberData(nameof(TestData_AttributeExists))]
[MemberData(nameof(TestData_AttributeDoesNotExist))]
Expand All @@ -63,6 +65,7 @@ public static void GetCustomAttribute(object target, Type attributeType, Attribu
}
}

[ActiveIssue("https://github.com/dotnet/runtimelab/issues/830" /* NativeAot */)]
[Theory]
[MemberData(nameof(TestData_AttributeExists))]
[MemberData(nameof(TestData_AttributeDoesNotExist))]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ public static void MethodReturnsRefToRefStruct_ThrowsNSE()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtimelab/issues/155" /* NativeAot */)]
public static void MethodTakesRefStructAsArg_DoesNotCopyValueBack()
{
MethodInfo mi = GetMethod(nameof(TestClass.TakesRefStructAsArg));
Expand All @@ -28,6 +29,7 @@ public static void MethodTakesRefStructAsArg_DoesNotCopyValueBack()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtimelab/issues/155" /* NativeAot */)]
public static void MethodTakesRefStructAsArgWithDefaultValue_DoesNotCopyValueBack()
{
MethodInfo mi = GetMethod(nameof(TestClass.TakesRefStructAsArgWithDefaultValue));
Expand Down Expand Up @@ -77,6 +79,7 @@ public static void PropertyTypedAsRefToRefStruct_AsPropInfo_ThrowsNSE()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtimelab/issues/155" /* NativeAot */)]
public static void PropertyIndexerWithRefStructArg_DoesNotCopyValueBack()
{
PropertyInfo pi = typeof(TestClassWithIndexerWithRefStructArg).GetProperty("Item");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -273,6 +273,7 @@ static ClassWithNormalCctor()
}
}

[ActiveIssue("https://github.com/dotnet/runtimelab/issues/155" /* NativeAot */)]
[Fact]
public static void GetUninitalizedObject_DoesNotRunBeforeFieldInitCctors()
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ public static void StaticThrow_UpdatesStackTraceAppropriately()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtimelab/issues/831" /* NativeAot */)]
public static void SetCurrentStackTrace_Invalid_Throws()
{
Exception e;
Expand All @@ -49,6 +50,7 @@ public static void SetCurrentStackTrace_Invalid_Throws()
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtimelab/issues/831" /* NativeAot */)]
public static void SetCurrentStackTrace_IncludedInExceptionStackTrace()
{
Exception e;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -527,6 +527,7 @@ public void GetTypeByName_Invalid(string typeName, Type expectedException, bool
}

[Fact]
[ActiveIssue("https://github.com/dotnet/runtimelab/issues/155" /* NativeAot */)]
public void GetTypeByName_InvokeViaReflection_Success()
{
MethodInfo method = typeof(Type).GetMethod("GetType", new[] { typeof(string) });
Expand Down

0 comments on commit 09d75d0

Please sign in to comment.