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.

Sign up for GitHub

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

Jump to bottom

Forward non-intercepted methods on class proxies to target #571

Merged
merged 5 commits into from Feb 3, 2021
Merged

Forward non-intercepted methods on class proxies to target #571

merged 5 commits into from Feb 3, 2021

Commits on Feb 3, 2021

  1. Non-proxied methods should forward to target 

    The (failing) tests being added here specify that if one chooses via
`IProxyGenerationHook.ShouldInterceptMethod` not to intercept a method
then invocations of that method on the proxy should be forwarded to
the target object (if present).
    @stakx
    stakx committed Feb 3, 2021
    Copy the full SHA
    56d74dd View commit details
    Browse the repository at this point in the history

  2. Forward non-intercepted methods to class proxy target

    @stakx
    stakx committed Feb 3, 2021
    Copy the full SHA
    41556e9 View commit details
    Browse the repository at this point in the history

  3. hook.ShouldInterceptMethod now broken for non-public methods 

    Using `IndirectlyCalledMethodGenerator` for non-public methods even when
`method.Proxyable == false` is wrong because it will cause interceptors
to be invoked. But if the hook decided that a method shouldn't be inter-
cepted, that is obviously wrong.
    @stakx
    stakx committed Feb 3, 2021
    Copy the full SHA
    aeb19c2 View commit details
    Browse the repository at this point in the history

  4. Skip interceptors for non-intercepted methods

    @stakx
    stakx committed Feb 3, 2021
    Copy the full SHA
    ea5b272 View commit details
    Browse the repository at this point in the history

  5. Update the changelog

    @stakx
    stakx committed Feb 3, 2021
    Copy the full SHA
    77743ac View commit details
    Browse the repository at this point in the history