feat(database): Improve error returned to user

Signed-off-by: Jo Vandeginste <Jo.Vandeginste@kuleuven.be>

pkg/database/gorm.go

@@ -45,7 +45,8 @@ func Connect(driver, dsn string, debug bool, logger *slog.Logger) (*gorm.DB, err
 	}
 
 	db, err := gorm.Open(d, &gorm.Config{
-		Logger: gormLogger,
+		Logger:         gormLogger,
+		TranslateError: true,
 	})
 	if err != nil {
 		return nil, err

pkg/database/user_test.go

@@ -220,7 +220,7 @@ func TestDatabaseUserCreateDoubleUsername(t *testing.T) {
 
 	err := defaultUser().Create(db)
 	require.Error(t, err)
-	require.Contains(t, err.Error(), "UNIQUE constraint failed")
+	assert.ErrorIs(t, err, gorm.ErrDuplicatedKey)
 
 	users, err := GetUsers(db)
 	require.NoError(t, err)

pkg/database/workouts.go

@@ -22,7 +22,10 @@ import (
 	"gorm.io/gorm/clause"
 )
 
-var ErrInvalidData = errors.New("could not convert data to a GPX structure")
+var (
+	ErrInvalidData          = errors.New("could not convert data to a GPX structure")
+	ErrWorkoutAlreadyExists = errors.New("user already has workout with exact start time")
+)
 
 type Workout struct {
 	Model
@@ -471,6 +474,15 @@ func (w *Workout) Delete(db *gorm.DB) error {
 }
 
 func (w *Workout) Create(db *gorm.DB) error {
+	err := w.create(db)
+	if errors.Is(err, gorm.ErrDuplicatedKey) {
+		return ErrWorkoutAlreadyExists
+	}
+
+	return err
+}
+
+func (w *Workout) create(db *gorm.DB) error {
 	if w.Data == nil {
 		return ErrInvalidData
 	}
@@ -479,6 +491,15 @@ func (w *Workout) Create(db *gorm.DB) error {
 }
 
 func (w *Workout) Save(db *gorm.DB) error {
+	err := w.save(db)
+	if errors.Is(err, gorm.ErrDuplicatedKey) {
+		return ErrWorkoutAlreadyExists
+	}
+
+	return err
+}
+
+func (w *Workout) save(db *gorm.DB) error {
 	if w.Data == nil {
 		return ErrInvalidData
 	}