Fix wrong table name

models/auth/oauth2_test.go

@@ -13,7 +13,12 @@ import (
 	"github.com/stretchr/testify/assert"
 )
 
-//////////////////// Application
+func TestApplicationTableName(t *testing.T) {
+	e := unittest.GetXORMEngine()
+	tableInfo, err := e.TableInfo(new(auth_model.OAuth2Application))
+	assert.NoError(t, err)
+	assert.EqualValues(t, "oauth2_application", tableInfo.Name)
+}
 
 func TestOAuth2Application_GenerateClientSecret(t *testing.T) {
 	assert.NoError(t, unittest.PrepareTestDatabase())

models/migrations/migrations.go

@@ -584,6 +584,8 @@ var migrations = []Migration{
 	NewMigration("Add missing field of commit status summary table", v1_23.AddCommitStatusSummary2),
 	// v297 -> v298
 	NewMigration("Add everyone_access_mode for repo_unit", v1_23.AddRepoUnitEveryoneAccessMode),
+	// v298 -> v299
+	NewMigration("Drop wrongly created table o_auth2_application", v1_23.DropWronglyCreatedTable),
 }
 
 // GetCurrentDBVersion returns the current db version

models/migrations/v1_18/v230.go

@@ -10,8 +10,9 @@ import (
 // AddConfidentialColumnToOAuth2ApplicationTable: add ConfidentialClient column, setting existing rows to true
 func AddConfidentialClientColumnToOAuth2ApplicationTable(x *xorm.Engine) error {
 	type OAuth2Application struct {
+		ID                 int64
 		ConfidentialClient bool `xorm:"NOT NULL DEFAULT TRUE"`
 	}
 
-	return x.Sync(new(OAuth2Application))
+	return x.Table("oauth2_application").Sync(new(OAuth2Application))
 }

models/migrations/v1_18/v230_test.go

@@ -11,12 +11,16 @@ import (
 	"github.com/stretchr/testify/assert"
 )
 
-func Test_AddConfidentialClientColumnToOAuth2ApplicationTable(t *testing.T) {
-	// premigration
-	type OAuth2Application struct {
-		ID int64
-	}
+// premigration
+type OAuth2Application struct {
+	ID int64
+}
 
+func (OAuth2Application) TableName() string {
+	return "oauth2_application"
+}
+
+func Test_AddConfidentialClientColumnToOAuth2ApplicationTable(t *testing.T) {
 	// Prepare and load the testing database
 	x, deferable := base.PrepareTestEnv(t, 0, new(OAuth2Application))
 	defer deferable()
@@ -36,7 +40,7 @@ func Test_AddConfidentialClientColumnToOAuth2ApplicationTable(t *testing.T) {
 	}
 
 	got := []ExpectedOAuth2Application{}
-	if err := x.Table("o_auth2_application").Select("id, confidential_client").Find(&got); !assert.NoError(t, err) {
+	if err := x.Table("oauth2_application").Select("id, confidential_client").Find(&got); !assert.NoError(t, err) {
 		return
 	}
 

models/migrations/v1_23/v298.go

@@ -0,0 +1,10 @@
+// Copyright 2024 The Gitea Authors. All rights reserved.
+// SPDX-License-Identifier: MIT
+
+package v1_23 //nolint
+
+import "xorm.io/xorm"
+
+func DropWronglyCreatedTable(x *xorm.Engine) error {
+	return x.DropTables("o_auth2_application")
+}