У меня есть два противоположных метода, которые я должен проверить
Первый метод
private boolean isGeoOrtInboundPortabilityToNonGeoSipFiber(Character oldNdipAffectation,
Character newNdipAffectation) {
return isNdip09NatifVoipSip(newNdipAffectation)
&& isNdipGeoOperator(oldNdipAffectation);
}
Второй способ
private boolean isNonGeoSipOutboundPortabilityToGeoFiberOrt(Character oldNdipAffectation,
Character newNdipAffectation) {
return isNdipGeoOperator(newNdipAffectation)
&& isNdip09NatifVoipSip(oldNdipAffectation);
}
Вот реализация
protected void checkRenumberingValidation(Character oldNdipAffectation,
NdipRenumberingRequest request, String basicatCode) {
var oldNdipIndicatif = findIndicatifByNd(request.getOldNdip());
var newNdipIndicatif = findIndicatifByNd(request.getNewNdip());
if (!isClassicIpPortedNds(request)
|| !isGeoOrtInboundPortabilityToNonGeoSipFiber(oldNdipAffectation, newNdipAffectation)
|| !isNonGeoSipOutboundPortabilityToGeoFiberOrt(oldNdipAffectation, newNdipAffectation)
|| !isRtcPortabilityToFiberSip(newNdipIndicatif, oldNdipAffectation, newNdipAffectation,
basicatCode)
&& (
!Objects.equals(newNdipIndicatif.getZne().getCzne(), oldNdipIndicatif.getZne().getCzne())
|| !Objects.equals(newNdipIndicatif.getCsitugeo(), oldNdipIndicatif.getCsitugeo()))
) {
throwFunctionalException(ERROR_34);
}
}
Если я передам «D» и «P», это вызовет throwFunctionalException (ERROR_34), и это нормально
!isGeoOrtInboundPortabilityToNonGeoSipFiber('D', 'P') // Throw exception which is expected
Проблема в том, что метод isRtcPortabilityToFiberSip() никогда не достигается, потому что, если я передам «P» и «D»,
!isGeoOrtInboundPortabilityToNonGeoSipFiber('P', 'D') // Pass
isNonGeoSipOutboundPortabilityToGeoFiberOrt('P', 'D') // Throw exception which is expected
!isRtcPortabilityToFiberSip(newNdipIndicatif, oldNdipAffectation, newNdipAffectation,
basicatCode) // never reached
Как достичь !isRtcPortabilityToFiberSip с помощью JUnit 5?
Я гуглил проблему без какого-либо решения.
У вас должно быть три разных тестовых функции:
@Test
public void testIsGeoOrtInboundPortabilityToNonGeoSipFiber() throws Exception
{
!isGeoOrtInboundPortabilityToNonGeoSipFiber('P', 'D') // Pass
}
@Test(expected = Exception.class)
public void testIsNonGeoSipOutboundPortabilityToGeoFiberOrt() throws Exception
{
isNonGeoSipOutboundPortabilityToGeoFiberOrt('P', 'D') // Throw exception which is expected
}
@Test
public void testIsRtcPortabilityToFiberSip() throws Exception
{
!isRtcPortabilityToFiberSip(newNdipIndicatif, oldNdipAffectation, newNdipAffectation,
basicatCode) // never reached, but now it will be!
}
Хорошей практикой является наличие одного утверждения на тест (хотя это гибко)
В вашем коде на самом деле нет утверждений (их можно было бы добавить, если checkRenumberingValidation возвращает логическое значение вместо того, чтобы генерировать исключение, когда объекты не равны), но условия проверяются адекватно, просто более окольным путем, чем обычно.
Чтобы узнать больше о тестировании, я настоятельно рекомендую JUnit FAQ (написано, когда JUnit 4 был актуален, но почти все концепции могут применяться в целом - это классика)
Посмотрите на assertThrows(...) — вы в основном тестируете исключение, а затем позволяете тесту продолжаться, если оно выдается. Вы также можете подумать о том, будут ли эти 3 вызова на самом деле 3 независимыми тестами - по крайней мере, первые 2 кажутся таковыми, поскольку нет смысла терпеть неудачу и передавать один и тот же ввод.