msquic icon indicating copy to clipboard operation
msquic copied to clipboard

[DeepTest] Add Unit Test cases for Cubic Congestion Control

Open saikat107 opened this issue 2 weeks ago • 3 comments

Description

This Pull request adds 14 test cases to test different scenarios of CubicCongestionControlInitialize. These test cases are generated by DeepTest Agents with Agency. Each test case tests one scenario and is annotated with the sepcific scenario as comments.

Testing

The following test cases are added

CubicTest.InitializeWithDefaultSettings
CubicTest.InitializeWithMtuBoundaries
CubicTest.InitializeWithInitialWindowBoundaries
CubicTest.InitializeWithSendIdleTimeoutBoundaries
CubicTest.VerifyHyStartInitialization
CubicTest.VerifyFunctionPointers
CubicTest.VerifyInitialStateFlags
CubicTest.VerifyZeroInitializedFields
CubicTest.MultipleSequentialInitializations
CubicTest.CanSendScenarios
CubicTest.SetExemption
CubicTest.GetSendAllowanceScenarios
CubicTest.GetterFunctions
CubicTest.ResetScenarios

Documentation

N/A

saikat107 avatar Dec 09 '25 23:12 saikat107

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 85.52%. Comparing base (0b1071e) to head (637f7af).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5648      +/-   ##
==========================================
+ Coverage   85.05%   85.52%   +0.47%     
==========================================
  Files          60       60              
  Lines       18663    18663              
==========================================
+ Hits        15873    15962      +89     
+ Misses       2790     2701      -89     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

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

codecov[bot] avatar Dec 10 '25 00:12 codecov[bot]

Not sure why the test in the CI is failing. In my local run, the test case pass.

BTW, here is (AI-agent generated) documentation of the added tests (I did check, but may have missed something) COVERAGE_REPORT.md

saikat107 avatar Dec 10 '25 23:12 saikat107

I started a re-run for the test, it seems unrelated to this change. I'll look into it. I'll try to review the PR shortly.

guhetier avatar Dec 11 '25 17:12 guhetier

@guhetier, abandoning this PR, as we pushed commits for bbr testing (which is not yet complete and require more work) to this branch. Opened a new PR #5653

saikat107 avatar Dec 12 '25 04:12 saikat107