search

philiplb / CRUDlex

CRUDlex is an easy to use CRUD generator for Symfony 4 and Silex 2 which is great for auto generated admin pages
https://philiplb.de/crudlex/
MIT License
109 stars 23 forks source link

Bug in Phony usage #59

Closed ezzatron closed 8 years ago

ezzatron commented 8 years ago

Hi there, I noticed you have a bug in the way you're using Phony. Lines like this one:

$this->fileProcessorHandle->updateFile->never();

Should probably be:

$this->fileProcessorHandle->updateFile->never()->called();

As calling never() by itself does nothing. See Verifying that a spy event happened an exact number of times.

philiplb commented 8 years ago

Now that's what I call service, thank you for noting. :) Fixed in 272c55c

ezzatron commented 8 years ago

No problem, it's fun to see people really using it in the wild :)