isotope / core

Core repository of Isotope eCommerce, an eCommerce extension for Contao Open Source CMS
https://isotopeecommerce.org
136 stars 107 forks source link

Add PHPUnit, fixtures, testcases for CompatibilityHelper (Adaption to Contao 5 Step 007) #2497

Closed Ernestopheles closed 10 months ago

Ernestopheles commented 10 months ago

Fixtures based on isotope-demo 2.8.1

After merging my PR there I am going to reinstall the Demo and export the database to create a new fixture file.

aschempp commented 10 months ago

aus meiner Sicht machen Tests für den CompatibilityHelper an dieser Stelle keinen Sinn. Die Methoden sind ja mehr oder weniger Einzeiler, und deren Funktion selber ist bereits getestet. Für ein Update auf Contao 5 sollten wir uns um Core-Probleme kümmern, nicht um "Grundlagen" wie das Testframework.

Ernestopheles commented 10 months ago

Aus meiner Berufserfahrung: ein Kollege meinte, er hätte ja nur eine Zuweisung eingebaut, das brauche man doch nicht zu testen. Jedoch hat es genau dort gekracht.

Ernestopheles commented 10 months ago

Außerdem werden wir die Tests noch brauchen, wenn ich den Ersatz für Haste/Units einbaue

aschempp commented 10 months ago

ich denke aber nicht dass wir dafür Functional Tests brauchen?

Ernestopheles commented 10 months ago

Dafür vermutlich unit Tests, sehe ich auch so

Ernestopheles commented 10 months ago

OK, dann lassen wir die Tests erstmal raus und ich benutze sie nur lokal.