Ejdamm / market-board

A lighweight marketplace written with PHP Slim framework
MIT License
1 stars 0 forks source link

Issue43 - Listingklass #49

Closed Ejdamm closed 4 years ago

Ejdamm commented 4 years ago

Issue #43 Databasanropen som rör listings har brutits ut i en egen klass.

La testerna under integrationstest eftersom både klassen och dess anslutning mot databasen testas, men ändå inte hela kedjen så passar inte i functionaltests. Kände inte att det var värt att skapa en basklass som det är gjort i functional. Men jag koperiade två funktioner därifrån så nu finns identiska kopior på två ställen. Ska vi bryta ur dem och lägga på annat ställe högre upp så båda kan nå dem? Det rör sig om clearDatabaseOf och createPDOPreparedConditions

För att kunna använda setUpBeforeClass() och tearDownAfterClass() behövde fler saker göras statiska.

I och med att jag döpte om en phinxmigration så gick det inte att backa den utan jag fick manuellt ta bort phinxtabellen.