mockito Tests

[Mariexshhx]

  @Test
  @DisplayName("Begriff kann hinzugefügt werden")
  void test1() {
    Begriff hallo = service.begriffHinzufuegen("hallo");
   verify(repo).save(new Begriff("hallo"));
  }

  @BeforeEach
  void getService() {
    repo = mock(GlossarRepository.class);
    service = new GlossarService(repo);

  }

Hallo, ich möchte meine Service Methode testen und verifyen, dass save mit dem richtigen Begriff aufgerufen wird. Ich bekomme, aber eine Nullpointer Exception, weil die id NULL ist, denn die id wird erst in der DB gesetzt. Ich mocke das Repository jedoch und möchte nur wissen, ob safe mit dem richtigen Begriff aufgerufen wird. Wieso muss für verify dann eine id gesetzt sein?

 

[KonradN]

Wenn das die GlossarService Klasse, die Du testen willst, so verlangt, dann ist es so. Wir kennen die Klasse ja nicht.

Wenn der GlossarService irgendwas erwartet von dem GlossarRepository, dann musst Du dieses Verhalten ganz klar vorgeben. Das und nichts anders macht das Mocking: Du bekommst da eine Instanz ohne viel Arbeit und Du kannst da vorgeben, was zurück gegeben werden soll!

Also schau dir die Methode begriffHinzufuegen an und überlege, was da aufgerufen wird. Und bei Aufrufen vom GlossarRepository kannst Du Dir überlegen, was du da testen willst und was Du da dann halt als Rückgabe von Aufrufen brauchst.

 

[Mariexshhx]

  public Begriff begriffHinzufuegen(String name) {
    Begriff begriff = new Begriff(name);
    return repository.save(begriff);
  }

Wenn das die GlossarService Klasse, die Du testen willst, so verlangt, dann ist es so. Wir kennen die Klasse ja nicht.

Wenn der GlossarService irgendwas erwartet von dem GlossarRepository, dann musst Du dieses Verhalten ganz klar vorgeben. Das und nichts anders macht das Mocking: Du bekommst da eine Instanz ohne viel Arbeit und Du kannst da vorgeben, was zurück gegeben werden soll!

Also schau dir die Methode begriffHinzufuegen an und überlege, was da aufgerufen wird. Und bei Aufrufen vom GlossarRepository kannst Du Dir überlegen, was du da testen willst und was Du da dann halt als Rückgabe von Aufrufen brauchst.

so sieht die Methode aus und ich wollte halt sicherstellen, dass der richtige Begriff gesavt wird

 

[Mariexshhx]

die id wird im Konstruktor auf null gesetzt

  @Test
  @DisplayName("Begriff kann hinzugefügt werden")
  void test1() {
    Begriff hallo = service.begriffHinzufuegen("hallo");
   verify(repo).save(hallo);
  }

Wenn ich es so probiere kriege ich diese Fehlermeldung

 

[Mariexshhx]

Wenn das die GlossarService Klasse, die Du testen willst, so verlangt, dann ist es so. Wir kennen die Klasse ja nicht.

Wenn der GlossarService irgendwas erwartet von dem GlossarRepository, dann musst Du dieses Verhalten ganz klar vorgeben. Das und nichts anders macht das Mocking: Du bekommst da eine Instanz ohne viel Arbeit und Du kannst da vorgeben, was zurück gegeben werden soll!

Also schau dir die Methode begriffHinzufuegen an und überlege, was da aufgerufen wird. Und bei Aufrufen vom GlossarRepository kannst Du Dir überlegen, was du da testen willst und was Du da dann halt als Rückgabe von Aufrufen brauchst.

und ich weiß ich kann vorgeben was das repo zurückgeben soll, aber das ist hier doch nicht sinnvoll oder?

 

[KonradN]

und ich weiß ich kann vorgeben was das repo zurückgeben soll, aber das ist hier doch nicht sinnvoll oder?

Das Problem hier ist, dass Du eben nicht so einfach verifizieren kannst auf ein neu erstelltes Objekt. Selbst wenn er Deine verify Vorgabe verstehen würde, hättest Du da ja eine andere Instanz als eben die Instanz, die da im Service erstellt wurde.

Daher würde ich da tatsächlich etwas vorgeben und dann sicher stellen, dass dies zurück gegeben wurde.

 

[Mariexshhx]

  @Test
  @DisplayName("Begriff kann hinzugefügt werden")
  void test1() {
    Begriff b = new Begriff(1,"hallo");
    when(repo.save(any())).thenReturn(b);
    Begriff hallo = service.begriffHinzufuegen("hallo");
    assertThat(hallo).isEqualTo(b);

  }

in etwa so?

 

[Mariexshhx]

  @Test
  @DisplayName("Begriff kann hinzugefügt werden")
  void test1() {
    Begriff b = new Begriff("hallo");
    when(repo.save(any())).thenReturn(b);
    Begriff hallo = service.begriffHinzufuegen("hallo");
    assertThat(hallo).isEqualTo(b);

  }

bzw so id muss dann nicht gesetzt werden

 

[KonradN]

Ich würde die Instanzen unterschiedlich machen. Denn Du willst ja wirklich sicher gehen, dass die Instanz, die von save zurück gegeben wurde, auch zurückgegeben wird. Wenn Begriff die equals Methode überschreiben sollte, dann wären zwei Instanzen, die mit new Begriff("hallo") erzeugt wurden, ggf. gleich.

Die Instanz kann also ganz unterschiedlich sein. Das kann also eine Instanz mit id sein aber auch etwas ganz anderes.

Es ginge also auch:

  @Test
  @DisplayName("Begriff kann hinzugefügt werden")
  void test1() {
    Begriff expected = new Begriff("SavedInstance");
    when(repo.save(any())).thenReturn(expected);
    Begriff saved = service.begriffHinzufuegen("WordToSave");
    assertThat(saved).isEqualTo(expected);
  }

• Die Instanzen sind also klar unterschiedlich
• Und die Variablen wirklich verständlich benennen. Wenn ich ein Ergebnis erwarte, dann ist das bei mir meist ein expected oder so.

 

[mrBrown]

Wenn dein Test erfolgreich läuft, solltest du in jedem Fall einmal die Service-Methode „kaputt“ machen. Möglichkeiten wären z.B. null statt des Strings übergeben oder an den String noch ein zufälliges Zeichen dran hängen - in beiden Fällen sollte der Test dann fehlschlagen.