Skip to content

allocator: add unit test and optimize constraint funcs#2

Open
wllenyj wants to merge 1 commit intowllenyj:mainfrom
studychao:chao/add_unit_test_
Open

allocator: add unit test and optimize constraint funcs#2
wllenyj wants to merge 1 commit intowllenyj:mainfrom
studychao:chao/add_unit_test_

Conversation

@wllenyj
Copy link
Owner

@wllenyj wllenyj commented Jan 7, 2022

This commit aims to:

  1. fix Constraint's set functions
  2. add some unit tests for Constraint

Signed-off-by: Chao Wu chaowu@linux.alibaba.com

@codecov-commenter
Copy link

Codecov Report

Merging #2 (1bad223) into main (5c1cf8d) will increase coverage by 1.73%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
+ Coverage   88.95%   90.68%   +1.73%     
==========================================
  Files           2        2              
  Lines         525      569      +44     
==========================================
+ Hits          467      516      +49     
+ Misses         58       53       -5     
Impacted Files Coverage Δ
crates/db-allocator/src/lib.rs 90.00% <100.00%> (+58.75%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c1cf8d...1bad223. Read the comment docs.

@studychao studychao force-pushed the chao/add_unit_test_ branch 3 times, most recently from 3d7cf98 to cd4edb3 Compare January 10, 2022 03:57
This commit aims to:
1. optimize Constraint's set functions
2. add some unit tests for Constraint

Signed-off-by: Chao Wu <chaowu@linux.alibaba.com>
@studychao studychao force-pushed the chao/add_unit_test_ branch from cd4edb3 to a76b72f Compare January 10, 2022 07:20
@wllenyj wllenyj force-pushed the main branch 3 times, most recently from abe1c45 to 0fb7201 Compare January 11, 2022 07:27
@wllenyj wllenyj force-pushed the main branch 18 times, most recently from 732edb6 to 3fca79d Compare February 15, 2022 12:40
@wllenyj wllenyj force-pushed the main branch 30 times, most recently from 23047c7 to 348dac4 Compare February 16, 2022 04:53
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.

3 participants