mirror/opencloud
1
0
Fork 0
mirror of https://github.com/opencloud-eu/opencloud.git synced 2026-05-25 00:46:37 -04:00
Code Issues Packages Projects Releases Wiki Activity

test(search): cover audio.artist instead of Title for case preservation

Browse Source 
The FIXME pointed at #2632 (dotted keys in KQL property restrictions),
which is now merged. Use audio.artist — the originally intended target
field for this regression — so the test matches its name: a nested
string field that is not on the lowercase allowlist.
This commit is contained in:
Dominik Schmidt
2026-04-22 09:51:37 +02:00
parent 796e5fd373
commit 87b1f6f630
1 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
services/search/pkg/bleve/backend_test.go
View File

@@ -138,16 +138,15 @@ var _ = Describe("Bleve", func() {
assertDocCount(rootResource.ID, "Size:>100000", 0)
})
// FIXME: switch Title to audio.artist once
// https://github.com/opencloud-eu/opencloud/pull/2632 lands
// (KQL grammar then accepts dotted keys).
It("preserves value case for fields not explicitly marked lowercase", func() {
parentResource.Document.Title = "Some Title"
parentResource.Document.Audio = &libregraph.Audio{
Artist: libregraph.PtrString("Some Artist"),
}
err := eng.Upsert(parentResource.ID, parentResource)
Expect(err).ToNot(HaveOccurred())
assertDocCount(rootResource.ID, `Title:"Some Title"`, 1)
assertDocCount(rootResource.ID, `Title:"some title"`, 0)
assertDocCount(rootResource.ID, `audio.artist:"Some Artist"`, 1)
assertDocCount(rootResource.ID, `audio.artist:"some artist"`, 0)
})
})