-
Notifications
You must be signed in to change notification settings - Fork 58
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
Basic common tests? #1
Comments
Sure, what tests were you thinking of? You could send a PR to the skeleton On Friday, 12 August 2016, Syed Irfaq R. ⚡️ [email protected]
Freek Van der Herten |
All the skeleton related methods that are present right now with guzzle example (Since it's commonly used)? Sure, I'll send a PR when I start writing. Thought, I'd get started with a skeleton for that as well..lol 😄 |
Good idea 😄 . I don't think the tests should include a Guzzle example though, since the skeleton should be very clean in my opinion. However, providing some basic test classes with tests for the code that is actually in the skeleton (like the firing of events in the channel class, update: hopefully no longer necessary because of laravel/framework#14804 😄), would be cool. It might also be a good idea to list some repos with tests (using Guzzle for instance as you suggested) that can be used as an example. Also, some basic testing guidelines, to keep the tests in all repos kind of uniform, might be nice addition. |
Can we have some basic common tests in this skeleton to make things easier just like other classes?
Just an idea.
cc: @themsaid @freekmurze @mpociot Thoughts?
The text was updated successfully, but these errors were encountered: