Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[DEF742] : EC: Sync winkelkar: timeout bij grote winkelkarren

- mock over het hoofd gezien. alleen de gegenereerde %OnNew() is er bij gekomen.

  • More
  • CR-532
  • finished reviewing
#dimmekes nakijken zie GeefMaatwerkLijnen()

#dimmekes nakijken zie GeefMaatwerkLijnen()

Ofwel test je dat MaatwerkLijn1 en MaatwerkLijn2 in de lijst zitten als je dat niet doet is het totaal niet noodzakelijk om daar variabelen aan te verspillen

Ofwel test je dat MaatwerkLijn1 en MaatwerkLijn2 in de lijst zitten als je dat niet doet is het totaal niet noodzakelijk om daar variabelen aan te verspillen

Lijkt niet echt te kloppen As WS.vhisie4.Winkelkar.dto.MaatwerkLijn ; Misschien beter om AssertEquals(WinkelkarUtils.GeefMaatwerkLijnen(Winkelkar).Count(),0) te schrijven dan kan de dim niet verke...

Lijkt niet echt te kloppen As WS.vhisie4.Winkelkar.dto.MaatwerkLijn ;

Misschien beter om AssertEquals(WinkelkarUtils.GeefMaatwerkLijnen(Winkelkar).Count(),0) te schrijven dan kan de dim niet verkeerd zijn

[UST2039] Vh4_SVC: Sync web naar caché
[UST2039] Vh4_SVC: Sync web naar caché
[UST2039] Vh4_SVC: Volwaardige sync web naar caché

- Mock - Fake zijn toegevoegd

  1. … 3 more files in changeset.
[UST2039] Vh4_SVC: Volwaardige sync web naar caché

- Herwerkt zodat ook productlijnen gesynchroniseerd kunnen worden.

- Voor herbereken blijft het enkel maatwerklijen.

- nem. :P

  1. … 20 more files in changeset.
Best even bespreken. Lijkt me niet zozeer een probleem dat mat berekend wordt op serverkant en de front-end dat niet meegeeft. Lijkt me ook niet zinvol om zelf te berekenen en te vergelijken met on...

Best even bespreken. Lijkt me niet zozeer een probleem dat mat berekend wordt op serverkant en de front-end dat niet meegeeft.
Lijkt me ook niet zinvol om zelf te berekenen en te vergelijken met onze eigen berekeningen.

  • More
  • CR-523
  • finished reviewing
Ik vind dit maar een zeer gevaarlijke constructie - beter het echte achterliggend probleem aanpakken.

Ik vind dit maar een zeer gevaarlijke constructie - beter het echte achterliggend probleem aanpakken.

[UW404] vh4 : Foutmelding op IKIDHash en timeout bij herbereken maatwerklijnen als het gaat over...
[UW404] vh4 : Foutmelding op IKIDHash en timeout bij herbereken maatwerklijnen als het gaat over...
[UW404] vh4 : Foutmelding op IKIDHash en timeout bij herbereken maatwerklijnen als het gaat over winkelkarrren met 67 maatwerklijnen.

- vergelijken op canonische waarde is nodig, maar nog niet correct omdat mat afmetingen ingevuld worden. Deze worden dus leeggemaakt en hoeven we niet te vergelijken.

    • -0
    • +20
    ./Kenmerkenvergelijker.cls.xml
  1. … 2 more files in changeset.
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
[UST2682] vh4 : vervallen offertes

- opkuis code die nog niet nodig is.

- voorbereiden testen - inject..

  1. … 6 more files in changeset.
[UST2682] vh4 : vervallen offertes

- poc. via soapUI ok. Dus beslist om geen updates (zetingegevenkenmerken voor TBX gaf n,og probleem.) uti te voeren, maar delete en toevoegen.

- testen nog niet ok. in progress

  1. … 5 more files in changeset.
[UST2682] vh4 : vervallen offertes

- opkuis voor poc - testen nog niet ok.

  1. … 2 more files in changeset.
[UST2682] vh4 : vervallen offertes

- Aangevuld wat mogelijks nodig is voor product en maatwerklijnen ... verder te bekijken / bespreken.

- Prijs wijzigingen vhisie4 vs Cache ?

  1. … 1 more file in changeset.
Ik zou dan voorstellen dat je zelf ook geen .ToArray gebruikt in je testen. Commit 39782 Joc en commit 39550 TomV -> vhUnitTest.WSimpl.Vhisie4.Winkelkar.EconConfiguratieConverter.Test

Ik zou dan voorstellen dat je zelf ook geen .ToArray gebruikt in je testen. Commit 39782 Joc en commit 39550 TomV -> vhUnitTest.WSimpl.Vhisie4.Winkelkar.EconConfiguratieConverter.Test

  • More
  • CR-136
  • finished reviewing
Volgens mij gaat deze test identiek hetzelfde testen dan één van de LijnCacheBuilder testen .. DRY

Volgens mij gaat deze test identiek hetzelfde testen dan één van de LijnCacheBuilder testen .. DRY