Mirrors/photoprism
1
0
Fork 0
mirror of https://github.com/photoprism/photoprism.git synced 2026-01-23 02:24:24 +00:00
Code Issues Projects Releases Wiki Activity

Tests: refactor restorePhoto to prevent TakenAt fields doubling up and breaking Postgres

Browse source
This commit is contained in:
Keith Martin 2025-12-03 16:28:27 +10:00
parent 1da065c915
commit 1427b82652
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
internal/photoprism/batch/save_test.go
View file

@ -3,6 +3,7 @@ package batch
import (
"strings"
"testing"
"time"
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
@ -333,7 +334,8 @@ func restorePhoto(t *testing.T, photoUID string, values entity.Values) {
delete(values, k)
}
}
if err := entity.Db().Model(&entity.Photo{}).Where("photo_uid = ?", photoUID).Updates(values).Error; err != nil {
// The TakenAt and TakenAtLocal are there to prevent the Photo.BeforeSave from adding these as required update fields and breaking in Postgres.
if err := entity.Db().Model(&entity.Photo{TakenAt: time.Now(), TakenAtLocal: time.Now()}).Where("photo_uid = ?", photoUID).Updates(values).Error; err != nil {
t.Fatalf("failed to restore photo %s: %v", photoUID, err)
}
if len(detailUpdates) > 0 {