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

Feature: allow interception of any generic method call when using Arg.AnyType #855

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JMolenkamp
Copy link

Allow for interception of any call to a generic method, regardless of used generic argument.
Related to issue 854

@dtchepak
Copy link
Member

Hi @JMolenkamp ,

Thanks a lot for this. Could you please add tests to this PR to demonstrate the change?

@JMolenkamp
Copy link
Author

JMolenkamp commented Jan 28, 2025

Added some tests. The first two fail without the change. The last one does not build without the change.

Maybe this PR needs to be split into two. Part of it is a fix for the first two tests, which should not fail, I think?
The other part is adding a MethodInfo property to CallInfo, allowing for something like the third test, which is similar to the use case that led me to make this PR.

@JMolenkamp
Copy link
Author

Moved the fixes from this PR to #862. This PR now only contains the feature request.
It is dependent upon #862 though, the test for demonstration purposes will fail without it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants