Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST3817] VHISIE4 - Fix Synchroniseer: returntype van dummy:

- Returntype verwijderd, quit nu niets meer

  • 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é
zie hierboven

zie hierboven

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebru...

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebruiker"()
Ik heb in ieder geval een test toegevoegd die controleert dat er geen verzendingmogelijkheden berekend worden voor standaard klanten.

aangezien deploy wordt er geen aanpassing doorgevoerd.

aangezien deploy wordt er geen aanpassing doorgevoerd.

  • More
  • CR-227
  • finished reviewing
Wat met klanten zonder transportkeuze?

Wat met klanten zonder transportkeuze?

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen

ExactAantalKeer(1) om intentie duidelijk te maken

ExactAantalKeer(1) om intentie duidelijk te maken

Wat met klanten zonder transportkeuze? Of is dat niet relevant?

Wat met klanten zonder transportkeuze? Of is dat niet relevant?

Beter een ExactAantalKeer(0) op de MailAPIMock, om duidelijk de intentie te tonen

Beter een ExactAantalKeer(0) op de MailAPIMock, om duidelijk de intentie te tonen

beter de vhTest.Fake.TECH.Mail ?

beter de vhTest.Fake.TECH.Mail ?

Aangezien dit de clou is van de test, kunt ge er misschien beter een ExactAantalKeer(1) bijzetten?

Aangezien dit de clou is van de test, kunt ge er misschien beter een ExactAantalKeer(1) bijzetten?

testje voor 0 palletten toevoegen

testje voor 0 palletten toevoegen

$$$Not

$$$Not

[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven

- call voorzien op webservice + dto

- config item en dummy implementatie

!!! nog geen implementatie !!!

    • -0
    • +14
    ./StartVerpakkingBerekening.cls.xml
  1. … 4 more files in changeset.
[UST2039] Vh4_SVC: Sync web naar caché

- Synchroniseer klasse en method implementatie naar analogie van bestaande winkelkar methods....

    • -0
    • +15
    ./Synchroniseer.cls.xml
  1. … 9 more files in changeset.
Deze code is overgenomen uit de originele dummy methods en verplaatst naar deze klasse. Altijd mogelijk om later andere data weer te geven als dat nodig blijkt te zijn.

Deze code is overgenomen uit de originele dummy methods en verplaatst naar deze klasse. Altijd mogelijk om later andere data weer te geven als dat nodig blijkt te zijn.

idem

idem

Sinds wanneer gaan we dims vermijden ? Ze kunnen in de intermediate language misschien geen nut hebben, maar naar leesbaarheid van de code en code completion wel.

Sinds wanneer gaan we dims vermijden ? Ze kunnen in de intermediate language misschien geen nut hebben, maar naar leesbaarheid van de code en code completion wel.

  • More
  • CR-10
  • finished reviewing
Overbodige lijn

Overbodige lijn