Jak Mogę potwierdzić wiadomość o wyjątku za pomocą adnotacji testowej JUnit?

I like the @Rule odpowiedz. Jeśli jednak z jakiegoś powodu nie chcesz używać zasad. Jest trzecia opcja.

@Test (expected = RuntimeException.class)
public void myTestMethod()
{
   try
   {
      //Run exception throwing operation here
   }
   catch(RuntimeException re)
   {
      String message = "Employee ID is null";
      assertEquals(message, re.getMessage());
      throw re;
    }
    fail("Employee Id Null exception did not throw!");
  }

W JUnit 4.13 możesz zrobić:

import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertThrows;

...

@Test
void exceptionTesting() {
  IllegalArgumentException exception = assertThrows(
    IllegalArgumentException.class, 
    () -> { throw new IllegalArgumentException("a message"); }
  );

  assertEquals("a message", exception.getMessage());
}

Działa to również w JUnit 5 , ale z różnymi importami:

import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertThrows;

...

Czy musisz używać @Test(expected=SomeException.class)? Kiedy musimy potwierdzić rzeczywiste przesłanie wyjątku, to właśnie robimy.

@Test
public void myTestMethod()
{
  try
  {
    final Integer employeeId = null;
    new Employee(employeeId);
    fail("Should have thrown SomeException but did not!");
  }
  catch( final SomeException e )
  {
    final String msg = "Employee ID is null";
    assertEquals(msg, e.getMessage());
  }
}

Właściwie, najlepiej jest użyć try/catch. Dlaczego? Ponieważ możesz kontrolować miejsce, w którym spodziewasz się wyjątku.

Rozważ ten przykład:

@Test (expected = RuntimeException.class)
public void someTest() {
   // test preparation
   // actual test
}

Co jeśli pewnego dnia kod zostanie zmodyfikowany i przygotowanie do testu spowoduje wywołanie RuntimeException? W takim przypadku rzeczywisty test nie jest nawet testowany i nawet jeśli nie rzuci żadnego wyjątku, test przejdzie.

Dlatego o wiele lepiej jest użyć try / catch niż polegać na adnotacji.

Raystorm miał dobrą odpowiedź. Ja też nie przepadam za zasadami. Robię coś podobnego, z wyjątkiem tego, że tworzę następującą klasę użyteczności, aby pomóc w czytelności i użyteczności, która jest jednym z dużych plusów adnotacji na pierwszym miejscu.

Dodaj tę klasę użyteczności:

import org.junit.Assert;

public abstract class ExpectedRuntimeExceptionAsserter {

    private String expectedExceptionMessage;

    public ExpectedRuntimeExceptionAsserter(String expectedExceptionMessage) {
        this.expectedExceptionMessage = expectedExceptionMessage;
    }

    public final void run(){
        try{
            expectException();
            Assert.fail(String.format("Expected a RuntimeException '%s'", expectedExceptionMessage));
        } catch (RuntimeException e){
            Assert.assertEquals("RuntimeException caught, but unexpected message", expectedExceptionMessage, e.getMessage());
        }
    }

    protected abstract void expectException();

}

Więc do mojego testu jednostkowego, potrzebuję tylko tego kodu:

@Test
public void verifyAnonymousUserCantAccessPrivilegedResourceTest(){
    new ExpectedRuntimeExceptionAsserter("anonymous user can't access privileged resource"){
        @Override
        protected void expectException() {
            throw new RuntimeException("anonymous user can't access privileged resource");
        }
    }.run(); //passes test; expected exception is caught, and this @Test returns normally as "Passed"
}

Nigdy nie lubiłem sposobu twierdzenia WYJĄTKÓW z Junit. Jeśli użyję "oczekiwanego" w adnotacji, wydaje mi się, że z mojego punktu widzenia naruszamy wzorzec "given, when, then", ponieważ "then" znajduje się na górze definicji testu.

Ponadto, jeśli używamy "@Rule", mamy do czynienia z tak dużą ilością kodu boilerplate. Tak więc, jeśli możesz zainstalować nowe biblioteki dla swoich testów, sugeruję, aby spojrzeć na AssertJ (ta Biblioteka jest teraz dostarczana z SpringBoot)

Następnie test, który jest nie naruszając zasad "given / when / then", a robi się to za pomocą AssertJ do weryfikacji:

1 - wyjątek jest tym, czego oczekujemy. 2-ma również oczekiwaną wiadomość

Będzie wyglądać tak:

 @Test
void should_throwIllegalUse_when_idNotGiven() {

    //when
    final Throwable raisedException = catchThrowable(() -> getUserDAO.byId(null));

    //then
    assertThat(raisedException).isInstanceOf(IllegalArgumentException.class)
            .hasMessageContaining("Id to fetch is mandatory");
}

Jeśli używasz @Rule, zestaw wyjątków jest stosowany do wszystkich metod testowych w klasie testowej.

Podoba mi się odpowiedź user64141, ale okazało się, że może być bardziej uogólniona. Oto moje zdanie:

public abstract class ExpectedThrowableAsserter implements Runnable {

    private final Class<? extends Throwable> throwableClass;
    private final String expectedExceptionMessage;

    protected ExpectedThrowableAsserter(Class<? extends Throwable> throwableClass, String expectedExceptionMessage) {
        this.throwableClass = throwableClass;
        this.expectedExceptionMessage = expectedExceptionMessage;
    }

    public final void run() {
        try {
            expectException();
        } catch (Throwable e) {
            assertTrue(String.format("Caught unexpected %s", e.getClass().getSimpleName()), throwableClass.isInstance(e));
            assertEquals(String.format("%s caught, but unexpected message", throwableClass.getSimpleName()), expectedExceptionMessage, e.getMessage());
            return;
        }
        fail(String.format("Expected %s, but no exception was thrown.", throwableClass.getSimpleName()));
    }

    protected abstract void expectException();

}

Zauważ, że pozostawienie instrukcji "fail" w bloku try powoduje przechwycenie odpowiedniego wyjątku twierdzenia; użycie return w instrukcji catch zapobiega temu.

Zaimportuj bibliotekę Catch-exception i użyj jej. Jest znacznie czystsza niż ExpectedException lub try-catch.

Przykład z ich dokumentów:

import static com.googlecode.catchexception.CatchException.*;
import static com.googlecode.catchexception.apis.CatchExceptionHamcrestMatchers.*;

// given: an empty list
List myList = new ArrayList();

// when: we try to get the first element of the list
catchException(myList).get(1);

// then: we expect an IndexOutOfBoundsException with message "Index: 1, Size: 0"
assertThat(caughtException(),
  allOf(
    instanceOf(IndexOutOfBoundsException.class),
    hasMessage("Index: 1, Size: 0"),
    hasNoCause()
  )
);

Wolałbym AssertJ do tego.

        assertThatExceptionOfType(ExpectedException.class)
        .isThrownBy(() -> {
            // method call
        }).withMessage("My message");

@Test (expectedExceptions = ValidationException.class, expectedExceptionsMessageRegExp = "This is not allowed")
public void testInvalidValidation() throws Exception{
     //test code
}