Over a million developers have joined DZone.
{{announcement.body}}
{{announcement.title}}

When Mockito’s InjectMocks Does Not Inject Mocks

DZone's Guide to

When Mockito’s InjectMocks Does Not Inject Mocks

How to perform unit tests in Java using Mockito's InjectMocks tool and how to get it working.

· DevOps Zone
Free Resource

Learn more about how CareerBuilder was able to resolve customer issues 5x faster by using Scalyr, the fastest log management tool on the market. 

Looking back at the time wasted yesterday while trying to make a trivial functionality work makes me wonder if it was a good idea to begin with…

It all started  (and ended) yesterday while I was mentoring a team on the fine art of Java unit testing.

We’ve decided to use Mockito’s InjectMocks due to the fact that most of the project's classes used Spring to fill private fields (don’t get me started).

For those of you who never used InjectMocks before — in the Mockito world we can auto-magically initialize and inject mock objects into the class under test. And it’s all done using annotations.

And so if I have the following class:

public class MyClass {

    @Resource
    private INetworkService networkService;

    @Resource
    private IFileService fileService;

    public boolean SomeMethod(){
        // some logic here

        // More logic here
        networkService.Send();

        return true;
    }
}

I can write a test fixture that looks like this:

@RunWith(MockitoJUnitRunner.class)
public class MyClassTest {
    @Mock
    private INetworkService networkService;

    @Mock
    private IFileService fileService;

    @InjectMocks
    MyClass myClass;

    @Test
    public void testSomeMethod() {
        boolean result = myClass.SomeMethod();

        assertTrue(result);
    }
}

And so the two dependencies (marked with @Mock) would be faked and inserted into MyClass by constructor, property or field injection.

The problem was that we couldn’t get it to work. The mocks were initialized and the class was created - without the dependencies. It took us some time but finally we’ve found the reason: the real class looked something like this — can you spot the difference?

public class MyClass {

    @Resource
    private INetworkService networkService;

    @Resource
    private IFileService fileService;

    private Integer version;

    public MyClass(Integer version) {
        this.version = version;
    }

    public boolean SomeMethod(){
        // some logic here

        // More logic here
        networkService.Send();

        return true;
    }
}

Did you see it?

In our real class we had a non-empty constructor which InjectMocks tried to use, but passed null since Integer canno be mocked by Mockito (it’s a final class). OnceMockito found a constructor to try and to use it didn’t even try to inject the two fields (lazy bastard). And so the dependencies inside the MyClass remained null causing a null reference exception to be thrown once used.

It’s not that Mockito guys didn’t do a good job, this behavior is documented – which makes it yet another case of RTFM.

The problem is that the tests who successfully run using this mechanism could one day fail just because someone decided to add another constructor.

Happy coding!

Find out more about how Scalyr built a proprietary database that does not use text indexing for their log management tool.

Topics:
unit testing ,mockito ,testing

Published at DZone with permission of Dror Helper, DZone MVB. See the original article here.

Opinions expressed by DZone contributors are their own.

{{ parent.title || parent.header.title}}

{{ parent.tldr }}

{{ parent.urlSource.name }}