Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add system test for the bridge. #282

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

evshary
Copy link
Contributor

@evshary evshary commented Dec 2, 2024

To improve the CI for the bridge, I would like to add some tests to ensure the availability of the bridge for every change.
Now I have a test item for MQTT pub and sub, and add Zenoh one will be the next step.

Signed-off-by: ChenYing Kuo <evshary@gmail.com>
Copy link

github-actions bot commented Dec 2, 2024

PR missing one of the required labels: {'enhancement', 'dependencies', 'new feature', 'breaking-change', 'documentation', 'internal', 'bug'}

@evshary evshary added the internal Changes not included in the changelog label Dec 2, 2024
Signed-off-by: ChenYing Kuo <evshary@gmail.com>
Signed-off-by: ChenYing Kuo <evshary@gmail.com>
@evshary evshary requested a review from JEnoch December 3, 2024 09:58
@evshary
Copy link
Contributor Author

evshary commented Dec 3, 2024

It seems like I need to add some timeout to ensure the test won't run forever...

Signed-off-by: ChenYing Kuo <evshary@gmail.com>
@evshary
Copy link
Contributor Author

evshary commented Dec 4, 2024

Ok, it passes now. We should stop runtime before doing anything that might panic the test.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
internal Changes not included in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant