Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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
Reverse merged revision(s) 37918-37917 from trunk2010: (WIJZIGINGEN ONGEDAAN GEMAAKT ! WEGWERPPALLET ALTIJD OP TUSSENETIKET ALS VERZENDWIJZE DHL IS, DUS LOS VAN UNISHOP VERZENDINGMOGELIJKHEDEN EN OPTIBOX PALLETBEREKENINGEN)

[UST1931] Hein: V1 LBX tussenetiket: wegwerppallet vermelden

- ColliDragerType wordt bepaald en opgeslagen bij zendingdetails.

(beter zou zijn als de volledige flow doorlopen wordt om dit te bepalen. Dwz bij request om palletten te bepalen reeds meegeven welke palletten / dragers er ondersteund worden voor een bepaalde klant (bv Hein). Deze waarden meegeven naar de optibox en het resultaat van de optibox bepaalt dan ColliDragerType dat via verzendingmogelijkheden, offerte,... op het etiket komt. Nog niet doorgevoerd omdat er voor andere klanten nog geen nood is voor extra info op etiket.)

........

[UST1931] Hein: V1 LBX tussenetiket: wegwerppallet vermelden

- Overbodige code is verwijderd uit test.

........

    • -84
    • +0
    ./ZendingDetails/TestColliDragerType.cls.xml
  1. … 9 more files in changeset.
[UST1931] Hein: V1 LBX tussenetiket: wegwerppallet vermelden

- Overbodige code is verwijderd uit test.

    • -10
    • +0
    ./ZendingDetails/TestColliDragerType.cls.xml
[UST1931] Hein: V1 LBX tussenetiket: wegwerppallet vermelden

- ColliDragerType wordt bepaald en opgeslagen bij zendingdetails.

(beter zou zijn als de volledige flow doorlopen wordt om dit te bepalen. Dwz bij request om palletten te bepalen reeds meegeven welke palletten / dragers er ondersteund worden voor een bepaalde klant (bv Hein). Deze waarden meegeven naar de optibox en het resultaat van de optibox bepaalt dan ColliDragerType dat via verzendingmogelijkheden, offerte,... op het etiket komt. Nog niet doorgevoerd omdat er voor andere klanten nog geen nood is voor extra info op etiket.)

    • -0
    • +94
    ./ZendingDetails/TestColliDragerType.cls.xml
  1. … 9 more files in changeset.
[UST1920] Hein: Duitsland als land toelaten bij personality VH

- Testen aangepast zodat ze niet falen na toevoegen van een nieuwe DOM.VKP.enu.Klant

[UST1920] Hein: Duitsland als land toelaten bij personality VH

- testvalue als string vervangen door ##class(...).classname(1)

[UST1920] Hein: Duitsland als land toelaten bij personality VH

- %ArrayOfDataTypes gebruikt voor klanten

[UST1920] Hein: Duitsland als land toelaten bij personality VH

- %ArrayOfDataTypes gebruikt voor klanten

[UST1920] Hein: Duitsland als land toelaten bij personality VH

- %ArrayOfDataTypes gebruikt voor klanten

[UST1920] Hein: Duitsland als land toelaten bij personality VH

- klanten array voor verkoopprijsBerekenaar

[UST1920] Hein: Duitsland als land toelaten bij personality VH

- Sessie voor LandenBepaler met klanten array

  1. … 7 more files in changeset.
[UST1920] Hein: Duitsland als land toelaten bij personality VH

- Test correctie -> Array van klanten.

[UST1920] Hein: Duitsland als land toelaten bij personality VH

- PersonalityID en KlantID in Sessie object aangepast voor VerkoopprijsBerekenaarFactory.

  1. … 4 more files in changeset.
[UST1920] Hein: Duitsland als land toelaten bij personality VH

- PersonalityID en KlantID in Sessie object aangepast voor TransportFirmaBepalerFactory.

  1. … 14 more files in changeset.
[UST1920] Hein: Duitsland als land toelaten bij personality VH

- Extra test voor alle klantID's transportfirmabepaler, na review

[UST1920] Hein: Duitsland als land toelaten bij personality VH

- Extra test voor alle personalities transportfirmabepaler, na review