From 21306817c1eaebaf6abfe1c174d8c2ee9342866e Mon Sep 17 00:00:00 2001 From: Gil Casadevall Date: Wed, 22 Jun 2016 13:38:20 +0200 Subject: [PATCH] Improved error messages --- main.go | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/main.go b/main.go index fdc9417..2f194bc 100644 --- a/main.go +++ b/main.go @@ -9,10 +9,11 @@ import ( ) const ( - slackTokenLabel string = "KUBEBOT_SLACK_TOKEN" - slackChannelsLabel string = "KUBEBOT_SLACK_CHANNELS_IDS" - slackAdminsLabel string = "KUBEBOT_SLACK_ADMINS_NICKNAMES" - slackCommandsLabel string = "KUBEBOT_SLACK_VALID_COMMANDS" + slackTokenLabel string = "KUBEBOT_SLACK_TOKEN" + slackChannelsLabel string = "KUBEBOT_SLACK_CHANNELS_IDS" + slackAdminsLabel string = "KUBEBOT_SLACK_ADMINS_NICKNAMES" + slackCommandsLabel string = "KUBEBOT_SLACK_VALID_COMMANDS" + notDefinedErrorMessage string = "%s env variable not defined" ) var ( @@ -21,16 +22,16 @@ var ( func validateEnvParams() error { if os.Getenv(slackTokenLabel) == "" { - return errors.New("slackTokenLabel env variable not defined") + return errors.New(fmt.Sprintf(notDefinedErrorMessage, slackTokenLabel)) } if os.Getenv(slackChannelsLabel) == "" { - return errors.New("slackChannelsLabel env variable not defined") + return errors.New(fmt.Sprintf(notDefinedErrorMessage, slackChannelsLabel)) } if os.Getenv(slackAdminsLabel) == "" { - return errors.New("slackAdminsLabel env variable not defined") + return errors.New(fmt.Sprintf(notDefinedErrorMessage, slackAdminsLabel)) } if os.Getenv(slackCommandsLabel) == "" { - return errors.New("slackCommandsLabel env variable not defined") + return errors.New(fmt.Sprintf(notDefinedErrorMessage, slackCommandsLabel)) } return nil