search

SeidChr / AutoTagger

Proof of Concept for a Instagram Best-Tag-Finder
1 stars 1 forks source link

Tests #3

Closed DariosKrimsKrams closed 6 years ago

DariosKrimsKrams commented 6 years ago

Tests: Die Aktuellen überarbeiten, Dependencies mocken als Unit u. Acceptance-Tests verpacken, DI überhaupt erstmal ordentlich bastlen

SeidChr commented 6 years ago

Habe ein xunit test projekt angelegt für komponenten tests. habe auch schon ein paar unserer consolen integration tests dorthin verschoben. leider sind es nach wie vor integration tests.

DariosKrimsKrams commented 6 years ago

closed weil keine Priorisierung gerade

SeidChr commented 6 years ago

Tests sollten wir permanent parallel entwickeln. Kein bedarf für ein extra Ticket an dieser stelle.

DariosKrimsKrams commented 6 years ago

Ja stimmt. Jedoch sollten wir die Tests mal richtig von vorne entwickeln! Die aktuelle Scheiße ist ja nicht zu gebrauchen :p Durch fehlendes Mock und DI sind Tests nicht vernünftig möglich, und meinerseits wenig Prio gerade drauf.

SeidChr commented 6 years ago

Es sollte da eine XUnit Test Klasse incl. weniger Mocks geben, in die ich mal einige Methoden aus der Test-Console verschoben habe. Hier ist auch der einzige fall den ich bisher kennengelernt habe bei dem DI und IoC nicht äquivalent zu verwenden sind :) DI braucht man nicht für Unittests. Man kann DI verwenden wenn man seinen Code nach IoC so strukturiert, dass man ihn auch sauber testen kann. das geht Hand in Hand. Aber tatsächlich war es zumindest bei meinem letzten Überblick alles im weitesten sinne nach diesem Paradigma ausgestaltet.

DariosKrimsKrams commented 6 years ago

ließ ma Slack ;)