Testing JUnit5: try ... catch arbeitet nicht sauber

[TM69]

Ich habe eine Spring - Funktion:

    public FieldDescription createFieldDescription(User creator, FieldDescription fieldDescription) throws ResourceNotFoundException {
        
        if (creator == null)
            throw new IllegalArgumentException("The creator must not be zero");
        
        if (fieldDescription == null)
            throw new IllegalArgumentException("The adding field description must not be zero");
        
        userRepository.findById(creator.getId())
            .orElseThrow(() -> new ResourceNotFoundException("Creator not found with the id :: " + creator.getId()));
        
        fieldDescription.setId(sequenceGeneratorService.generateSequence(FieldDescription.SEQUENCE_NAME));
        fieldDescription.setCreatedBy(creator);
        fieldDescription.setCreatedAt(LocalDateTime.now());
        
        return fieldDescriptionRepository.save(fieldDescription);
    }

und eine zugehörige JUnit5 Test-Funktion

    @Test
    void testCreateWiFieldDescriptionthWithNotExistsUserShouldThrowResourceNotFoundException() throws ResourceNotFoundException {
        // create a basic user
        String firstname = "firstname";
        String lastname = "lastname";
        String email = "email@email.com";
        String password = "password";
        
        User user = new User(
                firstname,                // password
                lastname,                 // lastname
                email,                     // email
                password                // password
                );
        boolean exception = false;
        
        try {
            userService.createUser(null, user);
        } catch (ResourceNotFoundException e) {
            exception = true;
            e.printStackTrace();
        }

        assertFalse(exception, "testCreateWiFieldDescriptionthWithNotExistsUserShouldThrowResourceNotFoundException(): Couldnt create user");
        assertNotNull(user);
        
        // create basic field description
        user.setId(999999);                   // set user to an illagel index
        
        String name = "aFieldname";
        EFieldType fieldType = EFieldType.TEXT;
        String options = "options";
        
        FieldDescription fieldDescription = new FieldDescription(name, fieldType, options);
        
        exception = false;
        
        try {
            fieldService.createFieldDescription(user, fieldDescription);
        } catch (ResourceNotFoundException e) {
            exception = true;
            e.printStackTrace();
        }
        
        assertTrue(exception, "testCreateWiFieldDescriptionthWithNotExistsUserShouldThrowResourceNotFoundException(): FieldDescription could be create");

        // clean up
        userService.removeUser(user.getId());
    }

Ich sehe das die zu testende Funktion wirklich eine ResourceNotFoundException wirft, wenn ein nicht existende User ID übergeben wurde. Oder anders gesagt der Abschnitt

    userRepository.findById(creator.getId())

            .orElseThrow(() -> new ResourceNotFoundException("Creator not found with the id :: " + creator.getId()));

arbeitet sauber, aber in der Test-Funktion an der Stelle

        try {
            fieldService.createFieldDescription(user, fieldDescription);
        } catch (ResourceNotFoundException e) {
            exception = true;
            e.printStackTrace();
        }

scheint es fast so als würde catch nicht anspringen.
Denn

        assertTrue(exception, "testCreateFieldDescriptionthWithNotExistsUserShouldThrowResourceNotFoundException(): FieldDescription could be create");

wird mit true quitiert und nicht wie eigentlich gedacht als false.

 

[thecain]

In Junit5 gibt es assertThrows welches genau für solche Szenarien da ist

 

[httpdigest]

in der Test-Funktion an der Stelle

        try {
            fieldService.createFieldDescription(user, fieldDescription);
        } catch (ResourceNotFoundException e) {
            exception = true;
            e.printStackTrace();
        }

scheint es fast so als würde catch nicht anspringen.
Denn

        assertTrue(exception, "testCreateFieldDescriptionthWithNotExistsUserShouldThrowResourceNotFoundException(): FieldDescription could be create");

wird mit true quitiert und nicht wie eigentlich gedacht als false.

Diese Angaben sind widersprüchlich. Du schreibst, dass es so scheint, als würde der catch-Block nicht ausgeführt werden, aber anschließend schreibst du, dass assertTrue(exception, ...) ja ohne AssertionError durchläuft. Also wurde doch der catch-Block durchlaufen, weil nur dann auch die boolean Variable exception auch auf true gesetzt wird. Also: Es hat alles ganz genau so funktioniert, wie von dir erwartet.

