-
Notifications
You must be signed in to change notification settings - Fork 954
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
API testing #2337
Labels
area:api
Related to celestia-node API
kind:testing
Related to unit tests
swamp
Related to Integration tests
Comments
Wondertan
added
swamp
Related to Integration tests
area:api
Related to celestia-node API
labels
Jun 7, 2023
Related #2240 as we need a way to nicely abstract nodes running over RPC and over direct Go |
Per sync discussion with @distractedm1nd we agree on the following order of execution:
|
distractedm1nd
added a commit
that referenced
this issue
Sep 13, 2023
vgonkivs
pushed a commit
to vgonkivs/celestia-node
that referenced
this issue
Oct 5, 2023
We should consider creating a testing app that imports the Celestia client and runs a test suite over the network. This will enable us to:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area:api
Related to celestia-node API
kind:testing
Related to unit tests
swamp
Related to Integration tests
Currently, we test high-level APIs via Swamp through Go APIs. However:
We should investigate which modules lack API coverage and ensure we test them directly over Go interfaces and over RPC.
The text was updated successfully, but these errors were encountered: