fix defaulted github values not being processed before use in attest …

…github pr command

cmd/kosli/attestPRGithub.go

@@ -99,8 +99,8 @@ func newAttestGithubPRCmd(out io.Writer) *cobra.Command {
 		payload: PRAttestationPayload{
 			CommonAttestationPayload: &CommonAttestationPayload{},
 		},
-		retriever: new(ghUtils.GithubConfig),
 	}
+	githubFlagsValues := new(ghUtils.GithubFlagsTempValueHolder)
 	cmd := &cobra.Command{
 		Use:     "github [IMAGE-NAME | FILE-PATH | DIR-PATH]",
 		Aliases: []string{"gh"},
@@ -129,13 +129,15 @@ func newAttestGithubPRCmd(out io.Writer) *cobra.Command {
 
 		},
 		RunE: func(cmd *cobra.Command, args []string) error {
+			o.retriever = ghUtils.NewGithubConfig(githubFlagsValues.Token, githubFlagsValues.BaseURL,
+				githubFlagsValues.Org, githubFlagsValues.Repository)
 			return o.run(args)
 		},
 	}
 
 	ci := WhichCI()
 	addAttestationFlags(cmd, o.CommonAttestationOptions, o.payload.CommonAttestationPayload, ci)
-	addAttestationGithubFlags(cmd, o.getRetriever().(*ghUtils.GithubConfig), ci)
+	addAttestationGithubFlags(cmd, githubFlagsValues, ci)
 	cmd.Flags().BoolVar(&o.assert, "assert", false, assertPREvidenceFlag)
 
 	err := RequireFlags(cmd, []string{"flow", "trail", "name",

cmd/kosli/cli_utils.go

@@ -139,7 +139,6 @@ func DefaultValue(ci, flag string) string {
 					return strings.Replace(result, "commit", "commits", 1)
 				}
 			}
-
 			return result
 		}
 	}

cmd/kosli/cli_utils_test.go

@@ -81,6 +81,26 @@ func (suite *CliUtilsTestSuite) TestDefaultValue() {
 			},
 			want: "some-sha",
 		},
+		{
+			name: "Lookup default org for Github.",
+			args: args{
+				ci:               github,
+				flag:             "org",
+				envVars:          map[string]string{"GITHUB_REPOSITORY_OWNER": "cyber-dojo"},
+				unsetTestsEnvVar: true,
+			},
+			want: "cyber-dojo",
+		},
+		{
+			name: "Lookup default repository for Github.",
+			args: args{
+				ci:               github,
+				flag:             "repository",
+				envVars:          map[string]string{"GITHUB_REPOSITORY": "cyber-dojo/dashboard"},
+				unsetTestsEnvVar: true,
+			},
+			want: "cyber-dojo/dashboard",
+		},
 		{
 			name: "Lookup an existing default for Bitbucket.",
 			args: args{

cmd/kosli/flags.go

@@ -55,11 +55,11 @@ func addGitlabFlags(cmd *cobra.Command, gitlabConfig *gitlabUtils.GitlabConfig,
 	cmd.Flags().StringVar(&gitlabConfig.Repository, "repository", DefaultValue(ci, "repository"), repositoryFlag)
 }
 
-func addAttestationGithubFlags(cmd *cobra.Command, githubConfig *ghUtils.GithubConfig, ci string) {
-	cmd.Flags().StringVar(&githubConfig.Token, "github-token", "", githubTokenFlag)
-	cmd.Flags().StringVar(&githubConfig.Org, "github-org", DefaultValue(ci, "org"), githubOrgFlag)
-	cmd.Flags().StringVar(&githubConfig.Repository, "repository", DefaultValue(ci, "repository"), repositoryFlag)
-	cmd.Flags().StringVar(&githubConfig.BaseURL, "github-base-url", "", githubBaseURLFlag)
+func addAttestationGithubFlags(cmd *cobra.Command, githubFlagsValueHolder *ghUtils.GithubFlagsTempValueHolder, ci string) {
+	cmd.Flags().StringVar(&githubFlagsValueHolder.Token, "github-token", "", githubTokenFlag)
+	cmd.Flags().StringVar(&githubFlagsValueHolder.Org, "github-org", DefaultValue(ci, "org"), githubOrgFlag)
+	cmd.Flags().StringVar(&githubFlagsValueHolder.Repository, "repository", DefaultValue(ci, "repository"), repositoryFlag)
+	cmd.Flags().StringVar(&githubFlagsValueHolder.BaseURL, "github-base-url", "", githubBaseURLFlag)
 }
 
 func addAttestationAzureFlags(cmd *cobra.Command, azureConfig *azUtils.AzureConfig, ci string) {

cmd/kosli/testHelpers.go

@@ -347,3 +347,17 @@ func ReportServerArtifactToEnv(paths []string, envName string, t *testing.T) {
 	err := o.run([]string{envName})
 	require.NoError(t, err, "server env should be reported without error")
 }
+
+func SetEnvVars(envVars map[string]string, t *testing.T) {
+	for key, value := range envVars {
+		err := os.Setenv(key, value)
+		require.NoErrorf(t, err, "error setting env variable %s", key)
+	}
+}
+
+func UnSetEnvVars(envVars map[string]string, t *testing.T) {
+	for key := range envVars {
+		err := os.Unsetenv(key)
+		require.NoErrorf(t, err, "error unsetting env variable %s", key)
+	}
+}