Nichtsdestotrotz ist diese Art der Prüfung und die Verwendung von boolean Variablen hierfür nicht sinnvoll. Siehe Nachricht von @thecain .

 

[TM69]

In Junit5 gibt es assertThrows welches genau für solche Szenarien da ist

Danke, aber wie wird denn eine negation geschrieben? Oder mit anderen Worten gefragt: Wie überprüfe ich, das keine Exception geworfen wurde?

 

[KonradN]

Wenn eine Exception nicht gefangen wird, ist der Test an der Stelle mit der Exception automatisch fehlgeschlagen.
Daher ist sowas nicht explizit zu testen.

 

[TM69]

Wenn eine Exception nicht gefangen wird, ist der Test an der Stelle mit der Exception automatisch fehlgeschlagen.
Daher ist sowas nicht explizit zu testen.

Das hab ich mir schon gedacht, allerdings frage ich mich wie ich z.B. folgenden Test anhand der von theCain geschriebenen Antworten "besser" schreiben könnte.

        // initialize: create a basic user
        String firstname = "firstname";
        String lastname = "lastname";
        String email = "email@email.com";
        String password = "password";
        
        User user = new User(
                firstname,                // password
                lastname,                 // lastname
                email,                     // email
                password                // password
                );
        
        ResourceNotFoundException thrown = Assertions.assertThrows(FieldDescription.class, () -> {
               //Code under test
      });

      Assertions.assertEquals("some message", exception.getMessage());
      
        boolean exception = false;
        
        try {
            userService.createUser(null, user);
        } catch (ResourceNotFoundException e) {
            exception = true;
            e.printStackTrace();
        }
        
        assertFalse(exception, "testCreateWiFieldDescriptionthMinimalistValidData(): Couldnt create user");
        
        // test: create basic field description
        String name = "aFieldname";
        EFieldType fieldType = EFieldType.TEXT;
        String options = "options";
        
        FieldDescription fieldDescription = new FieldDescription(name, fieldType, options);
        
        exception = false;
        
        try {
            fieldService.createFieldDescription(user, fieldDescription);
        } catch (ResourceNotFoundException e) {
            exception = true;
            e.printStackTrace();
        }
        
        assertFalse(exception, "testCreateFieldDescriptionthMinimalistValidData(): Couldnt create FieldDescription");
        
        Optional<FieldDescription> foundFieldDescription = fieldService.findFieldDescriptionById(fieldDescription.getId());
        
        assertTrue( foundFieldDescription.isPresent(), "testCreateFieldDescriptionthMinimalistValidData(): Not found FieldDescription" );
        
        assertEquals(name, foundFieldDescription.get().getName(), "testCreateWiFieldDescriptionthMinimalistValidData(): Name of FieldDescription not identical");
        assertEquals(fieldType, foundFieldDescription.get().getFieldType(), "testCreateWiFieldDescriptionthMinimalistValidData(): FieldType of FieldDescription not identical");
        assertEquals(options, foundFieldDescription.get().getOptions(), "testCreateWiFieldDescriptionthMinimalistValidData(): Options of FieldDescription not identical");
        
        // clean up
        fieldService.removeFieldDescription(fieldDescription.getId());
        userService.removeUser(user.getId());

 

[httpdigest]

Wann immer du sowas schreiben wolltest (Exception soll nicht fliegen):

exception = false;
try {
    fieldService.createFieldDescription(user, fieldDescription);
} catch (ResourceNotFoundException e) {
    exception = true;
    e.printStackTrace();
}
assertFalse(exception, "testCreateFieldDescriptionthMinimalistValidData(): Couldnt create FieldDescription");

schreibst du stattdessen jetzt einfach:

fieldService.createFieldDescription(user, fieldDescription);

Und statt (Exception soll fliegen):

exception = false;
try {
    fieldService.createFieldDescription(user, fieldDescription);
} catch (ResourceNotFoundException e) {
    exception = true;
    e.printStackTrace();
}
assertTrue(exception, "testCreateWiFieldDescriptionthWithNotExistsUserShouldThrowResourceNotFoundException(): FieldDescription could be create");

schreibst du jetzt halt:

assertThrows(ResourceNotFoundException.class, () -> fieldService.createFieldDescription(user, fieldDescription));