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

tests should not depend on external apis being up #421

Closed
nicolasochem opened this issue Apr 20, 2021 · 3 comments · Fixed by #501
Closed

tests should not depend on external apis being up #421

nicolasochem opened this issue Apr 20, 2021 · 3 comments · Fixed by #501
Assignees
Labels
enhancement New feature or request

Comments

@nicolasochem
Copy link
Contributor

CI tests appear broken now. Tests should run in isolation IMO. Have you considered recording and replaying API calls from external RPC services (RPC, tzstats, tzkt), for example like this:

https://www.mock-server.com/proxy/record_and_replay.html

@nicolasochem nicolasochem added the enhancement New feature or request label Apr 20, 2021
@nicolasochem
Copy link
Contributor Author

@vkresch
Copy link
Contributor

vkresch commented Apr 20, 2021

@nicolasochem thanks for reporting. I created a PR as a hotfix. The reason for the current error is the recent removal of the https://api.carthage.tzkt.io/v1/ domain name.

IMO it's always good to test the trd functions against the real API so that the regression tests are always up to date and detect compatibility issues between trd and the API on which it depends. When a test fails it means either the test or the function which it tests needs to be fixed which is a good process for the improvement of trd. Imagine for example the tzkt API gets removed and the trd tests report only passed tests. Let's hear other opinions, too.

@vkresch vkresch self-assigned this Apr 20, 2021
This was referenced Apr 20, 2021
@jdsika
Copy link
Contributor

jdsika commented Apr 20, 2021

Using recorded api calls is on my wish list for over 1 year now - Viktor has just started and will go towards this task! Important, thank you for also noticing :)

@jdsika jdsika added this to the v9.5 (Florence) milestone May 12, 2021
@jdsika jdsika linked a pull request Oct 8, 2021 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment