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

Prevent auto-stubbing failure due to inaccessible property accessors #847

Merged
merged 3 commits into from
Jun 6, 2019
Merged

Prevent auto-stubbing failure due to inaccessible property accessors #847

merged 3 commits into from
Jun 6, 2019

Commits on Jun 6, 2019

  1. Add regression tests for lazy SetupAllProperties 

    There are two groups of tests here:

 * Those that test `SetupAllProperties` (called either directly, or
   implicitly through `Mock.Of<T>`) followed by a property acces. This
   reflects `SetupAllProperties`' current implementation, which only
   sets up properties upon their first access (just-in-time).

 * Those that test `SetupAllProperties` itself, without any property
   access. Unsurprisingly, these currently pass, but the tests are
   still good to have in case we ever return `SetupAllProperties` to a
   ahead-of-time batch operation implementation.
    @stakx
    stakx committed Jun 6, 2019
    Configuration menu
    Copy the full SHA
    027985f View commit details
    Browse the repository at this point in the history

  2. Fix regression by checking accessor visibility

    @stakx
    stakx committed Jun 6, 2019
    Configuration menu
    Copy the full SHA
    cf32e84 View commit details
    Browse the repository at this point in the history

  3. Update the changelog

    @stakx
    stakx committed Jun 6, 2019
    Configuration menu
    Copy the full SHA
    b15051d View commit details
    Browse the repository at this point in the history