Skip to content

test: extend TaskCleanupOldTasks to cover all terminal phases#457

Merged
mvillmow merged 1 commit into
mainfrom
223-auto-impl
Apr 26, 2026
Merged

test: extend TaskCleanupOldTasks to cover all terminal phases#457
mvillmow merged 1 commit into
mainfrom
223-auto-impl

Conversation

@mvillmow
Copy link
Copy Markdown
Collaborator

Summary

Test Plan

  • Formatting check passed (just format-check)
  • Test file compiles cleanly
  • Follows existing test patterns in distributed_grpc_test.cpp

🤖 Generated with Claude Code

@mvillmow mvillmow enabled auto-merge (rebase) April 26, 2026 00:20
@github-actions
Copy link
Copy Markdown

✅ Dependency Audit

Severity Count
Critical 0
High 0
Medium 0
Low 0

See the Security tab for detailed findings.


Workflow: Dependency Audit

@github-actions
Copy link
Copy Markdown

Security Scan Results

  • ❌ Secret Scanning: Potential secrets found
  • ✅ SAST: Completed (check Security tab for details)
  • ✅ Dependency Scanning: Completed
  • ✅ C++ Static Analysis: Completed
  • ✅ Docker Image Scanning: 0 high, 22 medium vulnerabilities (acceptable)

Recommendations

  • Review findings in the GitHub Security tab
  • Check artifact uploads for detailed reports
  • Address critical Docker vulnerabilities immediately

Workflow: Security Scanning

auto-merge was automatically disabled April 26, 2026 15:02

Rebase merges are not allowed on this repository

Add test cases for TASK_PHASE_ERROR, TASK_PHASE_TIMEOUT, and
TASK_PHASE_CANCELLED in the TaskCleanupOldTasks E2E test to ensure
the full terminal state set is covered by cleanup verification.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mvillmow mvillmow enabled auto-merge (squash) April 26, 2026 18:30
@mvillmow mvillmow merged commit f7f827b into main Apr 26, 2026
8 of 9 checks passed
@mvillmow mvillmow deleted the 223-auto-impl branch April 26, 2026 18:30
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.

1 participant