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

Add more tests #23

Open
Giacomo92 opened this issue Oct 7, 2020 · 4 comments · May be fixed by #39
Open

Add more tests #23

Giacomo92 opened this issue Oct 7, 2020 · 4 comments · May be fixed by #39
Labels
enhancement New feature or request Hacktoberfest

Comments

@Giacomo92
Copy link
Member

Mock api response and test all endpoint

@Giacomo92 Giacomo92 added enhancement New feature or request Hacktoberfest labels Oct 7, 2020
@Djuki
Copy link

Djuki commented Oct 10, 2020

@Giacomo92 I would like to participate. I can start on Monday id that is fine with you.

I have a few questions?

  • Do I need API keys to test routes?
  • I noticed that only one API route was called from your main class FattureInCloud and that is 'clienti/lista'. Do you want me to test API routes using the Auth class only?

@Giacomo92
Copy link
Member Author

Hi @Djuki ,
yes you need API keys in order to create all the stub responses. You can go here and create a trial account.
You can find here all the info about the endpoints.
You have to authenticate first and the make call ti the FC API.

@Djuki
Copy link

Djuki commented Oct 12, 2020

@Giacomo92 Thanks.
I should work on the master, version compatible with Laravel 8? If yes how to install it into the laravel? The latest tag is 1.8 compatible with Laravel 7.0.

@Giacomo92
Copy link
Member Author

I have just released a new version. Use 2.0 with L8 support please

@Giacomo92 Giacomo92 linked a pull request Sep 16, 2021 that will close this issue
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Hacktoberfest
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants