Parameter specified as non-null is null when using Mokito anyObject() on Kotlin function

Question

My code as below, refering to the solution in https://stackoverflow.com/a/30308199/3286489

import org.mockito.Mock
import org.mockito.Mockito
import org.mockito.MockitoAnnotations
import org.mockito.Mockito.*

class SimpleClassTest {

    private fun <T> anyObject(): T {
        Mockito.anyObject<T>()
        return uninitialized()
    }

    private fun <T> uninitialized(): T = null as T
    lateinit var simpleObject: SimpleClass
    @Mock lateinit var injectedObject: InjectedClass


    @Before
    fun setUp() {
        MockitoAnnotations.initMocks(this)
    }

    @Test
    fun testSimpleFunction() {
        simpleObject = SimpleClass(injectedObject)

        verify(injectedObject).settingDependentObject(anyObject())

    }
}

I still have the below error

java.lang.IllegalArgumentException: Parameter specified as non-null is null: method my.package.InjectedClass.settingDependentObject, parameter dependentObject

Did I miss anything?

UPDATED Below is the code tested (simplest form and working)

class SimpleClass(val injectedClass: InjectedClass) {

    fun simpleFunction() {
        injectedClass.settingDependentObject(DependentClass(Response.Builder().build()))
    }
}

open class DependentClass(response: Response) {

}

open class InjectedClass() {
    lateinit var dependentObject: DependentClass

    fun settingDependentObject(dependentObject: DependentClass) {
        this.dependentObject = dependentObject
    }
}

miensol · Accepted Answer · 2016-05-21 10:19:02Z

18

By default Kotlin classes and members are final. Mockito cannot mock final classes or methods. Thus when you write:

verify(injectedObject).settingDependentObject(anyObject())

the real implementation is called which requires non null argument.

To fix that either open your class and method or, even better, change SimpleClass to accept an interface as its constructor argument and mock the interface instead.

edited May 21, 2016 at 10:19

answered May 21, 2016 at 7:42

miensol

41.9k9 gold badges127 silver badges116 bronze badges

Sign up to request clarification or add additional context in comments.

6 Comments

Elye Over a year ago

My injectObject class is already an open class. The settingDependentObject require a non-null object, but anyObject() returns null. That's why I thought the solution in stackoverflow.com/a/30308199/3286489 would help me overcome the null required issue. But it's still not working.

miensol Over a year ago

@Elye please add the InjectedClass source to the question

Elye Over a year ago

Added the class to be tested to the Update above. Thanks!

miensol Over a year ago

@Elye you need to open both class and method for Mockito to work

Elye Over a year ago

Ya, open settingDependentObject, makes the things work great. Thanks!

|

Jayson Minard · Accepted Answer · 2016-09-07 10:22:35Z

There is a project specifically to help deal with Kotlin "closed by default" in unit testing with Mockito. For JUNIT, you can use the kotlin-testrunner which is an easy way to make any Kotlin test automatically open up classes for testing as they are loaded by the classloader. Usage is simple, just add one annotation of @RunWith(KotlinTestRunner::class), for example:

@RunWith(KotlinTestRunner::class)
class MyKotlinTestclass {
   @Test 
   fun test() {
   ...
   }
}

This is thoroughly covered in the article Never say final: mocking Kotlin classes in unit tests

This covers your use case in an automatic way by allowing all classes to be mocked that otherwise would not be allowed.

fast3r · Accepted Answer · 2018-05-03 09:48:09Z

0

I ran into the same issue with Mockito when using RETURNS_DEEP_STUBS. It seems like nulls are still returned for nested objects, even when using the kotlin-allopen plugin.

Please check out and comment on this issue on Mockito if you're having the same problem.

answered May 3, 2018 at 9:48

fast3r

1,30813 silver badges15 bronze badges

Comments

Kyriakos Georgiopoulos · Accepted Answer · 2020-05-23 17:41:42Z

-1

You can use this function instead

inline fun <reified T : Any> any(): T = Mockito.any(T::class.java) ?: T::class.java.newInstance()

answered May 23, 2020 at 17:41

Kyriakos Georgiopoulos

5678 silver badges7 bronze badges

Collectives™ on Stack Overflow

Parameter specified as non-null is null when using Mokito anyObject() on Kotlin function

4 Answers 4

6 Comments

Comments

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

6 Comments

Comments

Comments

Comments

Linked

Related