Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gate UnsupportedEnum_TestData on Ref.Emit #49925

Merged
merged 1 commit into from
Mar 20, 2021
Merged

Conversation

MichalStrehovsky
Copy link
Member

These types are reflection emitted and will be null without ref emit.

These types are reflection emitted and will be null without ref emit.
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@stephentoub stephentoub merged commit 8f5ea3e into main Mar 20, 2021
@stephentoub stephentoub deleted the MichalStrehovsky-patch-1 branch March 20, 2021 11:10
MichalStrehovsky added a commit to MichalStrehovsky/runtime that referenced this pull request Mar 23, 2021
In dotnet#49925 I blindly followed the existing pattern but the existing pattern was wrong because with that we get failing tests:

```
System.InvalidOperationException : No data found for System.Tests.EnumTests.IsDefined_UnsupportedEnumType_ThrowsInvalidOperationException
```
MichalStrehovsky added a commit that referenced this pull request Mar 23, 2021
In #49925 I blindly followed the existing pattern but the existing pattern was wrong because with that we get failing tests:

```
System.InvalidOperationException : No data found for System.Tests.EnumTests.IsDefined_UnsupportedEnumType_ThrowsInvalidOperationException
```
@ghost ghost locked as resolved and limited conversation to collaborators Apr 19, 2021
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants