API: Fix status code when starring already starred dashboard (#63478)

Fixes #63133

pkg/services/star/starimpl/sqlx_store.go

@@ -5,12 +5,14 @@ import (
 	"database/sql"
 	"errors"
 
+	"github.com/grafana/grafana/pkg/infra/db"
 	"github.com/grafana/grafana/pkg/services/sqlstore/session"
 	"github.com/grafana/grafana/pkg/services/star"
 )
 
 type sqlxStore struct {
 	sess *session.SessionDB
+	db   db.DB
 }
 
 func (s *sqlxStore) Get(ctx context.Context, query *star.IsStarredByUserQuery) (bool, error) {
@@ -31,6 +33,9 @@ func (s *sqlxStore) Insert(ctx context.Context, cmd *star.StarDashboardCommand)
 		DashboardID: cmd.DashboardID,
 	}
 	_, err := s.sess.NamedExec(ctx, `INSERT INTO star (user_id, dashboard_id) VALUES (:user_id, :dashboard_id)`, entity)
+	if s.db.GetDialect().IsUniqueConstraintViolation(err) {
+		return nil
+	}
 	if err != nil {
 		return err
 	}

pkg/services/star/starimpl/sqlx_store_test.go

@@ -11,6 +11,6 @@ func TestIntegrationSQLxUserStarsDataAccess(t *testing.T) {
 		t.Skip("skipping integration test")
 	}
 	testIntegrationUserStarsDataAccess(t, func(ss db.DB) store {
-		return &sqlxStore{sess: ss.GetSqlxSession()}
+		return &sqlxStore{sess: ss.GetSqlxSession(), db: ss}
 	})
 }

pkg/services/star/starimpl/star.go

@@ -18,6 +18,7 @@ func ProvideService(db db.DB, cfg *setting.Cfg) star.Service {
 		return &Service{
 			store: &sqlxStore{
 				sess: db.GetSqlxSession(),
+				db:   db,
 			},
 		}
 	}

pkg/services/star/starimpl/xorm_store.go

@@ -35,6 +35,10 @@ func (s *sqlStore) Insert(ctx context.Context, cmd *star.StarDashboardCommand) e
 		}
 
 		_, err := sess.Insert(&entity)
+		if s.db.GetDialect().IsUniqueConstraintViolation(err) {
+			return nil
+		}
+
 		return err
 	})
 }