Skip to content

Conversation

@dreamjz
Copy link
Contributor

@dreamjz dreamjz commented Dec 17, 2023

Description

@codecov
Copy link

codecov bot commented Dec 17, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7611270) 71.23% compared to head (121eef0) 71.52%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #556      +/-   ##
==========================================
+ Coverage   71.23%   71.52%   +0.28%     
==========================================
  Files          36       36              
  Lines        5521     5521              
==========================================
+ Hits         3933     3949      +16     
+ Misses       1193     1175      -18     
- Partials      395      397       +2     

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

@dreamjz
Copy link
Contributor Author

dreamjz commented Dec 18, 2023

Description

Add test cases for:

  • Add operations: ZAdd
  • Delete operations: ZRem, ZRemRangeByRank, ZPopMin, ZPopMax

Copy link
Member

@bigboss2063 bigboss2063 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks.

@bigboss2063 bigboss2063 merged commit 36a8b6f into nutsdb:master Dec 18, 2023
@dreamjz dreamjz deleted the 503-test-unit_test_ZPeekMin branch December 18, 2023 11:23
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