test: add coverage for missing id parameter in GetCoverArt
Signed-off-by: Deluan <deluan@navidrome.org>
This commit is contained in:
@@ -45,12 +45,20 @@ var _ = Describe("MediaRetrievalController", func() {
|
|||||||
_, err := router.GetCoverArt(w, r)
|
_, err := router.GetCoverArt(w, r)
|
||||||
|
|
||||||
Expect(err).ToNot(HaveOccurred())
|
Expect(err).ToNot(HaveOccurred())
|
||||||
Expect(artwork.recvId).To(Equal("34"))
|
|
||||||
Expect(artwork.recvSize).To(Equal(128))
|
Expect(artwork.recvSize).To(Equal(128))
|
||||||
Expect(artwork.recvSquare).To(BeTrue())
|
Expect(artwork.recvSquare).To(BeTrue())
|
||||||
Expect(w.Body.String()).To(Equal(artwork.data))
|
Expect(w.Body.String()).To(Equal(artwork.data))
|
||||||
})
|
})
|
||||||
|
|
||||||
|
It("should return placeholder if id parameter is missing (mimicking Subsonic)", func() {
|
||||||
|
r := newGetRequest() // No id parameter
|
||||||
|
_, err := router.GetCoverArt(w, r)
|
||||||
|
|
||||||
|
Expect(err).To(BeNil())
|
||||||
|
Expect(artwork.recvId).To(BeEmpty())
|
||||||
|
Expect(w.Body.String()).To(Equal(artwork.data))
|
||||||
|
})
|
||||||
|
|
||||||
It("should fail when the file is not found", func() {
|
It("should fail when the file is not found", func() {
|
||||||
artwork.err = model.ErrNotFound
|
artwork.err = model.ErrNotFound
|
||||||
r := newGetRequest("id=34", "size=128", "square=true")
|
r := newGetRequest("id=34", "size=128", "square=true")
|
||||||
|
|||||||
Reference in New Issue
Block a user