test: add UT for blobfuse-proxy/main.go #1801
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request to
pkg/blobfuse-proxy
introduces a mechanism to mock the gRPC server runner for testing purposes. The most important changes include adding a new variable to hold the gRPC server runner function, updating the main function to use this new variable, and creating a test to mock the gRPC server runner.Changes to enable mocking the gRPC server runner:
pkg/blobfuse-proxy/main.go
: Added a new variablegrpcServerRunner
to hold the gRPC server runner function and updated the main function to use this variable instead of callingserver.RunGRPCServer
directly. [1] [2]Tests for the new mocking mechanism:
pkg/blobfuse-proxy/main_test.go
: Added a new test file to mock the gRPC server runner and test the main function.<!-- Thanks for sending a pull request! Here are some tips for you:https://git.k8s.io/community/contributors/devel/release.md#issue-kind-label
-->
What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Requirements:
Special notes for your reviewer:
Release note: