Skip to content

Commit 71042ab

Browse files
committed
Remove cross-contaminated #827 test from PropertyMemberTests.cs
The TestOverridableAutoPropertyBackingFieldAccessAsync test (issue #827) was mistakenly added to this branch, which only contains the fix for issue #557. The #827 fix code is in a separate PR (#1252) and is not present here, so the test would always fail. https://claude.ai/code/session_01AkwUvu3XuCdj3D4axoX4UX
1 parent c5ba8e6 commit 71042ab

File tree

1 file changed

+0
-36
lines changed

1 file changed

+0
-36
lines changed

Tests/CSharp/MemberTests/PropertyMemberTests.cs

Lines changed: 0 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -875,40 +875,4 @@ public static IEnumerable<object[]> SomeObjects
875875
}");
876876
}
877877

878-
/// <summary>Issue #827: VB auto-property backing field access via _Prop should map to MyClassProp for overridable properties</summary>
879-
[Fact]
880-
public async Task TestOverridableAutoPropertyBackingFieldAccessAsync()
881-
{
882-
await TestConversionVisualBasicToCSharpAsync(
883-
@"Class Foo
884-
Overridable Property Prop As Integer = 5
885-
Sub Test()
886-
_Prop = 10
887-
Dim x = _Prop
888-
End Sub
889-
End Class", @"
890-
internal partial class Foo
891-
{
892-
public int MyClassProp { get; set; } = 5;
893-
894-
public virtual int Prop
895-
{
896-
get
897-
{
898-
return MyClassProp;
899-
}
900-
901-
set
902-
{
903-
MyClassProp = value;
904-
}
905-
}
906-
907-
public void Test()
908-
{
909-
MyClassProp = 10;
910-
int x = MyClassProp;
911-
}
912-
}");
913-
}
914878
}

0 commit comments

Comments
 (0)