Skip to content

cleanup: Some more test cleanups, removing overly smart code.#2536

Merged
toktok-releaser merged 1 commit into
TokTok:masterfrom
iphydf:test-cleanups
Jan 10, 2024
Merged

cleanup: Some more test cleanups, removing overly smart code.#2536
toktok-releaser merged 1 commit into
TokTok:masterfrom
iphydf:test-cleanups

Conversation

@iphydf

@iphydf iphydf commented Jan 10, 2024

Copy link
Copy Markdown
Member

This change is Reviewable

@iphydf iphydf added this to the v0.2.19 milestone Jan 10, 2024
@iphydf
iphydf marked this pull request as ready for review January 10, 2024 02:59
@iphydf
iphydf requested a review from a team as a code owner January 10, 2024 02:59
@codecov

codecov Bot commented Jan 10, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (0426624) 69.00% compared to head (32b68cf) 68.96%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2536      +/-   ##
==========================================
- Coverage   69.00%   68.96%   -0.05%     
==========================================
  Files          96       96              
  Lines       28023    28022       -1     
==========================================
- Hits        19337    19324      -13     
- Misses       8686     8698      +12     

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

Comment thread toxcore/DHT_test.cc
Comment thread toxcore/test_util.hh
@pull-request-attention pull-request-attention Bot assigned Green-Sky and iphydf and unassigned iphydf and Green-Sky Jan 10, 2024
@toktok-releaser
toktok-releaser merged commit 32b68cf into TokTok:master Jan 10, 2024
@iphydf
iphydf deleted the test-cleanups branch January 10, 2024 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants