Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UW700] ConfirmOrder: (UT) ECommerceAPI niet gebruiken bij $$$Inject
[UW700] ConfirmOrder: (UT) ECommerceAPI niet gebruiken bij $$$Inject
  • More
  • CR-751
  • finished reviewing
Waarom gebruik je $$$ElkeWaarde bij de IndienMethodcall ? Dat wil je toch juist testen, want daar zat de bug in de code, hé? Je krijgt 'em van mij cadeau http://subversion02.vanhoecke.be/static/ogd...

Waarom gebruik je $$$ElkeWaarde bij de IndienMethodcall ?
Dat wil je toch juist testen, want daar zat de bug in de code, hé?
Je krijgt 'em van mij cadeau
.IndienMethodCall("Bepaal",PersonalityID, ..#KlantID)

[DEF626] OfferteBestellingValidator - foutmelding Personality ID op IsStandaardTransport
[DEF626] OfferteBestellingValidator - foutmelding Personality ID op IsStandaardTransport
excepties van gemaakt.

excepties van gemaakt.

LeverDatum als %String terwijl de interface zegt dat het een %Date moet zijn ??? Kijk eens naar de andere test-cases in deze klasse ook want hier is er toch het een en ander niet juist. Ofwel krij...

LeverDatum als %String terwijl de interface zegt dat het een %Date moet zijn ???

Kijk eens naar de andere test-cases in deze klasse ook want hier is er toch het een en ander niet juist.
Ofwel krijgt de init een %String binnen en gaat deze de datum omvormen ofwel gaat iedere testcase de init correct aanroepen.

Een VanHoeckeOfferte is toch altijd een vhisie4 versie. Ofwel corrigeren ofwel is het dode code die opgekuist mag worden.

Een VanHoeckeOfferte is toch altijd een vhisie4 versie.

Ofwel corrigeren ofwel is het dode code die opgekuist mag worden.

Het zijn ..Stubs ipv ..Mocks - kwestie van het juist te hebben

Het zijn ..Stubs ipv ..Mocks - kwestie van het juist te hebben

Dat durf ik niet met zekerheid te zeggen. De andere exceptieklasse voor de offertefoutmelding die overerft van invalidinput exceptie, past ook de exceptiecode niet aan.Mogelijks rekent andere softw...

Dat durf ik niet met zekerheid te zeggen. De andere exceptieklasse voor de offertefoutmelding die overerft van invalidinput exceptie, past ook de exceptiecode niet aan.Mogelijks rekent andere software op deze exceptiecode. ???

excepties gebruikt ipv tekst of codes.

excepties gebruikt ipv tekst of codes.

  • More
  • CR-511
  • finished reviewing
Lijkt mij kweet niet hoe gevaarlijk om hier enkel op tekst te vergelijken. Hoe zou beter zijn dat de foutmeldingen een lijst van meldingen worden met ipv een string een melding object met daarin ee...

Lijkt mij kweet niet hoe gevaarlijk om hier enkel op tekst te vergelijken. Hoe zou beter zijn dat de foutmeldingen een lijst van meldingen worden met ipv een string een melding object met daarin een code en een string en kan er per code makkelijker gemapped worden welke exceptie te gooien.
Als code kunnen we misschien de exceptie klasse naam gebruiken zo kunnen we iets generiek maken om de exceptie te throwen

We er voor zorgen dat je een andere foutcode naar buiten gooit

We er voor zorgen dat je een andere foutcode naar buiten gooit

TECH.Interface ipv TECH.Mockable

TECH.Interface ipv TECH.Mockable

[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar
[UST2758] Vh4: Geldigheid prijzen: beslag-winkelkarren niet controleren en BestelWinkelkar
Zal Als apart kaartje moeten behandelen want de geldigheidsprijzen op maatwerklijnen worden niet ingevuld en is iets meer werk als enkel refactoring.

Zal Als apart kaartje moeten behandelen want de geldigheidsprijzen op maatwerklijnen worden niet ingevuld en is iets meer werk als enkel refactoring.

  • More
  • CR-506
  • finished reviewing
Beetje onletten met benamingen -> Offerte en Winkelkar niet door elkaar halen Ook al wordt Vhisie3 vorm ( Offertes ) gebruikt , het blijven winkelkarren in Vhisie4.

Beetje onletten met benamingen -> Offerte en Winkelkar niet door elkaar halen

Ook al wordt Vhisie3 vorm ( Offertes ) gebruikt , het blijven winkelkarren in Vhisie4.

IsVerschillend lijkt mij een betere naam.

IsVerschillend lijkt mij een betere naam.

Waarom wordt dit terug van de Offerte gehaald ?? Is het niet logischer dat de MaatwerkLijn correct aangepast wordt ???

Waarom wordt dit terug van de Offerte gehaald ?? Is het niet logischer dat de MaatwerkLijn correct aangepast wordt ???

[UST2682] vh4 : vervallen offertes
[UST2682] vh4 : vervallen offertes
Eigenlijk is nen array met lijsten van events. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif maar ik zal er array van maken.

Eigenlijk is nen array met lijsten van events.
maar ik zal er array van maken.

  • More
  • CR-284
  • finished reviewing
AddEventToArrayList ?? Wuk is het nu Array of List ???

AddEventToArrayList ?? Wuk is het nu Array of List ???