Add version tests for quotes, reviews and signups

[dexbleeker]

Depends on #256

[jackozi]

Ergens wel onnodig om te testen of de gem werkt (versions) dat hoort eigenlijk gewoon in de gem gefixt te zijn.

[dexbleeker]

Ik kwam op het idee om te testen of ze er zijn (niet perse of de gem werkt) door dat verhaal met de trail van api keys. Kunnen we dat misschien op een betere manier testen misschien? Anders kan het misschien wel in 1 test ipv losse als je dat liever hebt.

[jackozi]

Of ze er zijn is toch testen of de gem werkt 😅

[dexbleeker]

Nou vooral op welke attributen hij wel en niet aanslaat, dus eigenlijk test je er onze has_paper_trail, except: [.. bijvoorbeeld mee IMO

[jackozi]

Dat is wel een goed punt. Zou wel fancy zijn om dan een helper te hebben zodat we iets van asserts_trail_on [attributes] hebben maar dat kunnen we in de toekomst wel fixen eventueel

[jackozi]

Nog even over nagedacht en het enige wat we dan aan het doen zijn is op twee plekken definiëren 1x in de model en 1x in de test. Ik zou er hier wel gewoon vanuit gaan dat de gem zijn eigen testen doet en het bij ons gewoon laten.

[dexbleeker]

Aan de ene kant helemaal met je eens, aan de andere kant is het nog een keer definiëren (beschrijven van gedrag) het hele punt van tests. Ik denk nog even na over of/hoe ik dit wil aanpakken.

[jackozi]

Niet helemaal mee eens, tests schrijf je om te verifiëren dat de code die je schrijft de verwachting die je hebt waar maakt, wel een interessante discussie en dit punt is best een twijfel geval.