Skip to content
This repository was archived by the owner on Feb 4, 2025. It is now read-only.

Commit 251fc82

Browse files
authored
Merge pull request #3009 from wordpress-mobile/issue/capture-post-media-error
Add log lines to capture the string error message
2 parents 8d0c501 + 180ec68 commit 251fc82

File tree

3 files changed

+7
-0
lines changed

3 files changed

+7
-0
lines changed

example/src/test/java/org/wordpress/android/fluxc/upload/MediaUploadModelTest.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
11
package org.wordpress.android.fluxc.upload;
22

33
import android.text.TextUtils;
4+
import android.util.Log;
45

56
import org.junit.Test;
7+
import org.mockito.Mockito;
68
import org.wordpress.android.fluxc.model.MediaUploadModel;
79
import org.wordpress.android.fluxc.store.MediaStore.MediaError;
810
import org.wordpress.android.fluxc.store.MediaStore.MediaErrorType;
@@ -38,6 +40,7 @@ public void testEquals() {
3840

3941
@Test
4042
public void testMediaError() {
43+
Mockito.mockStatic(Log.class);
4144
MediaUploadModel mediaUploadModel = new MediaUploadModel(1);
4245

4346
assertNull(mediaUploadModel.getMediaError());

fluxc/src/main/java/org/wordpress/android/fluxc/store/MediaStore.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
import org.wordpress.android.fluxc.utils.MediaUtils;
3535
import org.wordpress.android.fluxc.utils.MimeType;
3636
import org.wordpress.android.util.AppLog;
37+
import org.wordpress.android.util.AppLog.T;
3738

3839
import java.io.IOException;
3940
import java.net.ConnectException;
@@ -528,6 +529,7 @@ public static MediaErrorType fromHttpStatusCode(int code) {
528529
@NonNull
529530
public static MediaErrorType fromString(@Nullable String string) {
530531
if (string != null) {
532+
AppLog.e(T.API, "Media Error Type Conversion From: " + string);
531533
for (MediaErrorType v : MediaErrorType.values()) {
532534
if (string.equalsIgnoreCase(v.name())) {
533535
return v;

fluxc/src/main/java/org/wordpress/android/fluxc/store/PostStore.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@
5353
import org.wordpress.android.fluxc.store.ListStore.ListItemsRemovedPayload;
5454
import org.wordpress.android.fluxc.utils.ObjectsUtils;
5555
import org.wordpress.android.util.AppLog;
56+
import org.wordpress.android.util.AppLog.T;
5657
import org.wordpress.android.util.DateTimeUtils;
5758

5859
import java.util.ArrayList;
@@ -435,6 +436,7 @@ public enum PostErrorType {
435436

436437
public static PostErrorType fromString(String string) {
437438
if (string != null) {
439+
AppLog.e(T.API, "Post Error Type Conversion From: " + string);
438440
for (PostErrorType v : PostErrorType.values()) {
439441
if (string.equalsIgnoreCase(v.toString())) {
440442
return v;

0 commit comments

Comments
 (0)