[stakx]

This is a continuation of #1204, where a dedicated setup type was created for single stubbed properties' accessors (i.e. SetupProperty). Thanks to #1203, the same can also be done for SetupAllProperties.

A few additional notes:

• This gets rid of the RemoveAllPropertyAccessorSetups method in SetupCollection, restoring it to an add-only collection, which in turn means its use of lock-ing could be optimized in a similar fashion as was done for InvocationCollection.

• Gets rid of the AutoSetupPropertiesDefaultValueProvider hack and simplifies the interception pipeline.

• Moq should become a little more efficient in principle, since there will be fewer setups created overall.

Fixes #1066, likely fixes #1231.

[mo-russo]

Hello @stakx, since we got version 4.18.1, our mocks' subclasses that call SetupAllProperties are crashing.

public class MyInterfaceMock : Mock<IMyInterface>
{
  public MyInterfaceMock ()
  {
    SetupAllProperties(); // Crashes here
  }
}

This isn't allowed anymore?

[stakx]

@mo-russo, this is a regression. Could you please report the above again in a new issue, so that we can properly track it? Thanks.