Skip to content

Commit

Permalink
chore: remove final uses of repo List (#9275)
Browse files Browse the repository at this point in the history
Replacing with ListRepositories()

Signed-off-by: Daniel Helfand <helfand.4@gmail.com>
  • Loading branch information
danielhelfand committed May 2, 2022
1 parent 0137651 commit 216edee
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion test/e2e/fixture/repos/consequences.go
Expand Up @@ -39,7 +39,7 @@ func (c *Consequences) repo() (*v1alpha1.Repository, error) {
func (c *Consequences) get() (*v1alpha1.Repository, error) {
_, repoClient, _ := fixture.ArgoCDClientset.NewRepoClient()

repo, _ := repoClient.List(context.Background(), &repositorypkg.RepoQuery{})
repo, _ := repoClient.ListRepositories(context.Background(), &repositorypkg.RepoQuery{})
for i := range repo.Items {
if repo.Items[i].Repo == c.context.path {
return repo.Items[i], nil
Expand Down
10 changes: 5 additions & 5 deletions test/e2e/repo_management_test.go
Expand Up @@ -24,7 +24,7 @@ func TestAddRemovePublicRepo(t *testing.T) {
assert.NoError(t, err)
defer argoio.Close(conn)

repo, err := repoClient.List(context.Background(), &repositorypkg.RepoQuery{})
repo, err := repoClient.ListRepositories(context.Background(), &repositorypkg.RepoQuery{})

assert.Nil(t, err)
exists := false
Expand All @@ -39,7 +39,7 @@ func TestAddRemovePublicRepo(t *testing.T) {
_, err = fixture.RunCli("repo", "rm", repoUrl)
assert.NoError(t, err)

repo, err = repoClient.List(context.Background(), &repositorypkg.RepoQuery{})
repo, err = repoClient.ListRepositories(context.Background(), &repositorypkg.RepoQuery{})
assert.NoError(t, err)
exists = false
for i := range repo.Items {
Expand Down Expand Up @@ -82,7 +82,7 @@ func TestAddRemoveHelmRepo(t *testing.T) {
assert.NoError(t, err)
defer argoio.Close(conn)

repo, err := repoClient.List(context.Background(), &repositorypkg.RepoQuery{})
repo, err := repoClient.ListRepositories(context.Background(), &repositorypkg.RepoQuery{})

assert.NoError(t, err)
exists := false
Expand All @@ -97,7 +97,7 @@ func TestAddRemoveHelmRepo(t *testing.T) {
_, err = fixture.RunCli("repo", "rm", fixture.RepoURL(fixture.RepoURLTypeHelm))
assert.NoError(t, err)

repo, err = repoClient.List(context.Background(), &repositorypkg.RepoQuery{})
repo, err = repoClient.ListRepositories(context.Background(), &repositorypkg.RepoQuery{})
assert.NoError(t, err)
exists = false
for i := range repo.Items {
Expand Down Expand Up @@ -133,7 +133,7 @@ func TestAddHelmRepoInsecureSkipVerify(t *testing.T) {

defer argoio.Close(conn)

repo, err := repoClient.List(context.Background(), &repositorypkg.RepoQuery{})
repo, err := repoClient.ListRepositories(context.Background(), &repositorypkg.RepoQuery{})

if !assert.NoError(t, err) {
return
Expand Down

0 comments on commit 216edee

Please sign in to comment.