Add attribute telling the debugger not to evaluate destructive properties

[manfred-brands]

Fixes #4994
Fixes #4995

[stevenaw]

Thanks for fixing this @manfred-brands
This looks good to me. Just for the sake of considering all sides, do we think there's a chance anyone was reliant on our override of ToString() ?

EDIT: I agree the existing implementation can be problematic. I'm thinking instead about if it's worth a dedicated call-out in any release notes about it

@OsirisTerje Any concerns about this going into 4.4?

[manfred-brands]

do we think there's a chance anyone was reliant on our override of ToString() ?

The constraint doesn't work correctly anymore after calling that method, so I would say not.

[stevenaw]

Thanks @manfred-brands
I agree it's pretty unlikely outside perhaps the testing of custom constraints, similar to the usage in NUnit's suite.