Fixed invite email (#66)

Added error when starting test environment without test email

Author: aunefyren

config/config.go

@@ -67,6 +67,8 @@ func GetConfig() (*models.ConfigStruct, error) {
 		// Set new value
 		config.PoenskelistenEnvironment = "production"
 		anythingChanged = true
+	} else if config.PoenskelistenEnvironment == "test" && config.PoenskelistenTestEmail == "" {
+		return nil, errors.New("Pønskelisten environment is set to 'test', but no test e-mail is configured.")
 	}
 
 	if config.Timezone == "" {

controllers/user.go

@@ -167,8 +167,8 @@ func RegisterUser(context *gin.Context) {
 	}
 
 	// Create user in DB
-	record := database.Instance.Create(&user)
-	if record.Error != nil {
+	user, err = database.CreateUserInDB(user)
+	if err != nil {
 		log.Println("Failed to get create user. Error: " + err.Error())
 		context.JSON(http.StatusInternalServerError, gin.H{"error": "Failed to get create user."})
 		context.Abort()
@@ -200,7 +200,6 @@ func RegisterUser(context *gin.Context) {
 
 	// Return response
 	context.JSON(http.StatusCreated, gin.H{"message": "User created!"})
-
 }
 
 func GetUser(context *gin.Context) {

database/user.go

@@ -3,6 +3,7 @@ package database
 import (
 	"aunefyren/poenskelisten/models"
 	"errors"
+	"log"
 	"strings"
 	"time"
 
@@ -217,3 +218,15 @@ func UpdateUserInDB(userOriginal models.User) (user models.User, err error) {
 
 	return
 }
+
+func CreateUserInDB(userRequest models.User) (user models.User, err error) {
+	user = userRequest
+
+	record := Instance.Create(&user)
+	if record.Error != nil {
+		log.Println("Failed to create user in DB. Error: " + record.Error.Error())
+		return user, errors.New("Failed to create user in DB.")
+	}
+
+	return
+}

utilities/smtp.go

@@ -10,7 +10,6 @@ import (
 )
 
 func SendSMTPVerificationEmail(user models.User) error {
-
 	// Get configuration
 	config, err := config.GetConfig()
 	if err != nil {
@@ -38,11 +37,9 @@ func SendSMTPVerificationEmail(user models.User) error {
 	}
 
 	return nil
-
 }
 
 func SendSMTPResetEmail(user models.User) error {
-
 	// Get configuration
 	config, err := config.GetConfig()
 	if err != nil {
@@ -72,11 +69,9 @@ func SendSMTPResetEmail(user models.User) error {
 	}
 
 	return nil
-
 }
 
 func SendSMTPDeletedClaimedWish(user models.User, wish models.WishObject, wishlist models.WishlistUser) error {
-
 	// Get configuration
 	config, err := config.GetConfig()
 	if err != nil {
@@ -106,5 +101,4 @@ func SendSMTPDeletedClaimedWish(user models.User, wish models.WishObject, wishli
 	}
 
 	return nil
-
 }