Skip to content

Improve the record layer fuzz tests#750

Merged
philipch07 merged 1 commit intomasterfrom
pch07/improve-record-layer-fuzz-tests
Oct 17, 2025
Merged

Improve the record layer fuzz tests#750
philipch07 merged 1 commit intomasterfrom
pch07/improve-record-layer-fuzz-tests

Conversation

@philipch07
Copy link
Copy Markdown
Contributor

Description

  1. Moves the record layer fuzz tests from their own file to the test file so we don't have an extra file.
  2. Also rewrites some of the record layer tests to test DTLS v1.2 instead of DTLS v1.0 since we don't support v1.0.
  3. Adds new fuzz tests that checks for more than just panicking.

Reference issue

N/A, part of the effort to add fuzz tests to various Pion projects.

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.06%. Comparing base (daa0fd4) to head (b2525af).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #750      +/-   ##
==========================================
- Coverage   79.20%   79.06%   -0.14%     
==========================================
  Files         101      102       +1     
  Lines        6915     6917       +2     
==========================================
- Hits         5477     5469       -8     
- Misses       1063     1070       +7     
- Partials      375      378       +3     
Flag Coverage Δ
go 79.08% <ø> (-0.12%) ⬇️
wasm 59.89% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Member

@JoTurk JoTurk left a comment

Choose a reason for hiding this comment

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

Thank you, and sorry for taking time to get back to you on this one

<3

@philipch07 philipch07 force-pushed the pch07/improve-record-layer-fuzz-tests branch from b8dfeb3 to b2525af Compare October 17, 2025 15:46
@philipch07 philipch07 merged commit 9cfb13f into master Oct 17, 2025
30 of 32 checks passed
@philipch07 philipch07 deleted the pch07/improve-record-layer-fuzz-tests branch October 17, 2025 15:55
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