Use gorm Unscoped to force a hard delete instead of raw execing some SQL

Signed-off-by: Ying Li <[email protected]>

server/storage/sql_models.go

@@ -14,7 +14,6 @@ type TUFFile struct {
 
 // TableName sets a specific table name for TUFFile
 func (g TUFFile) TableName() string {
-	// NOTE: if this value changes, please also change it in SQLStorage.Delete
 	return "tuf_files"
 }
 

server/storage/sqldb.go

@@ -153,11 +153,10 @@ func isReadErr(q *gorm.DB, row TUFFile) error {
 	return nil
 }
 
-// Delete deletes all the records for a specific GUN - we have to do a raw execute, because
-// otherwise GORM will do a soft delete, because we have CreatedAt, ModifiedAt, and DeletedAt
-// columns (and we actually use CreatedAt and ModifiedAt)
+// Delete deletes all the records for a specific GUN - we have to do a hard delete using Unscoped
+// otherwise we can't insert for that GUN again
 func (db *SQLStorage) Delete(gun string) error {
-	return db.Exec("DELETE FROM tuf_files WHERE gun = ?", gun).Error
+	return db.Unscoped().Where(&TUFFile{Gun: gun}).Delete(TUFFile{}).Error
 }
 
 // GetKey returns the Public Key data for a gun+role

server/storage/sqldb_test.go

@@ -283,7 +283,7 @@ func TestSQLDBCheckHealthTableMissing(t *testing.T) {
 	require.Error(t, err, "Cannot access table:")
 }
 
-// TestSQLDBCheckHealthDBCOnnection asserts that if the DB is not connectable, the
+// TestSQLDBCheckHealthDBConnection asserts that if the DB is not connectable, the
 // health check fails.
 func TestSQLDBCheckHealthDBConnectionFail(t *testing.T) {
 	dbStore, cleanup := sqldbSetup(t)