Skip to content

Conversation

@swhitty
Copy link
Owner

@swhitty swhitty commented Nov 23, 2024

Updates AsyncSocket.Message to expose its bytes as payload: Data.

Extends the existing datagram tests to better cover sendMessage().

For these datagram tests:

  • Darwin uses AF_INET6 sockets
  • Linux uses AF_UNIX pair.

@codecov
Copy link

codecov bot commented Nov 23, 2024

Codecov Report

Attention: Patch coverage is 81.81818% with 2 lines in your changes missing coverage. Please review.

Project coverage is 94.77%. Comparing base (13778ca) to head (02e89e5).

Files with missing lines Patch % Lines
FlyingSocks/Sources/AsyncSocket.swift 81.81% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #136      +/-   ##
==========================================
+ Coverage   92.43%   94.77%   +2.34%     
==========================================
  Files          60       60              
  Lines        3938     3943       +5     
==========================================
+ Hits         3640     3737      +97     
+ Misses        298      206      -92     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@swhitty swhitty merged commit 54dc4b7 into main Nov 23, 2024
17 of 18 checks passed
@swhitty swhitty deleted the more-datagram-tests branch February 10, 2025 21:28
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.

2 participants