Merge pull request #119 from nhaarman/clean-tests

Use same test source for inline tests

Author: nhaarman

.travis.yml

@@ -5,6 +5,11 @@ language: java
 jdk:
 - oraclejdk7
 
+matrix:
+  include:
+  - jdk: oraclejdk7
+    env: TERM=dumb MOCK_MAKER=mock-maker-inline
+
 env:
   matrix:
   - TERM=dumb

build.gradle

@@ -3,3 +3,5 @@ plugins {
 }
 
 apply from: 'gradle/scripts/tagging.gradle'
+
+println ext.versionName

gradle.properties

@@ -24,3 +24,4 @@
 #
 
 isRelease = false
+publishToLocal = false

mockito-kotlin/build.gradle

@@ -21,27 +21,6 @@ repositories {
     jcenter()
 }
 
-sourceSets {
-    testInlineMockito {
-        compileClasspath += main.output + test.output
-        runtimeClasspath += main.output + test.output
-    }
-}
-
-configurations {
-    testInlineMockitoCompile.extendsFrom testCompile
-    testInlineMockitoRuntime.extendsFrom testRuntime
-}
-
-// define custom test task for running integration tests
-task testInlineMockito(type: Test) {
-    testClassesDir = sourceSets.testInlineMockito.output.classesDir
-    classpath = sourceSets.testInlineMockito.runtimeClasspath
-}
-
-test.dependsOn testInlineMockito
-
-
 dependencies {
     compile "org.jetbrains.kotlin:kotlin-stdlib:$kotlin_version"
     compile "org.jetbrains.kotlin:kotlin-reflect:$kotlin_version"
@@ -63,3 +42,26 @@ dokka {
     }
 }
 javadoc.dependsOn dokka
+
+//Switch inline
+task createTestResources << {
+    def mockMakerFile = new File("$projectDir/src/test/resources/mockito-extensions/org.mockito.plugins.MockMaker")
+    if (System.env.MOCK_MAKER != null) {
+        logger.warn("! Using MockMaker ${System.env.MOCK_MAKER}")
+        mockMakerFile.parentFile.mkdirs()
+        mockMakerFile.createNewFile()
+        mockMakerFile.write(System.env.MOCK_MAKER)
+    } else {
+        logger.warn("! Using default MockMaker")
+    }
+}
+
+task removeTestResources << {
+    def mockMakerFile = new File("$projectDir/src/test/resources/mockito-extensions/org.mockito.plugins.MockMaker")
+    if (mockMakerFile.exists()) {
+        mockMakerFile.delete()
+    }
+}
+
+test.dependsOn createTestResources
+test.finalizedBy removeTestResources

mockito-kotlin/publishing.gradle

@@ -40,6 +40,8 @@ uploadArchives {
                 )
             }
 
+            if (publishToLocal) repository(url: mavenLocal().url)
+
             pom.project {
                 name 'Mockito-Kotlin'
                 packaging 'jar'

mockito-kotlin/src/main/kotlin/com/nhaarman/mockito_kotlin/CreateInstance.kt

@@ -48,9 +48,9 @@ import java.lang.reflect.Array as JavaArray
 /**
  * Checks whether the resource file to enable mocking of final classes is present.
  */
-private var mockMakerInlineEnabled: Boolean? = null
+internal var mockMakerInlineEnabled: Boolean? = null
 
-private fun mockMakerInlineEnabled(jClass: Class<out Any>): Boolean {
+internal fun mockMakerInlineEnabled(jClass: Class<out Any>): Boolean {
     return mockMakerInlineEnabled ?:
             jClass.getResource("mockito-extensions/org.mockito.plugins.MockMaker")?.let {
                 mockMakerInlineEnabled = File(it.file).readLines().filter { it == "mock-maker-inline" }.isNotEmpty()

mockito-kotlin/src/test/kotlin/ArgumentCaptorTest.kt

@@ -3,7 +3,7 @@ import com.nhaarman.mockito_kotlin.*
 import org.junit.Test
 import java.util.*
 
-class ArgumentCaptorTest {
+class ArgumentCaptorTest : BaseTest() {
 
     @Test
     fun argumentCaptor_withSingleValue() {

mockito-kotlin/src/test/kotlin/BaseTest.kt

@@ -0,0 +1,16 @@
+import com.nhaarman.mockito_kotlin.mockMakerInlineEnabled
+import org.junit.After
+import org.junit.Before
+
+abstract class BaseTest {
+
+    @Before
+    open fun setup() {
+        mockMakerInlineEnabled = false
+    }
+
+    @After
+    open fun tearDown() {
+        mockMakerInlineEnabled = null
+    }
+}

mockito-kotlin/src/test/kotlin/CreateInstanceTest.kt

@@ -31,7 +31,7 @@ import org.junit.Test
 import java.util.*
 import kotlin.reflect.KClass
 
-class CreateInstanceTest {
+class CreateInstanceTest : BaseTest() {
 
     @Test
     fun byte() {
@@ -555,8 +555,8 @@ class CreateInstanceTest {
     }
 
     enum class MyEnum { VALUE, ANOTHER_VALUE }
+}
 
-    sealed class MySealedClass {
-        class MySealedClassMember : MySealedClass()
-    }
+sealed class MySealedClass {
+    class MySealedClassMember : MySealedClass()
 }

mockito-kotlin/src/test/kotlin/EqTest.kt

@@ -31,7 +31,7 @@ import org.junit.Before
 import org.junit.Test
 import org.mockito.Mockito
 
-class EqTest {
+class EqTest : BaseTest() {
 
     private val interfaceInstance: MyInterface = MyClass()
     private val openClassInstance: MyClass = MyClass()
@@ -40,13 +40,17 @@ class EqTest {
     private lateinit var doAnswer: Open
 
     @Before
-    fun setup() {
+    override fun setup() {
+        super.setup()
+
         /* Create a proper Mockito state */
         doAnswer = Mockito.doAnswer { }.`when`(mock())
     }
 
     @After
-    fun tearDown() {
+    override fun tearDown() {
+        super.tearDown()
+
         /* Close `any` Mockito state */
         doAnswer.go(0)
     }