Skip to content

Add end-to-end test for sending and receiving a test file - #55

Merged
dsilhavy merged 1 commit into
developmentfrom
feature/endToEndTest
Mar 9, 2026
Merged

Add end-to-end test for sending and receiving a test file#55
dsilhavy merged 1 commit into
developmentfrom
feature/endToEndTest

Conversation

@dsilhavy

@dsilhavy dsilhavy commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

This PR adds an end to end test for sending and receiving a sample file. The test is integrated into GitHub CI/CD pipeline via a new GitHub action.

@dsilhavy dsilhavy added this to the 0.12.2 milestone Mar 9, 2026
@dsilhavy
dsilhavy merged commit a51a55a into development Mar 9, 2026
2 checks passed
@dsilhavy
dsilhavy deleted the feature/endToEndTest branch March 9, 2026 15:09

@rjb1000 rjb1000 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't have time to look in detail, @dsilhavy, but is your end-to-end test exercising the tunnelled mode of operation between transmitter and receiver as well as the vanilla mode?

It was specifically the tunnelled mode (where the size of ALC packets is reduced to take account of the tunnelling overhead, and the UDP checksum of the tunnelled packets is calculated by the FLUTE transmitter rather than the kernel network stack) that tripped us up on #51 and #53.

@dsilhavy

Copy link
Copy Markdown
Contributor Author

@rjb1000 Thanks, the test only runs the vanilla mode not the tunneled mode of operation. I wanted to quickly add a functional test that verifies the basic behavior. I will check if I can add a test for the tunneled mode as well.

jordijoangimenez pushed a commit to jordijoangimenez/rt-libflute that referenced this pull request Jul 17, 2026
jordijoangimenez pushed a commit to jordijoangimenez/rt-libflute that referenced this pull request Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants