Augment the unit tests for the JoinClient with the happy cases, so that there's test coverage for this change:
Capture new member node names in etcd controller. This moves the side effect of the os.Hostname() call out of the client, so that it doesn't influence the tests.
Change the fake HTTP server to HTTPS, which will test the certificate handling properly.
Type of change
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
How Has This Been Tested?
[ ] Manual test
[x] Auto test added
Checklist:
[x] My code follows the style guidelines of this project
Description
This makes the code a lot easier.
Augment the unit tests for the
JoinClient
with the happy cases, so that there's test coverage for this change:os.Hostname()
call out of the client, so that it doesn't influence the tests.Type of change
How Has This Been Tested?
Checklist: