I tried to find if there’s a similar question but didn’t even know what keywords should I use 🙂
I got a method in an interface accepting an other interface as a parameter:
bool CanDoIt(AnInterface subject)
One of the implementations needs to fulfill the requirement that the method should return false for all concrete types (implementing AnInterface) different than X.
Simple implementation could look like (didn’t wrote it since got no UT for that):
bool CanDoIt(AnInterface subject)
{
if(!(subject is X)) return false;
else ...
}
I know that this is a code smell accepting an interface and then checking it’s concrete type – unfortunately for now I got to stick with it.
I would like to write a test that would check if the requirement is implemented correctly. So that test would fail if someone would ever add a new type implementing AnInterface and changed the method not to meet the requirement.
If writing such a test isn’t possible then what can I do to consider the coded tested? Would writing a test using few exemplary types would be enough, or is it better not to write it at all?
Some unit-testing systems allow you to add a unit test on an interface, with the testing system running that test on any class which implements that interface. On such a system, you could just write a test which calls CanDoIt
and asserts that either it returned false or that AnInterface is X
. A similar “test” would be, if using a contract framework, to add a contract like Contract.Promises(Contract.ReturnValue == (subject is X));
to CanDoIt
.
If your system does not support either of these strategies, you may be better off asking a test-framework specific version of this question (on SO).
1
You can’t fix your design problem with testing.
You could write a test for every known implementation of AnInterface
. When someone creates a new implementation, they would write a new test for CanDoIt
.
It looks like CanDoIt
should be a method on AnInterface
. If you can, I suggest that you create aCanDoIt
method on AnInterface
and have your method delegate to AnInterface
.
I don’t think you can, because someone can easily write
bool CanDoIt(AnInterface subject)
{
if(!(subject is X || subject is Y)) return false;
else ...
}
and you don’t know in advance about the new class Y.
Merely calling CanDoIt(someBogusClassThatImplementsAnInterface)
would still fail
You should still call CanDoIt(someBogusClassThatImplementsAnInterface)
to ensure someone doesn’t simply remove the type check within CanDoIt()
, but to get complete coverage you’re going to have to remove the code smell and require CanDoIt()
only accept X
instead of anything that implements the interface
. Since you’ve noted this is not an option, you’re pretty much stuck.
1