Prevent DebugLog logs messages when running code under JUnit.

[VictorAlbertos]

Hi.

I've been getting the next exception when running a JUnit test which calls methods annotated with @RxLogObservable

java.lang.RuntimeException: Method d in android.util.Log not mocked. See http://g.co/androidstudio/not-mocked for details.

    at android.util.Log.d(Log.java)
    at com.fernandocejas.frodo.internal.DebugLog.log(DebugLog.java:21)
    at com.fernandocejas.frodo.internal.MessageManager.printMessage(MessageManager.java:20)
    at com.fernandocejas.frodo.internal.MessageManager.printObservableInfo(MessageManager.java:25)
    at com.fernandocejas.frodo.internal.observable.FrodoObservable.getObservable(FrodoObservable.java:28)
    at com.fernandocejas.frodo.aspect.LogObservable.weaveAroundJoinPoint(LogObservable.java:39)
    at app.data.sections.chat.ChatRepository.getChats(ChatRepository.java:30)
    at app.data.sections.chat.ChatRepositoryTest.When_Remove_Chat_Then_Remove_Chat(ChatRepositoryTest.java:29)
....

I've been able to fix the error by adding the next configuration test to gradle:

android {
    testOptions {
        unitTests.returnDefaultValues = true
    }
}

This pull request tries to fix that. I've been able to compile the project and run successfully the unit tests of "frodo-runtime" module. But it seems that DebugLog class isn't covered by any of them (probably not need for that until now).

Also, I don't know how can I test this new functionality from the android sample app. I couldn't connect the "frodo-plugin" gradle locally with the sample app. Indeed, I don't even know if that's possible (I have a very limited knowledge about gradle plugins).

[dmitry-zaitsev]

I believe it would be better to just catch all Exceptions as it won't slow down execution of unit tests as much as checking the stack trace. Moreover, this implementation is tied to JUnit. What if we are just running our code on desktop?

[VictorAlbertos]

I tried to catch the exception but I think this kind of exception is not possible to catch.

[dmitry-zaitsev]

Should just work:

try {
    Log.d(tag, message);
} catch (Exception e) {
    // Ignore
}

[VictorAlbertos]

It didn't work when I tried it. It was my first approach, actually. But you can try it and let me know, maybe I missed something I don't know ;)