Delete `VerifyExpectModifiedAttributesWork`. This test has bit rotted since being...
authorMike Voorhees <mrvoorhe@users.noreply.github.com>
Mon, 17 Jul 2023 19:41:20 +0000 (15:41 -0400)
committerGitHub <noreply@github.com>
Mon, 17 Jul 2023 19:41:20 +0000 (12:41 -0700)
commit85715a289692c6637ea095d260d976e8e87f264d
tree4aa9ce41e9aab26cb12838c8165965319b2ea38e
parent670c1c8c80ab4a4a48dc82756f26340262310938
Delete `VerifyExpectModifiedAttributesWork`.  This test has bit rotted since being made netfx only and now fails due to etw feature removal logic no longer existing. (#89017)

This test dates back to earlier days when the linker had few mechanisms that modified bodies and UnityLinker had many.  I added this test to make sure that test framework support for this attribute didn't break since we depended on it.  Now days, the linker has many mechanisms that modify bodies and `[ExpectBodyModified]` is heavily used in tests.  I don't think there is any value in updating this test to use a different code pattern that results in needing `[ExpectBodyModified]`.
src/tools/illink/test/ILLink.RoslynAnalyzer.Tests/generated/ILLink.RoslynAnalyzer.Tests.Generator/ILLink.RoslynAnalyzer.Tests.TestCaseGenerator/TestFrameworkTests.g.cs
src/tools/illink/test/Mono.Linker.Tests.Cases/TestFramework/VerifyExpectModifiedAttributesWork.cs [deleted file]