Closed svagner closed 4 years ago
Yes, thanks for the PR 👍
I think we probably need two tests actually:
preserveFullHostName == true
preserveFullHostName == false
but this test must then split the copy of expectedHostname
in the test on the first .
(which the current test isn't doing).If you would like to update your PR to cover both of these scenarios that would be great. Otherwise I'll be happy to do this.
Once you bring your branch inline with master we should be able to merge
Thanks @tdinucci ! Branch was updated:)
Please follow the guide below
Description
Fix test
TestNewManager
Use only full hostname for tests (see os.Hostname())Fixes #2460
Type of change
How has this been tested?
Checklist: