Skip to content

test: fix race in WiresInnerHandlerOnlyOnce queue test - #599

Merged
JerrettDavis merged 1 commit into
mainfrom
fix/flaky-queue-decorator-test
Oct 20, 2026
Merged

JerrettDavis merged 1 commit into
mainfrom
fix/flaky-queue-decorator-test

Conversation

@JerrettDavis

Copy link
Copy Markdown
Owner

Test asserted Completed==1 immediately after handlers ran, but DurableQueueChannelDecorator's drain loop calls CompleteAsync after handlers return. Poll for completion (5s cap) instead.

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions

Copy link
Copy Markdown
Contributor

Code Coverage

Summary
  Generated on: 10/20/2026 - 01:26:53
  Coverage date: 10/20/2026 - 01:25:05 - 10/20/2026 - 01:26:46
  Parser: MultiReport (7x Cobertura)
  Assemblies: 23
  Classes: 1067
  Files: 591
  Line coverage: 74.6%
  Covered lines: 36848
  Uncovered lines: 12492
  Coverable lines: 49340
  Total lines: 97328
  Branch coverage: 61.8% (13501 of 21830)
  Covered branches: 13501
  Total branches: 21830
  Method coverage: 86.4% (6318 of 7310)
  Full method coverage: 74.2% (5428 of 7310)
  Covered methods: 6318
  Fully covered methods: 5428
  Total methods: 7310

@github-actions

Copy link
Copy Markdown
Contributor

Test Results

7 641 tests   7 622 ✅  2m 26s ⏱️
    7 suites     19 💤
    7 files        0 ❌

Results for commit ea5209c.

@JerrettDavis
JerrettDavis merged commit 0ac1891 into main Oct 20, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant