wkulhanek/bonob
1
0
Fork 0
mirror of https://github.com/wkulhanek/bonob.git synced 2025-12-21 17:33:29 +01:00
Code Issues 1 Packages Projects Releases Wiki Activity

Subsonic scrobble is a GET request...

Browse Source
This commit is contained in:
simojenki
2021-03-17 19:06:25 +11:00
parent c9543be3e7
commit b89240a4db
2 changed files with 17 additions and 40 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
src/navidrome.ts
View File

@@ -244,32 +244,6 @@ export class Navidrome implements MusicService {
else return response;
});
post = async (
{ username, password }: Credentials,
path: string,
q: {} = {},
config: AxiosRequestConfig | undefined = {}
) =>
axios
.post(`${this.url}${path}`, {
params: {
...q,
u: username,
...t_and_s(password),
v: "1.16.1",
c: "bonob",
},
headers: {
"User-Agent": "bonob",
},
...config,
})
.then((response) => {
if (response.status != 200 && response.status != 206)
throw `Navidrome failed with a ${response.status}`;
else return response;
});
getJSON = async <T>(
{ username, password }: Credentials,
path: string,
@@ -548,7 +522,10 @@ export class Navidrome implements MusicService {
},
scrobble: async (id: string) =>
navidrome
.post(credentials, `/rest/scrobble`, { id, submission: true })
.get(credentials, `/rest/scrobble`, {
id,
submission: true,
})
.then((_) => true)
.catch(() => false),
};

16
tests/navidrome.test.ts
View File

@@ -2170,9 +2170,9 @@ describe("Navidrome", () => {
it("should return true", async () => {
const id = uuid();
mockGET.mockImplementationOnce(() => Promise.resolve(ok(PING_OK)));
mockPOST.mockImplementationOnce(() => Promise.resolve(ok(EMPTY)));
mockGET
.mockImplementationOnce(() => Promise.resolve(ok(PING_OK)))
.mockImplementationOnce(() => Promise.resolve(ok(EMPTY)));
const result = await navidrome
.generateToken({ username, password })
@@ -2182,7 +2182,7 @@ describe("Navidrome", () => {
expect(result).toEqual(true);
expect(mockPOST).toHaveBeenCalledWith(`${url}/rest/scrobble`, {
expect(mockGET).toHaveBeenCalledWith(`${url}/rest/scrobble`, {
params: {
id,
submission: true,
@@ -2197,9 +2197,9 @@ describe("Navidrome", () => {
it("should return false", async () => {
const id = uuid();
mockGET.mockImplementationOnce(() => Promise.resolve(ok(PING_OK)));
mockPOST.mockImplementationOnce(() =>
mockGET
.mockImplementationOnce(() => Promise.resolve(ok(PING_OK)))
.mockImplementationOnce(() =>
Promise.resolve({
status: 500,
data: {},
@@ -2214,7 +2214,7 @@ describe("Navidrome", () => {
expect(result).toEqual(false);
expect(mockPOST).toHaveBeenCalledWith(`${url}/rest/scrobble`, {
expect(mockGET).toHaveBeenCalledWith(`${url}/rest/scrobble`, {
params: {
id,
submission: true,