Tests were skipped to support older clients (<6.9). Since supported clients have support for TenantManager, unskipped the test and removed the OldClients test.
Type of change
Please mark options that are relevant:
[ *] 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)
[ ] This change requires a documentation update
[ ] Build related change
How Has This Been Tested?
[ x] Ran go test -v -test.name=TestTenant which passed.
[ x] Ran make test which failed on TestHostContainerBacking because I don't have docker installed(?).
Checklist:
[ ] My code follows the CONTRIBUTIONguidelines of this project
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ x] I have added tests that prove my fix is effective or that my feature works
[ x] New and existing unit tests pass locally with my changes
Description
Type of change
Please mark options that are relevant:
How Has This Been Tested?
[ x] Ran
go test -v -test.name=TestTenant
which passed.[ x] Ran
make test
which failed onTestHostContainerBacking
because I don't have docker installed(?).Checklist:
[ ] My code follows the
CONTRIBUTION
guidelines of this project[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ x] I have added tests that prove my fix is effective or that my feature works
[ x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged