sharppcap icon indicating copy to clipboard operation
sharppcap copied to clipboard

Add a CaptureQueue class

Open SebMichaud opened this issue 7 months ago • 3 comments

Added a CaptureQueue class following issue 575 and its last message from @kayoub5

SebMichaud avatar Apr 22 '25 12:04 SebMichaud

Codecov Report

Attention: Patch coverage is 0% with 27 lines in your changes missing coverage. Please review.

Project coverage is 68.70%. Comparing base (beeb097) to head (11748ab). Report is 5 commits behind head on master.

Files with missing lines Patch % Lines
SharpPcap/LibPcap/CaptureQueue.cs 0.00% 27 Missing :warning:

:exclamation: There is a different number of reports uploaded between BASE (beeb097) and HEAD (11748ab). Click for more details.

HEAD has 3 uploads less than BASE
Flag BASE (beeb097) HEAD (11748ab)
7 4
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #580       +/-   ##
===========================================
- Coverage   81.16%   68.70%   -12.46%     
===========================================
  Files          51       52        +1     
  Lines        2814     2841       +27     
  Branches      328      327        -1     
===========================================
- Hits         2284     1952      -332     
- Misses        417      782      +365     
+ Partials      113      107        -6     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Apr 22 '25 13:04 codecov[bot]

Hi @kayoub5 can you see my commit from today ? I tried to add the suggested features. Anyway, it doesn't work and I don't know why. I need to cast the BlockingCollection to a simple List in the Flush method, because it's way more convenient for the final user. Could you review it and test it on your side ? Thanks

SebMichaud avatar May 06 '25 00:05 SebMichaud

@SebMichaud can you add a test for the CaptureQueue in the Tests folder that uses the class in the way that your labview side is using it? That would let us exercise the api and make it easier for others to help debug and resolve implementation issues.

chmorgan avatar May 25 '25 12:05 chmorgan