roc-streaming / roc-go

Golang bindings for Roc Toolkit.
https://roc-streaming.org
MIT License
22 stars 10 forks source link

Improve integration tests for sender and receiver #29

Closed gavv closed 1 year ago

gavv commented 4 years ago

Currently we have one simple test (sender_receiver_test.go added in #24), which runs sender in receiver in two goroutines and waits until receiver will get a few non-zero samples.

It would be nice to add a few more sophisticated tests. We don't need to test all possible combinations in bindings, since we assume that the underlying library is already tested; however covering a few important features and checking that everything works as expected would be helpful.

Things to cover:

For background, see the following links:

rafiramadhana commented 1 year ago

Hi @gavv . This issue seems interesting.

However, do you have any recommendation of docs/videos/etc to familiarize myself with this repo? (e.g. basic concepts, common glossary, api/code structure, etc)

Thanks.

gavv commented 1 year ago

@rafiramadhana Hi!

Since roc-go provides bindings for roc-toolkit, I guess the best place to start is docs for roc-toolkit itself: https://roc-streaming.org/toolkit/docs/

In particular, the following pages may be useful:

Also, information from this issue may help: https://github.com/roc-streaming/rt-tests/issues/1

Feel free to ask question here on in chat.

Let me know if you would like to be assigned.

rafiramadhana commented 1 year ago

@gavv I'm going to spent time for reading first.

I will let you know when I'm ready.

Thanks!

gavv commented 1 year ago

Given it another though, I decided this is actually too big for one task. I've created separate issue for each enhancement and provided more details:

gavv commented 1 year ago

@rafiramadhana Feel free to leave a comment in those issues if you would like to pick up one.