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 end to end tests #516

Open
jacobtomlinson opened this issue Apr 4, 2018 · 7 comments
Open

Add end to end tests #516

jacobtomlinson opened this issue Apr 4, 2018 · 7 comments

Comments

@jacobtomlinson
Copy link
Member

As well as the existing unit tests it would be good to add some end to end tests.

Some of this has already been added in #319 with the docker tests which checks the api endpoint as well as running the unit tests.

Good things to test would be:

  • The API becomes available within a certain time (10 seconds?).
  • Using the webhook connector and the hello skill a message can be sent and responsed to.
  • It is possible to use a database module and skill which uses it.
@sims34
Copy link
Contributor

sims34 commented Apr 27, 2018

Hi @jacobtomlinson !
I am student in CS and i would like contribute, but i'm new in the open source system how can i help, where i should start ?
Thk :)

@jacobtomlinson
Copy link
Member Author

Hey @sims34. This issue probably isn't the best place to start. Why don't you hop into our gitter channel and we can chat about it there.

@sims34
Copy link
Contributor

sims34 commented Apr 27, 2018

Hey,
ok thk

@sims34
Copy link
Contributor

sims34 commented Apr 29, 2018

Hi @jacobtomlinson
i sent you a message on gitter channel.

@jacobtomlinson
Copy link
Member Author

This is up for grabs again

@gsunit
Copy link

gsunit commented Jan 21, 2019

Hey, @jacobtomlinson
I would like to help by writing the tests. However, I am new to writing tests and would need your help.
Starting with the first test:

The API becomes available within a certain time (10 seconds?).

Which APIs will be concerned here? Please direct me to the relevant files.

@jacobtomlinson
Copy link
Member Author

Great to have you @gsunit. I highly recommend you take a look at the project documentation. The API referred to is covered in the documentation here.

The tests covered in this issue are for externally testing opsdroid, rather than unit tests. If you're new to testing and are interested in unit tests I would recommend looking at existing tests and improving them.

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

No branches or pull requests

4 participants