Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Add test for GetCurrentMethod called inside inlineable method#16071

Merged
jkotas merged 1 commit into
dotnet:masterfrom
jkotas:inlineable-getcurrentmethod
Feb 11, 2017
Merged

Add test for GetCurrentMethod called inside inlineable method#16071
jkotas merged 1 commit into
dotnet:masterfrom
jkotas:inlineable-getcurrentmethod

Conversation

@jkotas
Copy link
Copy Markdown
Member

@jkotas jkotas commented Feb 11, 2017

No description provided.

@jkotas
Copy link
Copy Markdown
Member Author

jkotas commented Feb 11, 2017

This is targeted test for the bug fixed by dotnet/coreclr#9537

cc @stephentoub @gkhanna79

Note that this test is not running by default because of https://github.com/dotnet/corefx/issues/15849

Assert.True(m.IsStatic);
Assert.True(m.IsPublic);
Assert.True(m.DeclaringType == typeof(MethodBaseTests));
throw new Exception();
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are we throwing an exception here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left over from me verifying whether the test runs or not...

@jkotas jkotas force-pushed the inlineable-getcurrentmethod branch 2 times, most recently from e691e5b to 837fca0 Compare February 11, 2017 22:02
@jkotas jkotas force-pushed the inlineable-getcurrentmethod branch from 837fca0 to a38b83b Compare February 11, 2017 22:03
@jkotas jkotas merged commit 1509797 into dotnet:master Feb 11, 2017
@karelz karelz modified the milestone: 2.0.0 Feb 13, 2017
@jkotas jkotas deleted the inlineable-getcurrentmethod branch February 13, 2017 22:04
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
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