Testen Teil 1

[murmal]

• Geschwindigkeit (Video spielt ohne Artefakte ab)

• myfs (FUSE-Methoden) in progress

  • getAttr
  • open
  • read
  • readDir (?)
  • release
  
  

• dmap

  • get/set-all
  • clear
  • set
  • getFreeBlock
  
  

• fat

  • get/set-all
  • addLastToFAT
  • addToFAT not tested because not used
  • addNextToFAT
  • deleteFromFAT
  • iterateFAT
  • getNext
  
  

• root

  • get/set-all
  • get
  • createEntry
  • deleteEntry
  • update
  • exists
  • getName
  
  

• filesystemIO

• mkfs.myfs
  
  

Aufbau der Tests

Pro Klasse die getestet wird existiert eine Testklasse.
Test_Case(s) sind jeweils für logisch zusammenhängende Methoden angesetzt.
Section(s) innerhalb der Test_Case(s) sind für einzelne Testfälle dieser Methoden gedacht.
Wenn es viele Testfälle zu einer Methode gibt können auch mehrer Section(s) in einem TEST_CASE gebündelt werden wodurch nach der Benennung der unter Test stehenden Methoden mit einem Komma getrennt eine zusammenfassende Beschreibung der Section(s) notiert werden soll.

Benennung

test-Klasse_unter_Test.cpp
TEST_CASE("Klasse_unter_Test.Methode_unter_Test / Klasse_unter_Test.Weitere_Methode_unter_Test, Beschreibung", "[Klasse_unter_Test]")
SECTION("Beschreibung des Einzelnen Testfalls")

Durchführung der Test für Fuse

Die myfs wird in jedem TestCase neu instanziiert. Dadurch können die Fuse-Methoden direkt getestet werden.

[HannesIlly]

Habe Test cases für getFreeBlock hinzugefügt. FindFreeBlock ist da ja auch mit drin (kann man nicht testen, weil privat). Ist glaub ich vollständig.
Die Tests sind in mehrere test cases mit mehreren sections aufgeteilt. Wenn das anders geplant war ggf. anpassen.

[HannesIlly]

IsAddressFull ist auch privat. Wird aber glaub ich sogar nie benutzt.

[HannesIlly]

Ich habe bei get/setAll mal den Haken rausgemacht, weil da noch die Assertions gefixt werden müssen.

[HannesIlly]

Bei den FAT tests muss man mindestens bei set-/getAll #39 beachten (Wenn man gekapselt testet; wenn nicht, dann bei allen Methoden). Wenn es bei uint16_t bleibt muss man nichts ändern.

[HannesIlly]

Habe grad Test cases für die Root hinzugefügt (ist meist ein Test für jeden möglichen Fall).
Ich habe nur die Methoden getestet, die wir gerade benutzen (die Methoden mit char* name als Parameter und update).
Bitte drüberschauen und ggf. abhaken.