Funkwhale/ultrasonic-app-subsonic-android
1
0
Fork 0
mirror of https://github.com/ultrasonic/ultrasonic synced 2025-02-18 04:30:48 +01:00
Code Issues Releases Wiki Activity

Merge pull request #74 from ultrasonic/add-delete-share

Browse Source 
Add delete share
This commit is contained in:
Yahor Berdnikau 2017-11-19 15:04:34 +01:00 committed by GitHub
commit 81bb21c841
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 49 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
subsonic-api/src/integrationTest/kotlin/org/moire/ultrasonic/api/subsonic/SubsonicApiDeleteShareTest.kt Normal file
View File

@ -0,0 +1,33 @@
package org.moire.ultrasonic.api.subsonic
import org.junit.Test
/**
* Integration test for [SubsonicAPIDefinition.deleteShare] call.
*/
class SubsonicApiDeleteShareTest : SubsonicAPIClientTest() {
@Test
fun `Should handle error response`() {
checkErrorCallParsed(mockWebServerRule) {
client.api.deleteShare(123).execute()
}
}
@Test
fun `Should handle ok response`() {
mockWebServerRule.enqueueResponse("ping_ok.json")
val response = client.api.deleteShare(12).execute()
assertResponseSuccessful(response)
}
@Test
fun `Should pass id in request params`() {
val id = 224L
mockWebServerRule.assertRequestParam(expectedParam = "id=$id") {
client.api.deleteShare(id).execute()
}
}
}

3
subsonic-api/src/main/kotlin/org/moire/ultrasonic/api/subsonic/SubsonicAPIDefinition.kt
View File

@ -204,6 +204,9 @@ interface SubsonicAPIDefinition {
@Query("description") description: String? = null, @Query("description") description: String? = null,
@Query("expires") expires: Long? = null): Call<SharesResponse> @Query("expires") expires: Long? = null): Call<SharesResponse>
@GET("deleteShare.view")
fun deleteShare(@Query("id") id: Long): Call<SubsonicResponse>
@GET("getGenres.view") @GET("getGenres.view")
fun getGenres(): Call<GenresResponse> fun getGenres(): Call<GenresResponse>

37
ultrasonic/src/main/java/org/moire/ultrasonic/service/RESTMusicService.java
View File

@ -1343,31 +1343,20 @@ public class RESTMusicService implements MusicService
return APIShareConverter.toDomainEntitiesList(response.body().getShares()); return APIShareConverter.toDomainEntitiesList(response.body().getShares());
} }
@Override @Override
public void deleteShare(String id, Context context, ProgressListener progressListener) throws Exception public void deleteShare(String id,
{ Context context,
checkServerVersion(context, "1.6", "Shares not supported."); ProgressListener progressListener) throws Exception {
if (id == null) {
throw new IllegalArgumentException("Id is null!");
}
Long shareId = Long.valueOf(id);
HttpParams params = new BasicHttpParams(); updateProgressListener(progressListener, R.string.parser_reading);
HttpConnectionParams.setSoTimeout(params, SOCKET_READ_TIMEOUT_GET_RANDOM_SONGS); Response<SubsonicResponse> response = subsonicAPIClient.getApi()
.deleteShare(shareId).execute();
List<String> parameterNames = new ArrayList<String>(); checkResponseSuccessful(response);
List<Object> parameterValues = new ArrayList<Object>(); }
parameterNames.add("id");
parameterValues.add(id);
Reader reader = getReader(context, progressListener, "deleteShare", params, parameterNames, parameterValues);
try
{
new ErrorParser(context).parse(reader);
}
finally
{
Util.close(reader);
}
}
@Override @Override
public void updateShare(String id, String description, Long expires, Context context, ProgressListener progressListener) throws Exception public void updateShare(String id, String description, Long expires, Context context, ProgressListener progressListener) throws Exception