third-party-mirrors/navidrome
1
0
Fork 0
mirror of https://github.com/navidrome/navidrome.git synced 2025-06-05 18:03:10 +03:00
Code Issues Packages Projects Releases Wiki Activity

Embed Last.FM error responses, making the tests faster

Browse Source
This commit is contained in:
Deluan 2021-05-27 21:04:03 -04:00
parent 89b12b34be
commit ed01946ace
3 changed files with 13 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
core/agents/lastfm_test.go
View File

@ -1,8 +1,10 @@
package agents package agents
import ( import (
"bytes"
"context" "context"
"errors" "errors"
"io/ioutil"
"net/http" "net/http"
"os" "os"
@ -14,6 +16,11 @@ import (
. "github.com/onsi/gomega" . "github.com/onsi/gomega"
) )
const (
lastfmError3 = `{"error":3,"message":"Invalid Method - No method with that name in this package","links":[]}`
lastfmError6 = `{"error":6,"message":"The artist you supplied could not be found","links":[]}`
)
var _ = Describe("lastfmAgent", func() { var _ = Describe("lastfmAgent", func() {
Describe("lastFMConstructor", func() { Describe("lastFMConstructor", func() {
It("uses default api key and language if not configured", func() { It("uses default api key and language if not configured", func() {
@ -59,8 +66,7 @@ var _ = Describe("lastfmAgent", func() {
}) })
It("returns an error if Last.FM call returns an error", func() { It("returns an error if Last.FM call returns an error", func() {
f, _ := os.Open("tests/fixtures/lastfm.artist.error3.json") httpClient.Res = http.Response{Body: ioutil.NopCloser(bytes.NewBufferString(lastfmError3)), StatusCode: 200}
httpClient.Res = http.Response{Body: f, StatusCode: 200}
_, err := agent.GetBiography("123", "U2", "mbid-1234") _, err := agent.GetBiography("123", "U2", "mbid-1234")
Expect(err).To(HaveOccurred()) Expect(err).To(HaveOccurred())
Expect(httpClient.RequestCount).To(Equal(1)) Expect(httpClient.RequestCount).To(Equal(1))
@ -76,8 +82,7 @@ var _ = Describe("lastfmAgent", func() {
Expect(httpClient.SavedRequest.URL.Query().Get("mbid")).To(BeEmpty()) Expect(httpClient.SavedRequest.URL.Query().Get("mbid")).To(BeEmpty())
}) })
It("calls again when last.fm returns an error 6", func() { It("calls again when last.fm returns an error 6", func() {
f, _ := os.Open("tests/fixtures/lastfm.artist.error6.json") httpClient.Res = http.Response{Body: ioutil.NopCloser(bytes.NewBufferString(lastfmError6)), StatusCode: 200}
httpClient.Res = http.Response{Body: f, StatusCode: 200}
_, _ = agent.GetBiography("123", "U2", "mbid-1234") _, _ = agent.GetBiography("123", "U2", "mbid-1234")
Expect(httpClient.RequestCount).To(Equal(2)) Expect(httpClient.RequestCount).To(Equal(2))
Expect(httpClient.SavedRequest.URL.Query().Get("mbid")).To(BeEmpty()) Expect(httpClient.SavedRequest.URL.Query().Get("mbid")).To(BeEmpty())
@ -115,8 +120,7 @@ var _ = Describe("lastfmAgent", func() {
}) })
It("returns an error if Last.FM call returns an error", func() { It("returns an error if Last.FM call returns an error", func() {
f, _ := os.Open("tests/fixtures/lastfm.artist.error3.json") httpClient.Res = http.Response{Body: ioutil.NopCloser(bytes.NewBufferString(lastfmError3)), StatusCode: 200}
httpClient.Res = http.Response{Body: f, StatusCode: 200}
_, err := agent.GetSimilar("123", "U2", "mbid-1234", 2) _, err := agent.GetSimilar("123", "U2", "mbid-1234", 2)
Expect(err).To(HaveOccurred()) Expect(err).To(HaveOccurred())
Expect(httpClient.RequestCount).To(Equal(1)) Expect(httpClient.RequestCount).To(Equal(1))
@ -132,8 +136,7 @@ var _ = Describe("lastfmAgent", func() {
Expect(httpClient.SavedRequest.URL.Query().Get("mbid")).To(BeEmpty()) Expect(httpClient.SavedRequest.URL.Query().Get("mbid")).To(BeEmpty())
}) })
It("calls again when last.fm returns an error 6", func() { It("calls again when last.fm returns an error 6", func() {
f, _ := os.Open("tests/fixtures/lastfm.artist.error6.json") httpClient.Res = http.Response{Body: ioutil.NopCloser(bytes.NewBufferString(lastfmError6)), StatusCode: 200}
httpClient.Res = http.Response{Body: f, StatusCode: 200}
_, _ = agent.GetSimilar("123", "U2", "mbid-1234", 2) _, _ = agent.GetSimilar("123", "U2", "mbid-1234", 2)
Expect(httpClient.RequestCount).To(Equal(2)) Expect(httpClient.RequestCount).To(Equal(2))
Expect(httpClient.SavedRequest.URL.Query().Get("mbid")).To(BeEmpty()) Expect(httpClient.SavedRequest.URL.Query().Get("mbid")).To(BeEmpty())
@ -171,8 +174,7 @@ var _ = Describe("lastfmAgent", func() {
}) })
It("returns an error if Last.FM call returns an error", func() { It("returns an error if Last.FM call returns an error", func() {
f, _ := os.Open("tests/fixtures/lastfm.artist.error3.json") httpClient.Res = http.Response{Body: ioutil.NopCloser(bytes.NewBufferString(lastfmError3)), StatusCode: 200}
httpClient.Res = http.Response{Body: f, StatusCode: 200}
_, err := agent.GetTopSongs("123", "U2", "mbid-1234", 2) _, err := agent.GetTopSongs("123", "U2", "mbid-1234", 2)
Expect(err).To(HaveOccurred()) Expect(err).To(HaveOccurred())
Expect(httpClient.RequestCount).To(Equal(1)) Expect(httpClient.RequestCount).To(Equal(1))
@ -188,8 +190,7 @@ var _ = Describe("lastfmAgent", func() {
Expect(httpClient.SavedRequest.URL.Query().Get("mbid")).To(BeEmpty()) Expect(httpClient.SavedRequest.URL.Query().Get("mbid")).To(BeEmpty())
}) })
It("calls again when last.fm returns an error 6", func() { It("calls again when last.fm returns an error 6", func() {
f, _ := os.Open("tests/fixtures/lastfm.artist.error6.json") httpClient.Res = http.Response{Body: ioutil.NopCloser(bytes.NewBufferString(lastfmError6)), StatusCode: 200}
httpClient.Res = http.Response{Body: f, StatusCode: 200}
_, _ = agent.GetTopSongs("123", "U2", "mbid-1234", 2) _, _ = agent.GetTopSongs("123", "U2", "mbid-1234", 2)
Expect(httpClient.RequestCount).To(Equal(2)) Expect(httpClient.RequestCount).To(Equal(2))
Expect(httpClient.SavedRequest.URL.Query().Get("mbid")).To(BeEmpty()) Expect(httpClient.SavedRequest.URL.Query().Get("mbid")).To(BeEmpty())

1
tests/fixtures/lastfm.artist.error3.json vendored
View File

@ -1 +0,0 @@
{"error":3,"message":"Invalid Method - No method with that name in this package","links":[]}

1
tests/fixtures/lastfm.artist.error6.json vendored
View File

@ -1 +0,0 @@
{"error":6,"message":"The artist you supplied could not be found","links":[]}