From b096654469c0ce0f95be38652696a935bdaffb8a Mon Sep 17 00:00:00 2001 From: Jason Dellaluce Date: Thu, 9 Feb 2023 13:55:57 +0000 Subject: [PATCH] test: adapt tests to service factory and move them into config package Signed-off-by: Jason Dellaluce --- main_test.go | 95 --------------------------------------- pkg/config/config_test.go | 83 ++++++++++++++++++++++++++++++++++ 2 files changed, 83 insertions(+), 95 deletions(-) delete mode 100644 main_test.go create mode 100644 pkg/config/config_test.go diff --git a/main_test.go b/main_test.go deleted file mode 100644 index da1873b..0000000 --- a/main_test.go +++ /dev/null @@ -1,95 +0,0 @@ -package main - -import ( - "context" - "testing" - - "github.com/FedeDP/GhEnvSet/pkg/config" - "github.com/FedeDP/GhEnvSet/pkg/poiana" - "github.com/stretchr/testify/assert" - "github.com/stretchr/testify/require" -) - -const testYAML = ` -orgs: - FedeDP: - actions: - variables: - orgVar1: "orgValue1" - secrets: - - orgSecret0 - repos: - GhEnvSet: - actions: - variables: - repoVar1: "repoValue1" - repoVar2: "repoValue2" - secrets: - - repoSecret0 - - repoSecret1 - - repoSecret2 -` - -func TestMainLoop(t *testing.T) { - ctx := context.Background() - - conf, err := config.FromData(testYAML) - require.NoError(t, err) - - provider, err := poiana.NewMockSecretsProvider(map[string]string{ - "orgSecret0": "orgValue0", - "repoSecret0": "repoValue0", - "repoSecret1": "repoValue1", - "repoSecret2": "repoValue2", - }) - require.NoError(t, err) - - // sync the org and the repos with the config - require.Len(t, conf.Orgs, 1) - for orgName, org := range conf.Orgs { - mockVarServ := poiana.NewMockVariableService() - mockSecServ := poiana.NewMockSecretsService() - require.Equal(t, orgName, "FedeDP") - require.NoError(t, org.Actions.Sync(ctx, provider, mockVarServ, mockSecServ, false)) - - // Check variables - vars, _, err := mockVarServ.ListVariables(ctx, nil) - assert.NoError(t, err) - assert.Equal(t, vars.TotalCount, len(conf.Orgs["FedeDP"].Actions.Variables)) - for _, v := range vars.Variables { - assert.Equal(t, conf.Orgs["FedeDP"].Actions.Variables[v.Name], v.Value) - } - - // Check secrets - secs, _, err := mockSecServ.ListSecrets(ctx, nil) - assert.NoError(t, err) - assert.Equal(t, secs.TotalCount, len(conf.Orgs["FedeDP"].Actions.Secrets)) - for _, sec := range secs.Secrets { - assert.Contains(t, conf.Orgs["FedeDP"].Actions.Secrets, sec.Name) - } - - require.Len(t, org.Repos, 1) - for repoName, repo := range org.Repos { - mockVarServ := poiana.NewMockVariableService() - mockSecServ := poiana.NewMockSecretsService() - require.Equal(t, repoName, "GhEnvSet") - require.NoError(t, repo.Actions.Sync(ctx, provider, mockVarServ, mockSecServ, false)) - - // Check variables - vars, _, err = mockVarServ.ListVariables(ctx, nil) - assert.NoError(t, err) - assert.Equal(t, vars.TotalCount, len(conf.Orgs["FedeDP"].Repos["GhEnvSet"].Actions.Variables)) - for _, v := range vars.Variables { - assert.Equal(t, conf.Orgs["FedeDP"].Repos["GhEnvSet"].Actions.Variables[v.Name], v.Value) - } - - // Check secrets - secs, _, err = mockSecServ.ListSecrets(ctx, nil) - assert.NoError(t, err) - assert.Equal(t, secs.TotalCount, len(conf.Orgs["FedeDP"].Repos["GhEnvSet"].Actions.Secrets)) - for _, sec := range secs.Secrets { - assert.Contains(t, conf.Orgs["FedeDP"].Repos["GhEnvSet"].Actions.Secrets, sec.Name) - } - } - } -} diff --git a/pkg/config/config_test.go b/pkg/config/config_test.go new file mode 100644 index 0000000..c5ccbb4 --- /dev/null +++ b/pkg/config/config_test.go @@ -0,0 +1,83 @@ +package config + +import ( + "context" + "testing" + + "github.com/FedeDP/GhEnvSet/pkg/poiana" + "github.com/stretchr/testify/assert" +) + +const testYAML = ` +orgs: + FedeDP: + actions: + variables: + orgVar1: "orgValue1" + secrets: + - orgSecret0 + repos: + GhEnvSet: + actions: + variables: + repoVar1: "repoValue1" + repoVar2: "repoValue2" + secrets: + - repoSecret0 + - repoSecret1 + - repoSecret2 +` + +func TestConfigSync(t *testing.T) { + org := "FedeDP" + repo := "GhEnvSet" + ctx := context.Background() + + conf, err := FromData(testYAML) + assert.NoError(t, err) + + factory := poiana.NewMockServiceFactory() + provider, err := poiana.NewMockSecretsProvider(map[string]string{"orgSecret0": "orgValue0", "repoSecret0": "repoValue0", "repoSecret1": "repoValue1", "repoSecret2": "repoValue2"}) + assert.NoError(t, err) + + err = conf.Sync(factory, provider, false) + assert.NoError(t, err) + + // Org + // Check variables + orgVars := factory.NewOrgVariableService(org) + vars, _, err := orgVars.ListVariables(ctx, nil) + assert.NoError(t, err) + assert.Equal(t, vars.TotalCount, len(conf.Orgs[org].Actions.Variables)) + for _, v := range vars.Variables { + assert.Equal(t, conf.Orgs[org].Actions.Variables[v.Name], v.Value) + } + + // Check secrets + orgSecs := factory.NewOrgSecretService(org) + secs, _, err := orgSecs.ListSecrets(ctx, nil) + assert.NoError(t, err) + assert.Equal(t, secs.TotalCount, len(conf.Orgs[org].Actions.Secrets)) + for _, sec := range secs.Secrets { + assert.Contains(t, conf.Orgs[org].Actions.Secrets, sec.Name) + } + + // Repo + // Check variables + repoVars := factory.NewRepoVariableService(org, repo) + vars, _, err = repoVars.ListVariables(ctx, nil) + assert.NoError(t, err) + assert.Equal(t, vars.TotalCount, len(conf.Orgs[org].Repos[repo].Actions.Variables)) + for _, v := range vars.Variables { + assert.Equal(t, conf.Orgs[org].Repos[repo].Actions.Variables[v.Name], v.Value) + } + + // Check secrets + repoSecs := factory.NewRepoSecretService(org, repo) + secs, _, err = repoSecs.ListSecrets(ctx, nil) + assert.NoError(t, err) + assert.Equal(t, secs.TotalCount, len(conf.Orgs[org].Repos[repo].Actions.Secrets)) + for _, sec := range secs.Secrets { + assert.Contains(t, conf.Orgs[org].Repos[repo].Actions.Secrets, sec.Name) + } +}