android - 如何在抛出的异常中模拟 Crashlytics 静态方法

Question

我正在使用 MVP 架构来构建我的应用程序。我的演示者调用一个 DataManager，它负责从网络或数据库中获取数据。当我使用 RxJava 时，我在 Presenter 中订阅 Observers 并将适当的状态传递给 UI。我的服务层有 Android 上下文，它还会创建一个我自己类型的异常，它也有对上下文的引用。

if (isNetworkConnected()) {
                    final Call<ServiceResponse<AppVersion>> call = mService.getAppVersion();
                    try {
                        final Response<ServiceResponse<AppVersion>> response = call.execute();
                        if (response.isSuccessful()) {
                            final ServiceResponse<AppVersion> serviceResponse = response.body(); response.body();
                            if (serviceResponse.isSuccess()) {
                                subscriber.onNext(serviceResponse.getData());
                            } else {
                                subscriber.onError(new CustomException(mContext, response.code(), response.message(), serviceResponse.getErrorList()));
                            }
                        } else {
                            subscriber.onError(new CustomException(mContext, response.code(), response.message(), response.errorBody().string()));
                        }
                    } catch (IOException e) {
                        e.printStackTrace();
                        subscriber.onError(e);
                    } finally {
                        subscriber.onCompleted();
                    }
                } else {
                    subscriber.onError(new NoInternetException());
                }

我的 CustomException 还在 Crashlytics 中记录了崩溃。当我对该代码进行单元测试时，我收到 Crashlytics 未初始化的异常。所以我需要模拟Crashlytics的静态方法logException。但是我应该如何传递这个模拟对象，因为演示者不接受这个对象？

 public staticErrorType getErrorType(Throwable throwable) {
        //409: Not handled as its a conflict response code and comes in PUT/POST
        if (throwable instanceof IOException) {
            return ErrorType.NO_INTERNET;
        } else if (throwable instanceof CustomException) {
            final int errorCode = ((CustomException) throwable).mErrorCode;
            if (errorCode == 404) {
                return ErrorType.NOT_FOUND;
            } else if (errorCode == 401) {
                return ErrorType.UNAUTORISED;
            } else if (errorCode == 403) {
                return ErrorType.FORBIDDEN;
            } else if (errorCode == 500 || errorCode == 502) {
                return ErrorType.NO_SERVER_TRY_AGAIN;
            } else if (errorCode > 500 && errorCode < 599) {
                return ErrorType.NO_SERVER_TRY_LATER;
            } else if (errorCode == 1000) {
                return ErrorType.NO_COURSE_ENROLLED;
            } else if (errorCode == 1001) {
                return ErrorType.NO_COURSE_STARTED;
            }
        }
        if (throwable != null) {
            Crashlytics.logException(throwable);
        }
        return ErrorType.SOME_THING_WENT_WRONG;
    }

Answer 1

你遇到的是什么some argue是静态方法的问题。静态方法 logException() 似乎无处不在，但它实际上隐藏了真正的依赖性。由于该类现在需要该依赖项才能运行，因此它使您的类难以测试。

一个好的解决方案是创建一个包含非静态方法的包装类。如果我们应用 this answer 中的解决方案对于您的代码，它看起来像这样:

public class CrashLyticsWrapper { 
    public CrashLyticsWrapper() {} 

    public void logException(Throwable t) {
         CrashLytics.logException(t);
    }
}

然后可以将其作为对需要它的类的依赖项传递给构造函数。由于它是一个非静态依赖项并且是您现在控制的类，因此您可以轻松地模拟它并在必要时对其进行验证。

一个单独的问题:异常就像值对象，在大多数情况下不应该有静态或非静态的依赖。做这样的事情:

public CustomException extends RuntimeException() {
     public CustomException() {
          Crashlytics.logException(this); //don't do this!
     }
}

生成脆弱且无法测试的代码。从子类异常重写的构造函数中可以看出，有一个用于原因的字段，一个用于消息的字段，仅此而已。一个很好的异常(exception)范围。如果您需要添加伴随异常的功能，您应该编写单独的错误处理程序，将异常作为数据或参数。这符合 SOLID 中的“单一责任”原则。 .