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.

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

Make class/mock properties visible #27

Open
Oliboy50 opened this issue Oct 26, 2016 · 3 comments
Open

Make class/mock properties visible #27

Oliboy50 opened this issue Oct 26, 2016 · 3 comments
Assignees

Comments

@Oliboy50
Copy link

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
Copy link
Member

Grummfy commented May 21, 2017

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
Copy link
Member

Hywan commented May 22, 2017

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

@Oliboy50 do you want to try a PR?

@Hywan Hywan self-assigned this May 22, 2017
@jubianchi
Copy link
Member

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants