Unnecessary internal/swallowed MissingMemberException in DeepCloneHelper

[Andrey Shchekin]

Hi all,

I have an integration tests scenario where setup calls GetAllTypeDefinitions() many times.

I'm going to improve that, however I found one Umbraco part that might be worth improving.

Specifically this: https://github.com/umbraco/Umbraco-CMS/blob/d50e49ad37fd5ca7bad2fd6e8fc994f3408ae70c/src/Umbraco.Core/Models/DeepCloneHelper.cs#L84.

For properties that are IDictionary it throws MissingMemberException, and that happens a lot in my tests. 

It makes the debugging slower and interferes with "Break on All Exceptions".

I have one definite change and one potential change I want to make there:

1. (Definite) Check that type is an interface/abstract class and skip attempt to create an instance
2. (Potenial) Handle Dictionaries.

What do you think about it?

Cheers,

Andrey