beenull/photoprism
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Tests: Refactor tests to prevent nil pointer

Browse source
This commit is contained in:
theresa 2021-09-01 14:05:15 +02:00
parent fd785faf68
commit 995b87373c
2 changed files with 16 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
internal/entity/face_test.go
View file

@ -216,6 +216,7 @@ func TestFirstOrCreateFace(t *testing.T) {
func TestFindFace(t *testing.T) {
t.Run("existing face", func(t *testing.T) {
assert.NotNil(t, FindFace("VF7ANLDET2BKZNT4VQWJMMC6HBEFDOG7"))
assert.Equal(t, 3, FindFace("VF7ANLDET2BKZNT4VQWJMMC6HBEFDOG7").Samples)
})
t.Run("empty id", func(t *testing.T) {

19
internal/entity/marker_test.go
View file

@ -45,6 +45,9 @@ func TestMarker_SaveForm(t *testing.T) {
}
assert.NotEmpty(t, m.SubjectUID)
assert.NotNil(t, m.GetSubject())
assert.NotNil(t, FindMarker(9).GetSubject())
assert.NotNil(t, FindMarker(10).GetSubject())
assert.Equal(t, "Jane Doe", m.GetSubject().SubjectName)
assert.Equal(t, "Jane Doe", FindMarker(9).GetSubject().SubjectName)
assert.Equal(t, "Jane Doe", FindMarker(10).GetSubject().SubjectName)
@ -58,6 +61,9 @@ func TestMarker_SaveForm(t *testing.T) {
t.Fatal(err3)
}
assert.NotNil(t, FindMarker(8).GetSubject())
assert.NotNil(t, FindMarker(9).GetSubject())
assert.NotNil(t, FindMarker(10).GetSubject())
assert.Equal(t, "Franzilein", FindMarker(8).GetSubject().SubjectName)
assert.Equal(t, "Franzilein", FindMarker(9).GetSubject().SubjectName)
assert.Equal(t, "Franzilein", FindMarker(10).GetSubject().SubjectName)
@ -206,6 +212,11 @@ func TestMarker_ClearSubject(t *testing.T) {
assert.Equal(t, "jqy1y111h1njaaad", m2.SubjectUID)
assert.Equal(t, "jqy1y111h1njaaad", m3.SubjectUID)
assert.Equal(t, "jqy1y111h1njaaad", m4.SubjectUID)
assert.NotNil(t, m.GetFace())
assert.NotNil(t, m2.GetFace())
assert.NotNil(t, m3.GetFace())
assert.NotNil(t, m4.GetFace())
assert.NotNil(t, FindMarker(15).GetFace())
assert.Equal(t, "PI6A2XGOTUXEFI7CBF4KCI5I2I3JEJHS", m.GetFace().ID)
assert.Equal(t, "PI6A2XGOTUXEFI7CBF4KCI5I2I3JEJHS", m2.GetFace().ID)
assert.Equal(t, "PI6A2XGOTUXEFI7CBF4KCI5I2I3JEJHS", m3.GetFace().ID)
@ -219,10 +230,10 @@ func TestMarker_ClearSubject(t *testing.T) {
t.Fatal(err)
}
t.Log(FindMarker(18).FaceID)
t.Log(FindMarker(17).FaceID)
t.Log(FindMarker(16).FaceID)
t.Log(FindMarker(15).FaceID)
assert.NotNil(t, FindMarker(17))
assert.NotNil(t, FindMarker(16))
assert.NotNil(t, FindMarker(15))
assert.NotNil(t, FindFace("PI6A2XGOTUXEFI7CBF4KCI5I2I3JEJHS"))
assert.Empty(t, m.SubjectUID)
assert.Equal(t, "", FindMarker(17).SubjectUID)