search

atoum / visibility-extension

The atoum visibility-extension allows you to override method visibility in your unit tests.
Other
4 stars 6 forks source link

Make class/mock properties visible #27

Open Oliboy50 opened 7 years ago

Oliboy50 commented 7 years ago

Why

The documentation only talks about method visibility... but nothing about property.

Question

Is it possible to make class/mock properties visible? And howto (maybe update the documentation accordingly).

Grummfy commented 7 years ago

I came back on this. I know that @Hywan & @jubianchi have been working on something new for the mocking system so probably some change will come

Hywan commented 7 years ago

Actually, this is not a bad idea to change the visibility of the properties. Thoughts @jubianchi?

@Oliboy50 do you want to try a PR?

jubianchi commented 7 years ago

@Hywan mocks in atoum have never been designer to work with properties: they are all about behaviors/services but not about state.

I'm not really in favor of this but if the community asks for the feature and it can be implemented without too much effort, why not.

To be fair, I won't have much time to spend on this issue, sorry.