WinkelkarExport

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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

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

- Bijkomende prop op winkelkar om aan te geven dat prijzen niet mogen overgenomen worden.

- reality kicks in : WinkelkarExport bevat Winkelkar.Maatwerklijn en niet WinkelkarExport.MaatwerkLijn

- zie DEF714 (nog uit te voeren)

  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...
geen tests? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

geen tests?

[UW462] bestelopmerkingen toevoegen aan winkelkarexport voor vhisie4
[UW462] bestelopmerkingen toevoegen aan winkelkarexport voor vhisie4
[UW462] bestelopmerkingen toevoegen aan wineklkarexport voor vhisie4
  1. … 1 more file in changeset.
[UST3684] [EPIC86] backporten van accept-code om daro en hein te doen werken

- Wensleverdatum toegevoegd op winkelkarexport-dto

[UST3001][EPIC50] Vh4: Verzendingmogelijkheden ook meegeven

- VerzendingMogelijkheiden worden op het vhisie4 export bestand geplaatst

    • -0
    • +52
    ./dto/VerzendingMogelijkheid.cls.xml
  1. … 1 more file in changeset.
  • More
  • CR-282
  • finished reviewing
[UST2486R] Vh4: Handels: Migratie na bestelling

- leveradres als APPS.common.dto.LeverAdres toevoegen

- nog wat testjes die moeten herwerkt worden

  1. … 5 more files in changeset.
Deze klasse mocht eigenlijk blijven bestaan en afhankelijk van welke offerte hij binnen kreeg ging door delegeren naar vhisie3 of vhisie4 versie, dan had je vorige opmerkingen qua fabriekje niet aa...

Deze klasse mocht eigenlijk blijven bestaan en afhankelijk van welke offerte hij binnen kreeg ging door delegeren naar vhisie3 of vhisie4 versie, dan had je vorige opmerkingen qua fabriekje niet aan uw been

Factory method hoort eigenlijk niet thuis op een service klasse

Factory method hoort eigenlijk niet thuis op een service klasse

Opkuisen

Opkuisen

Eigenlijk was het de moment om dit ServiceLocator patroon te vervangen door een fabriekje.

Eigenlijk was het de moment om dit ServiceLocator patroon te vervangen door een fabriekje.

Naamgeving van de klasse lijkt mij beter dat het gewoon OfferteConverter is en dat daar dan methods kunnen opkomen alla VanEcommerce en NaarEcommerce. Zodoende we maar één converter klasse hebben ...

Naamgeving van de klasse lijkt mij beter dat het gewoon OfferteConverter is en dat daar dan methods kunnen opkomen alla VanEcommerce en NaarEcommerce.

Zodoende we maar één converter klasse hebben die converteert in beide richtingen.