Fixed tests
This commit is contained in:
parent
74b8ec98ec
commit
811e816b2f
|
|
@ -93,7 +93,7 @@ func (r *ReconcileJenkinsBaseConfiguration) validateImagePullSecret(name string)
|
|||
r.logger.V(log.VWarn).Info("Docker Password is empty")
|
||||
return false, nil
|
||||
}
|
||||
if secret.Data["docker-email"] == nil {
|
||||
if secret.Data["docker-email"] == nil {
|
||||
r.logger.V(log.VWarn).Info("Docker Email is empty")
|
||||
return false, nil
|
||||
}
|
||||
|
|
|
|||
|
|
@ -217,7 +217,7 @@ func TestReconcileJenkinsBaseConfiguration_validateImagePullSecrets(t *testing.T
|
|||
baseReconcileLoop := New(fakeClient, nil, logf.ZapLogger(false),
|
||||
&jenkins, false, false, nil, nil)
|
||||
|
||||
got, err := baseReconcileLoop.validateImagePullSecrets()
|
||||
got, _ := baseReconcileLoop.validateImagePullSecrets()
|
||||
assert.Equal(t, got, false)
|
||||
})
|
||||
|
||||
|
|
@ -250,7 +250,7 @@ func TestReconcileJenkinsBaseConfiguration_validateImagePullSecrets(t *testing.T
|
|||
baseReconcileLoop := New(fakeClient, nil, logf.ZapLogger(false),
|
||||
&jenkins, false, false, nil, nil)
|
||||
|
||||
got, err := baseReconcileLoop.validateImagePullSecrets()
|
||||
got, _ := baseReconcileLoop.validateImagePullSecrets()
|
||||
assert.Equal(t, got, false)
|
||||
})
|
||||
|
||||
|
|
@ -283,7 +283,7 @@ func TestReconcileJenkinsBaseConfiguration_validateImagePullSecrets(t *testing.T
|
|||
baseReconcileLoop := New(fakeClient, nil, logf.ZapLogger(false),
|
||||
&jenkins, false, false, nil, nil)
|
||||
|
||||
got, err := baseReconcileLoop.validateImagePullSecrets()
|
||||
got, _ := baseReconcileLoop.validateImagePullSecrets()
|
||||
assert.Equal(t, got, false)
|
||||
})
|
||||
|
||||
|
|
@ -316,7 +316,7 @@ func TestReconcileJenkinsBaseConfiguration_validateImagePullSecrets(t *testing.T
|
|||
baseReconcileLoop := New(fakeClient, nil, logf.ZapLogger(false),
|
||||
&jenkins, false, false, nil, nil)
|
||||
|
||||
got, err := baseReconcileLoop.validateImagePullSecrets()
|
||||
got, _ := baseReconcileLoop.validateImagePullSecrets()
|
||||
assert.Equal(t, got, false)
|
||||
})
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue