Merge branch 'tests-experimenting' into development

Author: sofakingforever

analytics/build.gradle

@@ -37,17 +37,32 @@ android {
         }
     }
 
+//    testOptions {
+//        unitTests.returnDefaultValues = true
+//    }
 }
 
 dependencies {
+
     implementation fileTree(dir: 'libs', include: ['*.jar'])
 
+    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
+
     implementation 'com.android.support:appcompat-v7:28.0.0-rc01'
     testImplementation 'junit:junit:4.12'
+
+
     androidTestImplementation 'com.android.support.test:runner:1.0.2'
     androidTestImplementation 'com.android.support.test.espresso:espresso-core:3.0.2'
-    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
-
+    // required if you want to use Mockito for unit tests
+    testImplementation 'org.mockito:mockito-core:2.7.22'
+    // required if you want to use Mockito for Android tests
+    androidTestImplementation 'org.mockito:mockito-android:2.7.22'
+
+//    testCompile 'org.powermock:powermock:1.6.5'
+//    testCompile 'org.powermock:powermock-module-junit4:1.6.5'
+//
+//    testCompile 'org.powermock:powermock-api-mockito:1.6.5'
 
 
 }

analytics/src/main/java/com/sofakingforever/analytics/Analytics.kt

@@ -3,6 +3,7 @@ package com.sofakingforever.analytics
 import android.content.Context
 import android.util.Log
 import com.sofakingforever.analytics.events.base.Event
