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

Code completition #45

Open
EddieOfAlhana opened this issue Oct 21, 2013 · 1 comment
Open

Code completition #45

EddieOfAlhana opened this issue Oct 21, 2013 · 1 comment

Comments

@EddieOfAlhana
Copy link

There is a large drawback of the facade pattern. As with Laravel in general ide code completition does not work with laravel test helpers.
Would you think it would be a best practice to add @method (http://www.phpdoc.org/docs/latest/for-users/phpdoc/tags/method.html) annotation tags to Should.php, Assert.php aso?
Or would it be better to create something like barryvdh/laravel-ide-helper to generate a helper file for laravel-test-helpers dinamically in case some new methods are introduced in phpunit?

@CreepGin
Copy link

If you add the facade aliases to app.php, wouldn't barryvdh/laravel-ide-helper be able to generate the proper classes for auto-completion?

Edit: 2nd thought, probably not... Should and Assert do not seem to be extending the laravel Facade class

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

No branches or pull requests

2 participants