fix

Author: wxiaoguang

models/unittest/fixtures_loader.go

@@ -31,6 +31,7 @@ func (f *fixturesLoader) prepareFieldValue(v any) any {
 	}
 	return v
 }
+
 func (f *fixturesLoader) loadFixtures(file string) error {
 	data, err := os.ReadFile(file)
 	if err != nil {
@@ -51,10 +52,8 @@ func (f *fixturesLoader) loadFixtures(file string) error {
 
 	switch f.engine.Dialect().URI().DBType {
 	case schemas.MSSQL:
-		_, _ = f.engine.Exec("SET IDENTITY_INSERT [%s] ON", tableName)
-		defer func() {
-			_, _ = f.engine.Exec("SET IDENTITY_INSERT [%s] OFF", tableName)
-		}()
+		_, _ = f.engine.Exec(fmt.Sprintf("SET IDENTITY_INSERT `%s` ON", tableName))
+		// we can't revert this behavior, because some tests also depends on this, for example: Test_UpdateBadgeColName
 	}
 
 	var sqlBuf []byte

modules/system/appstate_test.go

@@ -36,8 +36,6 @@ func (*testItem2) Name() string {
 }
 
 func TestAppStateDB(t *testing.T) {
-	assert.NoError(t, unittest.PrepareTestDatabase())
-
 	as := &DBStore{}
 
 	item1 := new(testItem1)