djpbessems
/
ContainerImage.Pinniped
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

In TestImpersonationProxy, instead of failing in this case just skip the test. 

Signed-off-by: Matt Moyer <moyerm@vmware.com>
This commit is contained in:
Matt Moyer 2021-03-26 16:28:33 -05:00
parent 3359311228
commit c6d7724b67
No known key found for this signature in database
GPG Key ID: EAE88AD172C5AE2D
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
test/integration/concierge_impersonation_proxy_test.go
View File

@ -226,9 +226,10 @@ func TestImpersonationProxy(t *testing.T) { //nolint:gocyclo // yeah, it's compl
// Auto mode should have decided that the impersonator will be disabled. We need to manually enable it. // Auto mode should have decided that the impersonator will be disabled. We need to manually enable it.
// However, the cluster does not support load balancers so we should enable it without a load balancer // However, the cluster does not support load balancers so we should enable it without a load balancer
// and use squid to make requests. (e.g. kind) // and use squid to make requests. (e.g. kind)
require.NotEmpty(t, env.Proxy, if env.Proxy == "" {
"test cluster does not support load balancers but also doesn't have a squid proxy... "+ t.Skip("test cluster does not support load balancers but also doesn't have a squid proxy... " +
"this is not a supported configuration for test clusters") "this is not a supported configuration for test clusters")
}
// Check that no load balancer has been created by the impersonator's "auto" mode. // Check that no load balancer has been created by the impersonator's "auto" mode.
library.RequireNeverWithoutError(t, func() (bool, error) { library.RequireNeverWithoutError(t, func() (bool, error) {