Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1454] Melding ivm corona / COVID19 weghalen van MP's orderbevestiging mails
  1. … 1 more file in changeset.
Probeer dubbele constructies zoals $$$Not ($$$HasLength ()) te vermijden door simpelweg te schrijven :         If (Boodschap = "") { [dat heb je voor als de reviewer een dyslexie-afwijking heeft...

Probeer dubbele constructies zoals $$$Not ($$$HasLength ()) te vermijden door simpelweg te schrijven :

        If (Boodschap = "") { 


[dat heb je voor als de reviewer een dyslexie-afwijking heeft :-P ]

ICT-1373: SFS kalender rekening houden met sluiting door corona, aangepaste leverdatum voorstellen
ICT-1373: SFS kalender rekening houden met sluiting door corona, aangepaste leverdatum voorstellen
[ICT-1373] SFS kalender rekening houden met sluiting door corona, aangepaste leverdatum voorstellen

Melding weergeven op de mail die naar de MP wordt gestuurd via de BesteldeOfferteMelder/MailVerzender/Handel

  1. … 1 more file in changeset.
[UW] Layout mail bestelde offerte beetje veranderen volgens aanwijzigen DTI.

- dank u DTI

  1. … 5 more files in changeset.
[UST4047] Vh4: MP: Beslag Design: niet meeleveren aanvinken bij "migratie" winkelkar voor doorbestellen

- review opmerkingen

  1. … 2 more files in changeset.
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)

Bij nieuwe code mag je api's newen

Bij nieuwe code mag je api's newen

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

- Falende testjes

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...
hier gingen toch ook nog Vhisie4 = $$$True testjes toegevoegd worden?

hier gingen toch ook nog Vhisie4 = $$$True testjes toegevoegd worden?

[VRB541] Vhisie4: Klant en referentie worden niet ingevuld in mail naar handel: aangepaste tests
  1. … 2 more files in changeset.
[VRB541] Vhisie4: Klant en referentie worden niet ingevuld in mail naar handel
[VRB541] Vhisie4: Klant en referentie worden niet ingevuld in mail naar handel