Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate sha1 unit test to new framework #470

Merged
merged 8 commits into from
May 10, 2024

Conversation

Shivshankar-Reddy
Copy link
Contributor

This migrates unit tests related to sha1 to new framework, ref: #428.

Copy link

codecov bot commented May 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.67%. Comparing base (0342a81) to head (6691d73).

Additional details and impacted files
@@             Coverage Diff              @@
##           unstable     #470      +/-   ##
============================================
- Coverage     69.80%   69.67%   -0.13%     
============================================
  Files           109      109              
  Lines         61791    61791              
============================================
- Hits          43135    43055      -80     
- Misses        18656    18736      +80     
Files Coverage Δ
src/server.c 88.60% <ø> (ø)
src/sha1.c 98.66% <ø> (ø)

... and 14 files with indirect coverage changes

src/unit/test_sha1.c Outdated Show resolved Hide resolved
Signed-off-by: Shivshankar-Reddy <shiva.sheri.github@gmail.com>
Signed-off-by: Shivshankar-Reddy <shiva.sheri.github@gmail.com>
Signed-off-by: Shivshankar-Reddy <shiva.sheri.github@gmail.com>
Signed-off-by: Shivshankar-Reddy <shiva.sheri.github@gmail.com>
Signed-off-by: Shivshankar-Reddy <shiva.sheri.github@gmail.com>
Signed-off-by: Shivshankar-Reddy <shiva.sheri.github@gmail.com>
src/unit/test_sha1.c Outdated Show resolved Hide resolved
Signed-off-by: Madelyn Olson <madelyneolson@gmail.com>
@madolson madolson merged commit e242799 into valkey-io:unstable May 10, 2024
17 checks passed
@Shivshankar-Reddy Shivshankar-Reddy deleted the test-sha1 branch May 10, 2024 17:13
karthyuom pushed a commit to karthyuom/valkey that referenced this pull request May 13, 2024
This migrates unit tests related to sha1 to new framework, ref: valkey-io#428.

---------

Signed-off-by: Shivshankar-Reddy <shiva.sheri.github@gmail.com>
Signed-off-by: Madelyn Olson <madelyneolson@gmail.com>
Co-authored-by: Madelyn Olson <madelyneolson@gmail.com>

Signed-off-by: Karthick Ariyaratnam <karthyuom@gmail.com>
srgsanky pushed a commit to srgsanky/valkey that referenced this pull request May 19, 2024
This migrates unit tests related to sha1 to new framework, ref: valkey-io#428.

---------

Signed-off-by: Shivshankar-Reddy <shiva.sheri.github@gmail.com>
Signed-off-by: Madelyn Olson <madelyneolson@gmail.com>
Co-authored-by: Madelyn Olson <madelyneolson@gmail.com>
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