Skip to content
This repository has been archived by the owner on Feb 1, 2022. It is now read-only.

Remove uneeded Zimple requirement #14

Closed
frastel opened this issue May 18, 2013 · 2 comments
Closed

Remove uneeded Zimple requirement #14

frastel opened this issue May 18, 2013 · 2 comments

Comments

@frastel
Copy link
Contributor

frastel commented May 18, 2013

As far as i have seen the requirement jtreminio/zimple is never used in the project, so it could be removed. Is this correct or did I miss something?

@jtreminio
Copy link
Member

Zimple will be used for dependency management to make my unit tests simpler. Right now I am unable to properly unit test controllers because of hard dependencies introduced by the new keyword - Zimple will help with this.

@frastel
Copy link
Contributor Author

frastel commented May 18, 2013

Mmmh, after I have finished the nginx integration I could move the logic from the createAction into an own service and add an according test for this service. This way you would see that you do not need a controller test (functional test) for everything.

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

No branches or pull requests

2 participants