You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The #791 fix added three log lines to QfcStreamingDequeueConfidenceGate (launch, zero-acceptance checkpoint, scan bound reached). The launch and checkpoint lines are content-asserted by tests; the scan-bound line emitted by LogScanBoundReached is not asserted by any test, so a regression in its content (the Bound= value or the Decision=stop token) would pass the suite. AC1 of #791 requires the bound decision to be logged.
Command/flags used: static review of branch bug/quickfiler-high-confidence-cancel-teardown-and-deadline-defects-791 at 5953636
Data source or fixture: none (code review finding N3 in code-review.2026-09-06T15-31.md)
Steps to Reproduce
Search QuickFiler.Test for the literals scan bound reached, Bound=, and Decision=stop: zero matches.
Compare with the launch and checkpoint lines, which QfcStreamingDequeueConfidenceGateTests.Part4.cs asserts through the injected debugLog delegate.
Expected Behavior
A test drives the gate to ScanCapReached (item cap and time ceiling) and asserts the emitted line carries the cutoff, scanned and accepted counts, the bound that fired, and the stop decision.
Actual Behavior
The ScanCapReached tests assert the stop reason and that no further take occurs, but not the log line content.
Logs / Screenshots
Attached minimal logs or screenshot
Snippet: none; static finding.
Impact / Severity
Blocker
High
Medium
Low
Low: a diagnostics-only gap; the behavior itself is tested.
Verified fixed by PR #893 (consolidated item #872). QuickFiler.Test/Controllers/QfcStreamingDequeueConfidenceGateTests.Part4.cs:207-218 asserts the scan-bound line content (scan bound reached, Bound=scan-cap, Decision=stop), and line 303 asserts Bound=zero-acceptance-ceiling. Closing as fixed by #893.
Activity
drmoisan commentedon Sep 28, 2026
Verified fixed by PR #893 (consolidated item #872).
QuickFiler.Test/Controllers/QfcStreamingDequeueConfidenceGateTests.Part4.cs:207-218asserts the scan-bound line content (scan bound reached,Bound=scan-cap,Decision=stop), and line 303 assertsBound=zero-acceptance-ceiling. Closing as fixed by #893.