sitrunlab / LearnZF2

Learn ZF2 Website
http://learnzf2.sitrun-tech.com/
BSD 3-Clause "New" or "Revised" License
19 stars 14 forks source link

using phpspec/prophecy for mocking #138

Closed samsonasik closed 9 years ago

samsonasik commented 9 years ago

see https://samsonasik.wordpress.com/2015/08/29/testing-zend-framework-2-factory-using-prophecy/ . I think it can simplify the tests ;)

acelaya commented 9 years ago

Sure!

samsonasik commented 9 years ago

@acelaya ok, I created #139 for it. The tests that still need to be updated is tests for Application\Module and LearnZF2I18n\Module .

samsonasik commented 9 years ago

prophecy for Application\Module tests has been implemented. now need to implement LearnZF2I18n\Module tests to be prophesized, @acelaya could you help? Thanks ^^

acelaya commented 9 years ago

Yes, I'll try to do it this afternoon ;-)

Alejandro Celaya Alastrué El 1/9/2015 0:34, "Abdul Malik Ikhsan" notifications@github.com escribió:

prophecy for Application\Module tests has been implemented. now need to implement LearnZF2I18n\Module tests to be prophesized, @acelaya https://github.com/acelaya could you help? Thanks ^^

— Reply to this email directly or view it on GitHub https://github.com/sitrunlab/LearnZF2/issues/138#issuecomment-136518694.

ghost commented 9 years ago

Sorry, for the late answer. Been busy with life and the system. Will have a look at this tomorrow.

acelaya commented 9 years ago

I'll porobably won't be finally able to do it until weekend

2015-09-03 21:05 GMT+02:00 Stanimir Dimitrov notifications@github.com:

Sorry, for the late answer. Been busy with life and the system. Will have a look at this tomorrow.

— Reply to this email directly or view it on GitHub https://github.com/sitrunlab/LearnZF2/issues/138#issuecomment-137543832.

Alejandro Celaya Alastrué http://www.alejandrocelaya.com

samsonasik commented 9 years ago

it's ok ;)

acelaya commented 9 years ago

Just opened the pull request to implement this on the i18n module #140