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

test(proto): add and update tests for proto/varlogpb/metadata.go #933

Open
wants to merge 1 commit into
base: refactor_remove_unnecessary_code_from_proto_varlogpb_log_entry
Choose a base branch
from

Conversation

ijsong
Copy link
Member

@ijsong ijsong commented Dec 10, 2024

What this PR does

  • Removed unused functions from metadata.go
  • Added comprehensive unit tests for metadata.go and metadata_test.go
  • Improved test coverage for various scenarios

@ijsong ijsong requested a review from hungryjang as a code owner December 10, 2024 14:55
Copy link

codecov bot commented Dec 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.84%. Comparing base (1001a00) to head (cd27630).

Additional details and impacted files
@@                                        Coverage Diff                                         @@
##           refactor_remove_unnecessary_code_from_proto_varlogpb_log_entry     #933      +/-   ##
==================================================================================================
+ Coverage                                                           79.15%   79.84%   +0.69%     
==================================================================================================
  Files                                                                 179      179              
  Lines                                                               21607    21587      -20     
==================================================================================================
+ Hits                                                                17102    17237     +135     
+ Misses                                                               3694     3561     -133     
+ Partials                                                              811      789      -22     

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

@ijsong ijsong force-pushed the test_proto_add_and_update_tests_for_proto_varlogpb_metadata.go branch from b98a528 to faec5d6 Compare December 11, 2024 03:49
@ijsong ijsong force-pushed the refactor_remove_unnecessary_code_from_proto_varlogpb_log_entry branch from 382277e to a62e2db Compare December 13, 2024 01:53
@ijsong ijsong force-pushed the test_proto_add_and_update_tests_for_proto_varlogpb_metadata.go branch from faec5d6 to d43ca78 Compare December 13, 2024 01:53
- Removed unused functions from metadata.go
- Added comprehensive unit tests for metadata.go and metadata_test.go
- Improved test coverage for various scenarios
@ijsong ijsong force-pushed the refactor_remove_unnecessary_code_from_proto_varlogpb_log_entry branch from a62e2db to 1001a00 Compare December 13, 2024 02:01
@ijsong ijsong force-pushed the test_proto_add_and_update_tests_for_proto_varlogpb_metadata.go branch from d43ca78 to cd27630 Compare December 13, 2024 02:01
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.

1 participant