UTs added for mount secret case, updated for config map as well #444
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.
Description
Unit tests updated for mount secret/credential. The test framework is a bit different as to enable more accurate reporting of the coverage. A single TestMain running both the secret and config map case reported coverage for only one of them (due to how go's coverage tool is).
A main test function now serializes how the tests are run that have a dependency of requiring a mock proxy server.
GitHub Issues
List the GitHub issues impacted by this PR:
| GitHub Issue # |
(dell/csm#1614)
Checklist:
How Has This Been Tested?
Unit tests run for both secret and config map case.
PASS
coverage: 69.8% of statements
ok github.com/dell/csi-powermax/csireverseproxy/v2 60.492s coverage: 69.8% of statements