MaatwerkLijn.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[DEF681] VH4: BUDI: HerberekenMaatwerk =} Mail offerte overzicht: oude offertes blijven dagelijks...
[DEF681] VH4: BUDI: HerberekenMaatwerk =} Mail offerte overzicht: oude offertes blijven dagelijks...
Bij de verwachtmethodcall op de WinkelkarServiceMock heb je overal een lege string meegegeven wat op zich niet fout is aangezien deze een default waarde heeft. Andere parameters die voor deze metho...

Bij de verwachtmethodcall op de WinkelkarServiceMock heb je overal een lege string meegegeven wat op zich niet fout is aangezien deze een default waarde heeft. Andere parameters die voor deze method ook optioneel zijn, worden ingevuld met een random waarde. Mss kan je naar analogie ook hier een RandomIsGeldig meegeven.

Idem opmerking hierboven

Idem opmerking hierboven

Bij voorkeur #dim (mét type), Set wordt ook toegelaten, een tussenvorm zou ik vermijden.

Bij voorkeur #dim (mét type), Set wordt ook toegelaten, een tussenvorm zou ik vermijden.

[DEF681] VH4: BUDI: HerberekenMaatwerk =} Mail offerte overzicht: oude offertes blijven dagelijks vermeld (door herberekenmaatwerk probleem met ongeldige lades)

- op een winkelkar maatwerklijn kunnen opslaan dat ze in vhisie4 ongeldig is en niet herberekend mag worden.

- wordt nergens op false gezet, tenzij als default bij creatie, omdat een update aan een lijn, het verwijderen en terug aanmaken ervan impliceert.

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

- Gebruik maken van MeeleverenBepaler + factory

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

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

- DoorHandelMeeleverens op Winkelkar.MaatwerkLijn zetten

- first make it work, refactor volgt.

  1. … 4 more files in changeset.
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...
[UST3920] EC: app: herbestel bij probleem: DI1 en nog niet verwerken
[UST3920] EC: app: herbestel bij probleem: DI1 en nog niet verwerken
[UST392] EC: app: herbestel bij probleem: DI1 en nog niet verwerken

- Maken dat we HerbestellingBijProbleem kunnen opslaan

[VRB530] Vh4: EconConfiguratieSleutel doorgeven
[VRB530] Vh4: EconConfiguratieSleutel doorgeven