chore: fix broken TestValidatePermissions unit test (argoproj#8147)

Signed-off-by: Alexander Matyushentsev <[email protected]>
fishman · Jan 11, 2022 · 5d770d6 · 5d770d6
1 parent dfd40ef
commit 5d770d6
Showing 1 changed file with 2 additions and 11 deletions.
diff --git a/util/argo/argo_test.go b/util/argo/argo_test.go
@@ -604,14 +604,7 @@ func TestValidatePermissions(t *testing.T) {
  },
  }
  db := &dbmocks.ArgoDB{}
- db.On("ListClusters", context.Background()).Return(&argoappv1.ClusterList{
- Items: []argoappv1.Cluster{
- {
- Name: "dind",
- Server: "https://127.0.0.1:6443",
- },
- },
- }, nil)
+ db.On("GetClusterServersByName", context.Background(), "does-not-exist").Return(nil, nil)
  conditions, err := ValidatePermissions(context.Background(), &spec, &proj, db)
  assert.NoError(t, err)
  assert.Len(t, conditions, 1)
@@ -677,9 +670,7 @@ func TestValidatePermissions(t *testing.T) {
  Name: "does-exist",
  Server: "https://127.0.0.1:6443",
  }
- db.On("ListClusters", context.Background()).Return(&argoappv1.ClusterList{
- Items: []argoappv1.Cluster{cluster},
- }, nil)
+ db.On("GetClusterServersByName", context.Background(), "does-exist").Return([]string{"https://127.0.0.1:6443"}, nil)
  db.On("GetCluster", context.Background(), "https://127.0.0.1:6443").Return(&cluster, nil)
  conditions, err := ValidatePermissions(context.Background(), &spec, &proj, db)
  assert.NoError(t, err)