search

fmalk / codeigniter-phpunit

Hack to make CodeIgniter work with PHPUnit.
234 stars 61 forks source link

Autoloading all controllers due to #8 #9

Closed feryardiant closed 10 years ago

feryardiant commented 10 years ago

I hope it fix #8

Check my feryardiant@e7fc26b5 for more detail

fmalk commented 10 years ago

@feryardiant I merged it because it did solve the issue. However, requiring everything at once would increase memory usage, and could break test run in environments such as shared hosts. So I improved it in 2c9f644394d6fbc3d7ca3f4cc5310b26b7e7f8c6 to use spl_autoload_register and include files as needed. It eventually became v1.4.1 (after passing a Travis CI run). Thanks for the great input as always!

feryardiant commented 10 years ago

Ah! you right. sorry about that, I forget about spl_autoload_register.

Glad it helps.