Update maxQueuingCompounded in BulkheadSemaphoreFactory

[ahusseini]

Pull Request

The issue or feature being addressed

Fixes #2637, caused by overflow in BulkheadSemaphoreFactory.

Details on the issue fix or feature implementation

Set compound math to long before evaluating Min with int.MaxValue.

Confirm the following

• I started this PR by branching from the head of the default branch
• I have targeted the PR to merge into the default branch
• I have included unit tests for the issue/feature
• I have successfully run a local build

[ahusseini]

@dotnet-policy-service agree

[martincostello]

Thanks for the PR - could you add a test for this please that validates the exception is no longer thrown?

[ahusseini]

@martincostello added unit tests

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.23%. Comparing base (2f078f9) to head (b246dfe).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2638   +/-   ##
=======================================
  Coverage   96.23%   96.23%           
=======================================
  Files         311      311           
  Lines        7329     7329           
  Branches     1013     1013           
=======================================
  Hits         7053     7053           
  Misses        222      222           
  Partials       54       54           

Flag	Coverage Δ
linux	96.23% <100.00%> (ø)
macos	96.23% <100.00%> (ø)
windows	96.22% <100.00%> (ø)

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.

[martincostello]

Thanks!