+import com.sofakingforever.analytics.exceptions.EventNotTrackedException
 
 /**
  * The *Analytics* class is in charge of tracking any *Event* implementation.
@@ -47,8 +48,7 @@ class Analytics(context: Context, private vararg val dispatchers: AnalyticsDispa
                 try {
                     dispatcher.track(it)
                 } catch (e: Exception) {
-                    Log.e("Analytics", "${dispatcher.kit.name} dispatcher couldn't fire \"${it.javaClass.name}\" event", e)
-                    settings.exceptionHandler?.onException(e)
+                    settings.exceptionHandler?.onException(EventNotTrackedException(dispatcher, it, e))
                 }
             }
 
@@ -58,18 +58,10 @@ class Analytics(context: Context, private vararg val dispatchers: AnalyticsDispa
 
     fun setKitEnabled(kit: AnalyticsKit, enabled: Boolean) {
         enabledKitMap[kit] = enabled
-//        dispatchers.filter { d -> d.kit == kit }
-//                .forEach { dispatcher ->
-//
-//                }
     }
 
     fun setDispatcherEnabled(dispatcherName: String, enabled: Boolean) {
         enabledDispatcherMap[dispatcherName] = enabled
-//        dispatchers.filter { d -> d.dispatcherName == dispatcherName }
-//                .forEach { dispatcher ->
-//                    dispatcher.enabled = enabled
-//                }
     }
 
 }

analytics/src/main/java/com/sofakingforever/analytics/exceptions/EventNotTrackedException.kt

@@ -0,0 +1,10 @@
+package com.sofakingforever.analytics.exceptions
+
+import com.sofakingforever.analytics.AnalyticsDispatcher
+import com.sofakingforever.analytics.events.base.Event
+
+class EventNotTrackedException(message: String?, cause: Throwable?) : RuntimeException(message, cause) {
+
+    constructor(dispatcher: AnalyticsDispatcher, event: Event, t: Throwable) : this("${dispatcher.dispatcherName} dispatcher couldn't fire \"${event.javaClass.name}\" event", t)
+
+}

analytics/src/main/java/com/sofakingforever/analytics/kits/logger/LoggerDispatcherImpl.kt

@@ -22,7 +22,6 @@ class LoggerDispatcherImpl(override val init: Boolean) : AnalyticsDispatcher {
     constructor() : this(true)
 
 
-
     private val tag = dispatcherName
 
     override val kit: AnalyticsKit = LoggerKit.instance
@@ -42,7 +41,8 @@ class LoggerDispatcherImpl(override val init: Boolean) : AnalyticsDispatcher {
     override fun trackInviteEvent(inviteEvent: InviteEvent) {
         Log.d(tag, "Tracking inviteEvent ${inviteEvent.packageName}")
     }
+
     override fun setUserProperty(property: SetUserProperty) {
-        // used only for certain analytics
+        Log.d(tag, "Tracking user property ${property.key} = ${property.value}")
     }
 }

analytics/src/test/java/com/sofakingforever/library/AnalyticsUnitTest.kt

@@ -0,0 +1,103 @@
+package com.sofakingforever.library
+
+import android.content.Context
+import android.util.Log
+import com.sofakingforever.analytics.Analytics
+import com.sofakingforever.analytics.AnalyticsSettings
+import com.sofakingforever.analytics.exceptions.EventNotTrackedException
+import com.sofakingforever.analytics.exceptions.UnsupportedEventException
+import com.sofakingforever.library.dispatcher.TestKit
+import com.sofakingforever.library.dispatcher.TestableDispatcher
+import com.sofakingforever.library.events.InitDispatcherEvent
+import com.sofakingforever.library.events.TestContentViewEvent
+import com.sofakingforever.library.events.TestCustomEvent
+import com.sofakingforever.library.events.UnsupportedEvent
+import org.junit.Test
+import org.mockito.Mockito
+import org.mockito.Mockito.mock
+
+class AnalyticsUnitTest {
+
+    private val contextMock = mock(Context::class.java)
+
+    private lateinit var analytics: Analytics
+
+    private val dispatcher = TestableDispatcher()
+    private var raisedException: Exception? = null
+
+    init {
+        Mockito.`when`(contextMock.applicationContext).thenReturn(contextMock)
+    }
+
+    @Test
+    fun testAnalytics() {
+
+        analytics = Analytics(contextMock, dispatcher).apply {
+
+            this.settings.exceptionHandler = object : AnalyticsSettings.ExceptionHandler {
+                override fun onException(e: Exception) {
+                    raisedException = e
+                }
+
+            }
+        }
+
+        // track some events
+        trackTestEvents()
+
+        // assert events
+        assertEvents()
+
+        // assert no exceptions were raised
+        assert(raisedException == null)
+
+        // track an unsupported event
+        analytics.track(UnsupportedEvent())
+
+        // assert UnsupportedEventException was raised
+        assert(raisedException != null)
+        assert(raisedException is EventNotTrackedException)
+        assert((raisedException as EventNotTrackedException).cause is UnsupportedEventException)
+
+    }
+
+
+    private fun trackTestEvents() {
+
+        analytics.track(TestCustomEvent(1))
+
+        analytics.setDispatcherEnabled(TestableDispatcher.DispatcherName, false)
+
+        analytics.track(TestCustomEvent(2))
+
+        analytics.setDispatcherEnabled(TestableDispatcher.DispatcherName, true)
+
+        analytics.track(TestCustomEvent(3))
+
+        analytics.setKitEnabled(TestKit.instance, false)
+
+        analytics.track(TestCustomEvent(4))
+
+        analytics.setKitEnabled(TestKit.instance, true)
+
+        analytics.track(TestCustomEvent(5))
+
+        analytics.track(TestContentViewEvent(1))
+
+    }
+
+
+    private fun assertEvents() {
+
+
+        val eventList = dispatcher.eventList
+
+        // expect to find 3 custom events , 1 contentview and 1 init event
+        assert(eventList.size == 5)
+        assert(eventList[0] is InitDispatcherEvent)
+        assert((eventList[1] as TestCustomEvent).number == 1)
+        assert((eventList[2] as TestCustomEvent).number == 3)
+        assert((eventList[3] as TestCustomEvent).number == 5)
+        assert((eventList[4] as TestContentViewEvent).number == 1)
+    }
+}

analytics/src/test/java/com/sofakingforever/library/DispatcherTest.kt

@@ -0,0 +1,41 @@
+package com.sofakingforever.library
+
+import android.app.Application
+import com.sofakingforever.library.dispatcher.TestableDispatcher
+import com.sofakingforever.library.events.InitDispatcherEvent
+import com.sofakingforever.library.events.TestCustomEvent
+import org.junit.Test
+
+
+class DispatcherTest {
+
+    private lateinit var raisedException: Exception
+
+    @Test
+    fun testDispatcher() {
+
+        val dispatcher = TestableDispatcher()
+
+        dispatcher.initDispatcher(Application())
+
+        dispatcher.track(TestCustomEvent(1))
+
+        dispatcher.track(TestCustomEvent(2))
+
+        dispatcher.track(TestCustomEvent(3))
+
+        dispatcher.track(TestCustomEvent(4))
+
+        dispatcher.track(TestCustomEvent(5))
+
+        val eventList = dispatcher.eventList
+
+        // expect to find 5 events + init event
+        assert(eventList.size == 6)
+        assert(eventList[0] is InitDispatcherEvent)
+        assert((eventList[1] as TestCustomEvent).number == 1)
+
+        // that's enough
+
+    }
+}

analytics/src/test/java/com/sofakingforever/library/ExampleUnitTest.java

@@ -0,0 +1,17 @@
+package com.sofakingforever.library.dispatcher
+
+import com.sofakingforever.analytics.AnalyticsKit
+
+class TestKit private constructor() : AnalyticsKit {
+    override val name: String = "Test Kit"
+
+    private object Holder {
+        val INSTANCE = TestKit()
+    }
+
+    companion object {
+        val instance: TestKit by lazy { Holder.INSTANCE }
+    }
+
+
+}

analytics/src/test/java/com/sofakingforever/library/dispatcher/TestKit.kt

@@ -0,0 +1,55 @@
+package com.sofakingforever.library.dispatcher
+
+import android.content.Context
+import com.sofakingforever.analytics.AnalyticsDispatcher
+import com.sofakingforever.analytics.AnalyticsKit
+import com.sofakingforever.analytics.events.ContentViewEvent
+import com.sofakingforever.analytics.events.CustomEvent
+import com.sofakingforever.analytics.events.InviteEvent
+import com.sofakingforever.analytics.events.SetUserProperty
+import com.sofakingforever.analytics.events.base.Event
+import com.sofakingforever.library.events.InitDispatcherEvent
+
+class TestableDispatcher : AnalyticsDispatcher {
+
+    override val init: Boolean = true
+
+    override val kit: AnalyticsKit = TestKit.instance
+
+    override val dispatcherName: String = DispatcherName
+
+    val eventList: MutableList<Event> = mutableListOf()
+
+    override fun initDispatcher(context: Context) {
+        track(InitDispatcherEvent())
+    }
+
+    override fun trackContentView(contentView: ContentViewEvent) {
+        eventList.add(contentView)
+    }
+
+    override fun trackCustomEvent(event: CustomEvent) {
+        eventList.add(event)
+    }
+
+    override fun trackInviteEvent(inviteEvent: InviteEvent) {
+        eventList.add(inviteEvent)
+    }
+
+    override fun setUserProperty(property: SetUserProperty) {
+        eventList.add(property)
+    }
+
+    override fun track(event: Event) {
+        if (event is InitDispatcherEvent) {
+            eventList.add(event)
+        } else {
+            super.track(event)
+        }
+    }
+
+    companion object {
+        const val DispatcherName = "TestDispatcher"
+    }
+
+}

analytics/src/test/java/com/sofakingforever/library/dispatcher/TestableDispatcher.kt

@@ -0,0 +1,5 @@
+package com.sofakingforever.library.events
+
+import com.sofakingforever.analytics.events.base.Event
+
+class InitDispatcherEvent : Event

analytics/src/test/java/com/sofakingforever/library/events/InitDispatcherEvent.kt

@@ -0,0 +1,9 @@
+package com.sofakingforever.library.events
+
+import com.sofakingforever.analytics.AnalyticsKit
+import com.sofakingforever.analytics.events.ContentViewEvent
+
+class TestContentViewEvent(val number: Int) : ContentViewEvent {
+    override fun getViewName(kit: AnalyticsKit): String = "Content View #$number"
+
+}

analytics/src/test/java/com/sofakingforever/library/events/TestContentViewEvent.kt

@@ -0,0 +1,17 @@
+package com.sofakingforever.library.events
+
+import com.sofakingforever.analytics.AnalyticsKit
+import com.sofakingforever.analytics.events.CustomEvent
+
+class TestCustomEvent(val number: Int) : CustomEvent {
+    override fun getEventName(kit: AnalyticsKit): String = "Custom Event #$number"
+
+    override fun getParameters(kit: AnalyticsKit): MutableMap<String, Any> {
+        val parameters = super.getParameters(kit)
+        parameters["int"] = -2
+        parameters["long"] = +2L
+        parameters["boolean"] = true
+        parameters["string"] = "fun #$number"
+        return parameters
+    }
+}

analytics/src/test/java/com/sofakingforever/library/events/TestCustomEvent.kt

@@ -0,0 +1,9 @@
+package com.sofakingforever.library.events
+
+import com.sofakingforever.analytics.events.base.Event
+
+/**
+ * This unsupported event is not handled in the Dispatcher.
+ * You must override the dispatcher's track function to handle it
+ */
+class UnsupportedEvent : Event