Skip to content

Stabilize DiscretePolicy touch tick count test#759

Merged
bitfaster merged 7 commits intomainfrom
copilot/update-stable-touch-updates-test
Apr 13, 2026
Merged

Stabilize DiscretePolicy touch tick count test#759
bitfaster merged 7 commits intomainfrom
copilot/update-stable-touch-updates-test

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 13, 2026

  • Inspect the existing DiscretePolicy and AfterAccessPolicy tick-count tests and the referenced commit
  • Run formatting, build, and unit tests to capture the current baseline
  • Update BitFaster.Caching.UnitTests/Lru/DiscretePolicyTests.cs to use the same polling loop pattern as AfterAccessPolicyTests
  • Run dotnet format, build, and targeted/unit tests to verify the fix
  • Create the pull request with the test stabilization change

Copilot AI and others added 4 commits April 13, 2026 07:00
Agent-Logs-Url: https://github.com/bitfaster/BitFaster.Caching/sessions/9dea696c-9322-4f51-9c6b-4c54780aee47

Co-authored-by: bitfaster <12851828+bitfaster@users.noreply.github.com>
Agent-Logs-Url: https://github.com/bitfaster/BitFaster.Caching/sessions/9dea696c-9322-4f51-9c6b-4c54780aee47

Co-authored-by: bitfaster <12851828+bitfaster@users.noreply.github.com>
Agent-Logs-Url: https://github.com/bitfaster/BitFaster.Caching/sessions/9dea696c-9322-4f51-9c6b-4c54780aee47

Co-authored-by: bitfaster <12851828+bitfaster@users.noreply.github.com>
@bitfaster bitfaster marked this pull request as ready for review April 13, 2026 17:15
@coveralls
Copy link
Copy Markdown

coveralls commented Apr 13, 2026

Coverage Status

coverage: 98.978%. first build — copilot/update-stable-touch-updates-test into main

@bitfaster bitfaster merged commit 3d54588 into main Apr 13, 2026
16 checks passed
@bitfaster bitfaster deleted the copilot/update-stable-touch-updates-test branch April 13, 2026 20:18
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.

3 participants