Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Komt er mogelijk bij als frontbevestiging ook niet mee geleverd mag worden...

Komt er mogelijk bij als frontbevestiging ook niet mee geleverd mag worden...

Iemand met de zelfde voornaam als ik heeft mij ooit gezegd van dat net niet te doen...

Iemand met de zelfde voornaam als ik heeft mij ooit gezegd van dat net niet te doen...

Dikke vinger syndroom

Dikke vinger syndroom

voot voot

voot voot

niet meeleverens. of zoiets. (best gelijk met test hierboven zeker?)

niet meeleverens. of zoiets. (best gelijk met test hierboven zeker?)

lijn mag weg

lijn mag weg

beter DOM.VKP.enu.Klant gebruiken? (idem bij Beslag)

beter DOM.VKP.enu.Klant gebruiken? (idem bij Beslag)

[BUS740] VH4: MP: Galimberti: personality + authenticatieservice initieel opzetten (nog zonder de...
[BUS740] VH4: MP: Galimberti: personality + authenticatieservice initieel opzetten (nog zonder de...
[BUS740] VH4: MP: Galimberti: personality + authenticatieservice initieel opzetten (nog zonder de afwijkingen van Galimberti)

- de test winkelkar is voor meerdere klanten bruikbaar , want klantid speelt hier geen rol meer. (stond op Hein, maar daarom op 1000 gezet)

- nog een test vergeten aanpassen.

Naamgeving stub ipv mock

Naamgeving stub ipv mock

Is deze test overbodig?

Is deze test overbodig?

casing

casing

Casing! (zeker bij nieuwe klasses)

Casing! (zeker bij nieuwe klasses)

[UST4047] Vh4: MP: Beslag Design: niet meeleveren aanvinken bij "migratie" winkelkar voor doorbestellen

- nog een gevolg van de review opm aanpassingen over het hoofd gezien.

Bij nieuwe code mag je api's newen

Bij nieuwe code mag je api's newen

TECH.Interface is niet genoeg om een mock te laten genereren.... Open / closed principe : extend,not modify inherited behaviour...

TECH.Interface is niet genoeg om een mock te laten genereren....
Open / closed principe : extend,not modify inherited behaviour...

Waarom nog een interface maken? Is de (stilzwijgende) afspraak niet om objecten te laten overerven van TECH.Mockable? In deze situatie zou je een Geen aanpasser als basis kunnen gebruiken of een ab...

Waarom nog een interface maken? Is de (stilzwijgende) afspraak niet om objecten te laten overerven van TECH.Mockable?
In deze situatie zou je een Geen aanpasser als basis kunnen gebruiken of een abstracte base-class.
Daarenboven kies je beter 1 manier om aan te geven dat je mocks/fakes wil: ofwel overerven van Interface ofwel van Mockable

Casing!

Casing!

Naamgeving kan iets explicieter. Bv. Test: NietMeeleverens worden aangepast

Naamgeving kan iets explicieter. Bv. Test: NietMeeleverens worden aangepast

Naast het maken van een "Standaard" (aanpasser) kon je hier ook een "Geen" (aanpasser) maken. Dit is mogelijk nog anders dan het standaardgedrag.

Naast het maken van een "Standaard" (aanpasser) kon je hier ook een "Geen" (aanpasser) maken. Dit is mogelijk nog anders dan het standaardgedrag.

Je zou kunnen aangeven in de test dat de KlantId en personalityId van de winkelkar gebruikt worden om de juiste aanpasser te maken.

Je zou kunnen aangeven in de test dat de KlantId en personalityId van de winkelkar gebruikt worden om de juiste aanpasser te maken.

[UST4047] Vh4: MP: Beslag Design: niet meeleveren aanvinken bij "migratie" winkelkar voor doorbestellen...
[UST4047] Vh4: MP: Beslag Design: niet meeleveren aanvinken bij "migratie" winkelkar voor doorbestellen...
[UST4047] Vh4: MP: Beslag Design: niet meeleveren aanvinken bij "migratie" winkelkar voor doorbestellen

- ExportWinkelKar aanpasser + factory

- test factory + test aanpassers

    • -0
    • +23
    ./PasAanTest.cls.xml
  1. … 13 more files in changeset.