Merge pull request #513 from traPtitech/fix/update-user-privilege

🐛 enable to update user privilege
traPtitech · Jan 5, 2024 · 7184525 · 7184525
2 parents a05162a + d39511f
commit 7184525
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/infra/db/model.go b/infra/db/model.go
@@ -66,7 +66,7 @@ type Provider struct {
 type User struct {
 	ID uuid.UUID `gorm:"type:char(36); primaryKey"`
 	// アプリの管理者かどうか
-	Privilege  bool `gorm:"<-:create; not null"` // Do not update
+	Privilege  bool `gorm:"not null"`
 	State      int
 	IcalSecret string   `gorm:"not null"`
 	Provider   Provider `gorm:"foreignKey:UserID; constraint:OnDelete:CASCADE;"`

diff --git a/infra/db/user_test.go b/infra/db/user_test.go
@@ -67,7 +67,7 @@ func Test_saveUser(t *testing.T) {
 		assert.Equal("hoge2", token.AccessToken)
 	})
 
-	t.Run("Can't update privilege", func(t *testing.T) {
+	t.Run("Update privilege", func(t *testing.T) {
 		u, err := getUser(r.db, user.ID)
 		assert.NoError(err)
 		assert.False(u.Privilege)
@@ -80,7 +80,7 @@ func Test_saveUser(t *testing.T) {
 
 		u, err = getUser(r.db, user.ID)
 		assert.NoError(err)
-		assert.False(u.Privilege)
+		assert.True(u.Privilege)
 	})
 }