ThreeMammals / Ocelot

96%

LAST BUILD ON BRANCH develop

branch: develop

CHANGE BRANCH

x

Reset

• develop
• 2080-adds-parameters-in-this-case-those-of-polly-v8-to-fine-tune-circuit-breaker-behavior
• 2168-watchkube
• 2254-coveralls
• 2334-Butterfly
• 2334-CacheManager
• 2334-OpenTracing
• 2339-fix-tests
• 2344-upgrade-to-vs2026
• 2354-follow-up-2
• 2371-Ocelot.Provider.Eureka
• 2378-Ocelot.Provider.Consul
• 2378-Ocelot.Provider.Polly
• 2379-follow-up
• 24.0.1-Ocelot.Testing
• 24.1.0-beta
• 24.1.0-beta.1
• 24.1.0-beta.2
• 25.0.0-beta.2
• Burgyn-patch-1
• SDK-10.0.203
• TestStack.BDDfy.Xunit
• allow-cache-filtering-based-on-reseponse-statuscode
• allow-empty-required-claim-value
• bug-2299
• bug-2304-PollKube
• bug/2346-query-param-id-regression
• copilot/2363-followup-2
• copilot/add-trx-logger-for-unit-tests
• copilot/fix-failing-unit-test-and-thread-leak
• copilot/implement-circuit-breaker-pattern
• copilot/net1005-bump-packages
• feat-2319
• feat-2320-http-handler
• feat-2330
• feat_enableHttp2
• feat_rm_newtonsoft
• feature/1658---Global-UpstreamHeaderTransform-settings-in-GlobalConfiguration-section
• feature/2316-auth-opts
• feature/2338-qos-opts
• feature/651-merge-custom-properties
• feature/842-authenticationoptions-in-globalconfig
• feature/add-feature-for-ocelot-pipeline-configuration
• feature/add-global-rate-limit
• feature/downstream-service-name
• feature/multiple-values-in-routeclaims
• feature/sse-support
• feature/websocket-buffer-size-25.0
• fix-invalid-header-502
• fix-static-route-special-chars
• fix/aggregate-routekeys-nandan
• fix/roundrobin-null-service-name
• harris-patch-2
• issues-1869
• issues-930
• main
• merge-conflicts
• mgs25/unittests-xunitv3-migration
• patch-1
• patch/24.0.1
• pre-release-24.1
• raman-m-patch-1
• raman/24.0
• release/24.0
• release/24.1
• release/24.1-dev
• service-discovery-path-prefix
• user/carl-aggregate-methods

Build # 27360992851

Build Type

push

github

Committed by web-flow

Commit Message

Harden the Kube provider, repository FileConfigurationPoller, and infrastructure InMemoryBus, and stabilize their tests (#2399)

* Add TRX logger to workflow and fix foreground thread in InMemoryBus
* Fix workflow to use MTP-compatible TRX reporting options
* Search for unit-tests.trx in bin/Debug/ subfolders
* Fix: Dispose timer in FileConfigurationPoller.StopAsync to prevent foreground thread leak
- The StopAsync method was swapping out the timer reference but never disposing it.
System.Threading.Timer holds a background thread that must be disposed to properly clean up resources. Without disposal, the timer's background thread remains alive, causing the 'Foreground threads were left running' error during test execution.
- This fix ensures the timer is properly disposed after stopping it, preventing thread leaks and allowing unit tests to complete cleanly.

* Fix: Handle ObjectDisposedException in Kube.GetEndpoint() during async operations
- The GetEndpoint() method can be called while Dispose() is executing in another thread, causing ObjectDisposedException when the _kubeApi is disposed before the HTTP request completes. This race condition commonly occurs during application shutdown.
- The fix adds specific exception handling for ObjectDisposedException in GetEndpoint(), treating it as an expected error condition during service cleanup rather than an unexpected failure.
This prevents the exception from propagating and causing test failures or application errors during shutdown.

* Add unit tests for code coverage in PR #14 - FileConfigurationPoller and InMemoryBus
* Add acceptance tests for Kube ObjectDisposedException handling in PR #14
* Move KubeTests
* Disable TRX file creation due to expected exit code 0
* Remove the Unit Tests step workaround for exit code overrides and TRX asset uploads
* Coverlet table
* coverlet doesn't show statistics...
* Add comprehensive unit tests for Kube.cs exception handling
* Add comprehensive unit tests covering... (continued)

Coverage Stats

6867 of 7129 relevant lines covered (96.32%)

3599.47 hits per line