Should a static class/method/property be used in a unit test development environment, given that there is no way to test it without introducing a wrapper that is again not testable?
Another scenario is that when the static members are used within the unit tested target, the static member cannot be mocked. Thus, you have to test the static members when the unit tested target is tested. You want to isolate it when the static member performs calculation.
Testing static method is no different than testing any other method. Having static method as a dependency inside another tested module raises problems (as it's been mentioned - you can't mock/stub it with free tools). But if the static method itself is unit tested you can simply treat it as working, reliable component .
Overall, there's nothing wrong (as in, it doesn't disrupt unit testing/TDD) with static methods when:
Math.Floor
might be considered reliable - using it shouldn't raise "Look out, it's static!" warning; one might assume Microsoft does its job)When static methods will cause problems and should be avoided? Basically only when they interact with/ do something you cannot control (or mock):
Edit: two examples on when static method will make unit testing hard
1
public int ExtractSumFromReport(string reportPath)
{
var reportFile = File.ReadAllText(reportPath);
// ...
}
How do you deal with File.ReadAllText
? This will obviously go to file system to retrieve file content, which is major no-no when unit testing. This is example of static method with external dependency. To avoid that, you usually create wrapper around file system api or simply inject it as dependency/delegate.
2
public void SaveUser(User user)
{
var session = SessionFactory.CreateSession();
// ...
}
What about this? Session is non-trivial dependency. Sure, it might come as ISession
, but how do force SessionFactory
to return mock? We can't. And we can't create easy to detemine session object either.
In cases like above, it's best to avoid static methods altogether.
Static methods CAN be unit tested. They can't be mocked (generally; there are some frameworks to do this like Moles .
You can't mock static methods/properties. So, when your classA
uses some static member of classB
you can't test classA
in isolation.
UPDATE: I don't see any problem of wrapping some static class into object. It doesn't take lot of time, but it allows you to decrease coupling in your system.
Technically you can mock the static method in Java with PowerMock , but if you need to do this I would seriously recommend to refactor your code. I think that static methods should always be private
and used only inside the classes they are defined in, for internal purposes. I consider publicly exposed static method as a code smell .
无论编程语言如何,我都会发现这种威胁,但由于我的日常生活与 PHP 有关,这里是处理 PHP 静态方法的好资源http://docs.mockery.io/en/latest/reference/public_static_properties.html
The technical post webpages of this site follow the CC BY-SA 4.0 protocol. If you need to reprint, please indicate the site URL or the original address.Any question please contact:yoyou2525@163.com.