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

Basic common tests? #1

Open
irazasyed opened this issue Aug 12, 2016 · 3 comments
Open

Basic common tests? #1

irazasyed opened this issue Aug 12, 2016 · 3 comments

Comments

@irazasyed
Copy link
Member

irazasyed commented Aug 12, 2016

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?

@freekmurze
Copy link
Member

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]
wrote:

Can we some basic common tests in this skeleton to make things easier just
like other classes?

Just an idea.

cc: @themsaid https://github.com/themsaid @freekmurze
https://github.com/freekmurze @mpociot https://github.com/mpociot
Thoughts?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#1, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAdiDdBAdAQaRKRax38D4L7OXyZPz2hoks5qfKrOgaJpZM4JjSMl
.

Freek Van der Herten
https://spatie.be
+32 495 84 27 91

@irazasyed
Copy link
Member Author

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 😄

@casperboone
Copy link
Member

casperboone commented Aug 13, 2016

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.

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

3 participants