Skip to content

test: cover audit retention worker lifecycle - #2620

Open
Chris0Jeky wants to merge 1 commit into
mainfrom
issue-1452/audit-retention-lifecycle
Open

test: cover audit retention worker lifecycle#2620
Chris0Jeky wants to merge 1 commit into
mainfrom
issue-1452/audit-retention-lifecycle

Conversation

@Chris0Jeky

Copy link
Copy Markdown
Owner

Refs #1452

Summary

  • Exercise AuditRetentionWorker through StartAsync and its real ExecuteAsync loop.
  • Prove the first cleanup receives the configured batch size and a bounded retention cutoff.
  • Verify the worker heartbeat and bounded StopAsync cancellation while the configured idle delay is pending.

Verification

  • AuditRetentionWorkerTests: 5/5
  • Mutation check: bypassing the production cleanup call caused the new test to time out; production code was restored and the class returned to 5/5.
  • git diff --check: passed

Two coordinator review passes completed for lifecycle correctness and scope/test quality.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Pending

Development

Successfully merging this pull request may close these issues.

1 participant