Merge remote-tracking branch 'origin/main' into tinglwan-fix-miscellaneous-tests

Author: ting-lan-wang

oracle/clause_builder.go

@@ -300,7 +300,6 @@ func OnConflictClauseBuilder(c clause.Clause, builder clause.Builder) {
 			if len(conflictColumns) == 0 {
 				// If no columns specified, try to use primary key fields as default
 				if stmt.Schema == nil || len(stmt.Schema.PrimaryFields) == 0 {
-					stmt.AddError(fmt.Errorf("OnConflict requires either explicit columns or primary key fields"))
 					return
 				}
 				for _, primaryField := range stmt.Schema.PrimaryFields {

oracle/create.go

@@ -219,7 +219,6 @@ func buildBulkInsertPLSQL(db *gorm.DB, createValues clause.Values) {
 		conflictColumns := onConflict.Columns
 		if len(conflictColumns) == 0 {
 			if len(schema.PrimaryFields) == 0 {
-				db.AddError(fmt.Errorf("OnConflict requires either explicit columns or primary key fields"))
 				return
 			}
 			for _, primaryField := range schema.PrimaryFields {
@@ -255,7 +254,6 @@ func buildBulkMergePLSQL(db *gorm.DB, createValues clause.Values, onConflictClau
 	conflictColumns := onConflict.Columns
 	if len(conflictColumns) == 0 {
 		if schema == nil || len(schema.PrimaryFields) == 0 {
-			db.AddError(fmt.Errorf("OnConflict requires either explicit columns or primary key fields"))
 			return
 		}
 		for _, primaryField := range schema.PrimaryFields {

tests/joins_table_test.go

@@ -66,7 +66,6 @@ type PersonAddress struct {
 }
 
 func TestOverrideJoinTable(t *testing.T) {
-	t.Skip()
 	DB.Migrator().DropTable(&Person{}, &Address{}, &PersonAddress{})
 
 	if err := DB.SetupJoinTable(&Person{}, "Addresses", &PersonAddress{}); err != nil {
@@ -95,7 +94,7 @@ func TestOverrideJoinTable(t *testing.T) {
 		t.Fatalf("Should have one address left")
 	}
 
-	if DB.Find(&[]PersonAddress{}, "person_id = ?", person.ID).RowsAffected != 1 {
+	if DB.Find(&[]PersonAddress{}, "\"person_id\" = ?", person.ID).RowsAffected != 1 {
 		t.Fatalf("Should found one address")
 	}
 
@@ -113,7 +112,7 @@ func TestOverrideJoinTable(t *testing.T) {
 		t.Fatalf("Failed to find address, got error %v, length: %v", err, len(addresses3))
 	}
 
-	if DB.Unscoped().Find(&[]PersonAddress{}, "person_id = ?", person.ID).RowsAffected != 2 {
+	if DB.Unscoped().Find(&[]PersonAddress{}, "\"person_id\" = ?", person.ID).RowsAffected != 2 {
 		t.Fatalf("Should found soft deleted addresses with unscoped")
 	}