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

API integration tests #7

Open
gavv opened this issue Apr 13, 2020 · 0 comments
Open

API integration tests #7

gavv opened this issue Apr 13, 2020 · 0 comments
Labels
good first issue Good for newcomers help wanted Contributions are welcome tests Improvements or additions to tests

Comments

@gavv
Copy link
Member

gavv commented Apr 13, 2020

We should add at least one integration test in Java that creates a sender, a receiver, transfers samples, and checks that they were transferred correctly.

Such an integration test for the C API can be found here: https://github.com/roc-streaming/roc-toolkit/blob/master/src/tests/public_api/test_sender_receiver.cpp

We can go further and add a few more tests, similar to tests described roc-streaming/roc-toolkit#328. I don't think we need to replicate all of them, but it'd be nice to test at least some of the key features.

See also roc-streaming/roc-go#29 for more hints.

@gavv gavv added the tests Improvements or additions to tests label Apr 13, 2020
@gavv gavv changed the title API integration test API integration tests Apr 13, 2020
@gavv gavv added good first issue Good for newcomers hacktoberfest help wanted Contributions are welcome labels Sep 28, 2020
@gavv gavv removed the hacktoberfest label Dec 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Contributions are welcome tests Improvements or additions to tests
Projects
None yet
Development

No branches or pull requests

1 participant