испытания на заводе класса 3 партии

Question

Мое приложение использует баночку третьей стороны (нет доступа к источнику, и т.д.) У меня есть завод, который создает объект (назовем его Foo) правильно из настроек, т.е.

public FooFactoryImpl implements FooFactory { 
    private final Settings settings; 
    private final OtherDependency other; 

    @Inject 
    public FooFactoryImpl(Settings settings, OtherDependency other) { 
     this.settings = settings; 
     this.other = other; 
    } 

    public Foo create(String theirArg) { 
     Foo newFoo = new Foo(theirArg); // there is no no-arg constructor 

     // This isn't exactly the way I do it but this is shorter and close enough 
     newFoo.setParamOne(settings.get("ParamOne")); 
     newFoo.setParamTwo(settings.get("ParamTwo")); 
     // etc. 
    } 
} 

Я бы хотел, чтобы блок протестировал эту фабрику с помощью Mockito - убедитесь, что созданный объект настроен правильно. Но, конечно, я столкнулся с this problem; то есть, потому что мой завод называет new, я не могу ввести шпион.

Одним из возможных решений является ввести что-то вроде:

public FooFactoryDumb implements FooFactory { 
    public Foo create(String theirArg) { 
     return new Foo(theirArg); 
    } 
} 

А потом что-то вроде:

public FooFactoryImpl implements FooFactory { 
    @Inject @Dumb private FooFactory inner; 

    // snip, see above 

    public create(String theirArg) { 
     Foo newFoo = inner.create(theirArg); 
     // etc. 
    } 
} 

Это, кажется, как много шаблонного кода просто чтобы дать возможность модульного тестирования. Это smells bad для меня, но я могу ошибаться. Есть ли способ лучше?

Answer 1

Ну, оказывается, что мне пришлось использовать PowerMock, потому что методы третьей стороны были окончательными. Так как я уже использую PowerMock, я понял, что я могу просто сделать это:

@Before 
public void setUp() throws Exception { 
    Foo toReturn = PowerMockito.mock(Foo.class); 
    PowerMockito.whenNew(Foo.class).withAnyArguments().thenReturn(toReturn); 
} 

И тогда я не должен трогать мой исходный класс вообще.

Примечание: Если вы сделаете это, вы должны подготовить оба класса для PowerMock, то есть сделать

@PrepareForTest({ Foo.class, FooFactoryImpl.class }) 

Answer 2

Существует аналогичный, но более простой способ сделать это: добавить защищенный метод к вашей фабрике, чтобы создать Foo:

protected Foo create(String theirArg){ 
    return new Foo(theirArg); 
} 

затем в тестах вашего завода, создать Test Double вашего FactoryImpl и переопределение создавание метод:

private class FooFactoryImplTestDouble extends FooFactoryImpl{ 
    ... 
    @Override 
    protected Foo create(String theirArg){ 

     //create and return your spy here 
    } 
} 

Answer 3

Создайте новый класс:

public class FooFactory3rd { 
    public Foo create3rdParty(String theirArg) { 
     return new Foo(theirArg); 
    } 
} 

Затем измените свой класс:

public FooFactoryImpl implements FooFactory { 
    private final Settings settings; 
    private final OtherDependency other; 
    private final FooFactory3rd fooFactory3rd; 

    @Inject 
    public FooFactoryImpl(Settings settings, OtherDependency other, FooFactory3rd fooFactory3rd) { 
     this.settings = settings; 
     this.other = other; 
     this.fooFactory3rd = fooFactory3rd; 
    } 

    public Foo create(String theirArg) { 
     Foo newFoo = fooFactory3rd.create3rdParty(theirArg); 

     // This isn't exactly the way I do it but this is shorter and close enough 
     newFoo.setParamOne(settings.get("ParamOne")); 
     newFoo.setParamTwo(settings.get("ParamTwo")); 
     // etc. 
    } 
} 

И в тестовом коде:

Foo fooMock = mock(Foo.class); 
FooFactory3rd fooFactory3rdMock = mock(FooFactory3rd.class); 
when(fooFactory3rdMock.create3rdParty(any(String.class)).thenReturn(fooMock); 

FooFactoryImpl fooFactoryImpl = new FooFactoryImpl(settings, other, fooFactory3rdMock); 
fooFactoryImpl.create("any string"); 

Таким образом, вы можете придать свой fooMock. Когда вы вызываете fooFactoryImpl.create("any string"), ваше издеваемое Foo вызывается под обложкой.

Или, если вы хотите идти дальше, вам даже не нужен конструктор arg FooFactory3rd. Просто объявите

private final FooFactory3rd fooFactory3rd = new FooFactory3rd(); 

И в своем тесте используйте отражение, чтобы изменить его на издеваемую FooFactory3rd.

Answer 4

Сделайте шаг назад и подумать о том, что договор FooFactoryImpl есть. Это значит, что он должен создать полностью функциональный Foo, что бы это ни значило.Таким образом, если договор Foo заключается в том, что он выполняет X, Y и Z, то договор FooFactoryImpl заключается в том, что он создает объекты, которые делают X, Y и Z.

Это случай такого рода испытаний в который SUT состоит из более чем одного класса. Меня не волнует, называете ли вы это единичным тестом, интеграционным тестом, тестом подсистемы, тестом на совместную работу или другим именем. Дело в том, что единственным значимым тестом FooFactoryImpl является тест, который также тестирует Foo. Вместо написания тестового класса только для Foo, напишите тестовый класс, который совместно проверит два класса.

Итак, если договор Foo должен выполнять X, Y и Z, то ваши тестовые примеры будут делать следующие вещи с помощью FooFactoryImpl.

• Вызов create и тест, созданный объект делает X.
• вызов create и тест, созданный объект делает Y.
• Вызов create и тест, созданный объект делает Z.

Я считаю, что это единственный разумный способ атаковать эту проблему. Трудная часть придумывает убедительное имя для тестового класса.