[hot_reload] Enable some CustomAttributes tests on Mono; new issue for async (#79044)

lambdageek · web-flow · commit 067e43a1363f · 2022-11-30T22:05:47.000-05:00
The async test was marked with the CustomAttributes tracking issue, but it's actually unrelated. Fixes #52993
diff --git a/src/libraries/System.Runtime.Loader/tests/ApplyUpdateTest.cs b/src/libraries/System.Runtime.Loader/tests/ApplyUpdateTest.cs
@@ -119,7 +119,6 @@ void FirstCallAfterUpdate()
         }
 
         [ConditionalFact(typeof(ApplyUpdateUtil), nameof (ApplyUpdateUtil.IsSupported))]
-        [ActiveIssue("https://github.com/dotnet/runtime/issues/52993", TestRuntimes.Mono)]
         void ClassWithCustomAttributes()
         {
             ApplyUpdateUtil.TestCase(static () =>
@@ -191,7 +190,6 @@ public void CustomAttributeUpdates()
             });
         }
 
-        [ActiveIssue("https://github.com/dotnet/runtime/issues/52993", TestRuntimes.Mono)]
         [ConditionalFact(typeof(ApplyUpdateUtil), nameof (ApplyUpdateUtil.IsSupported))]
         public void CustomAttributeDelete()
         {
@@ -231,7 +229,7 @@ public void CustomAttributeDelete()
             });
         }
 
-        [ActiveIssue("https://github.com/dotnet/runtime/issues/52993", TestRuntimes.Mono)]
+        [ActiveIssue("https://github.com/dotnet/runtime/issues/79043", TestRuntimes.Mono)]
         [ConditionalFact(typeof(ApplyUpdateUtil), nameof (ApplyUpdateUtil.IsSupported))]
         public void AsyncMethodChanges()
         {

Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,6 @@ void FirstCallAfterUpdate()`
`119`	`119`	`}`
`120`	`120`
`121`	`121`	`[ConditionalFact(typeof(ApplyUpdateUtil), nameof (ApplyUpdateUtil.IsSupported))]`
`122`		`- [ActiveIssue("https://github.com/dotnet/runtime/issues/52993", TestRuntimes.Mono)]`
`123`	`122`	`void ClassWithCustomAttributes()`
`124`	`123`	`{`
`125`	`124`	`ApplyUpdateUtil.TestCase(static () =>`
`@@ -191,7 +190,6 @@ public void CustomAttributeUpdates()`
`191`	`190`	`});`
`192`	`191`	`}`
`193`	`192`
`194`		`- [ActiveIssue("https://github.com/dotnet/runtime/issues/52993", TestRuntimes.Mono)]`
`195`	`193`	`[ConditionalFact(typeof(ApplyUpdateUtil), nameof (ApplyUpdateUtil.IsSupported))]`
`196`	`194`	`public void CustomAttributeDelete()`
`197`	`195`	`{`
`@@ -231,7 +229,7 @@ public void CustomAttributeDelete()`
`231`	`229`	`});`
`232`	`230`	`}`
`233`	`231`
`234`		`- [ActiveIssue("https://github.com/dotnet/runtime/issues/52993", TestRuntimes.Mono)]`
	`232`	`+ [ActiveIssue("https://github.com/dotnet/runtime/issues/79043", TestRuntimes.Mono)]`
`235`	`233`	`[ConditionalFact(typeof(ApplyUpdateUtil), nameof (ApplyUpdateUtil.IsSupported))]`
`236`	`234`	`public void AsyncMethodChanges()`
`237`	`235`	`{`