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
[UST2390] Vh4: Handels: Transportmogelijkheden doorgeven

- Mock nodig voor testen.

    • -0
    • +49
    ./InternationaleZending/Service.cls.xml
  1. … 2 more files in changeset.
[UST1920] Hein: Duitsland als land toelaten bij personality VH

- Sessie voor LandenBepaler met klanten array

    • -1
    • +1
    ./InternationaleZending/impl/LandenBepalerFactory.cls.xml
  1. … 7 more files in changeset.
[UST1920] Hein: Duitsland als land toelaten bij personality VH

- PersonalityID en KlantID in Sessie object aangepast voor VerkoopprijsBerekenaarFactory.

    • -0
    • +1
    ./InternationaleZending/impl/VerkoopprijsBerekenaarFactory.cls.xml
  1. … 5 more files in changeset.
[UST1920] Hein: Duitsland als land toelaten bij personality VH

- PersonalityID en KlantID in Sessie object aangepast voor TransportFirmaBepalerFactory.

    • -1
    • +1
    ./InternationaleZending/impl/ZendingDetails.cls.xml
    • -0
    • +1
    ./InternationaleZending/impl/TransportFirmaBepalerFactory.cls.xml
  1. … 14 more files in changeset.
[UST1920] Hein: Duitsland als land toelaten bij personality VH

- Beschikbarelanden refactored -> opsplitsen Van Hoecke klanten (tax export) en Hein klant en personality.

- Beschikbarelanden refactored -> Personality en klantID worden meegegeven vanuit de ExterneVerkoopService.

- Hein klant en personality geeft enkel land DE op de webshop.

- Beschikbare landen unishop losgekoppeld van Beschikbare landen in de offerteBerekenaar voor DHL.

    • -0
    • +19
    ./InternationaleZending/impl/LandenBepalerFactory.cls.xml
    • -0
    • +24
    ./InternationaleZending/LandenBepaler.cls.xml
  1. … 19 more files in changeset.
[UST1914] Klant Hein: altijd in 40€-DHL-modus draaien

- refact ivm injecten van beide klassen transportfirmabepaler en verkoopprijsberekenaar in zendingdetails.

    • -0
    • +18
    ./InternationaleZending/impl/VerkoopprijsBerekenaarFactory.cls.xml
    • -0
    • +18
    ./InternationaleZending/impl/TransportFirmaBepalerFactory.cls.xml
  1. … 6 more files in changeset.
[UST1884] [rvJCL] Hein - Transportkost offertedata (domeinobject) en verzenditems (persistent) opsplitsen aankoop en verkoop

- refactoring na review.

-> testnaam verduidelijkt.

-> Hernoemen verzendmogelijkheid.

    • -97
    • +1
    ./InternationaleZending/impl/ZendingDetails.cls.xml
  1. … 3 more files in changeset.
[UST1884] [rvJCL] Hein - Transportkost offertedata (domeinobject) en verzenditems (persistent) opsplitsen aankoop en verkoop

- refactoring na review.

-> dims toegevoegd.

-> testnaam verduidelijkt.

-> Hernoemen verzendmogelijkheid.

    • -0
    • +42
    ./InternationaleZending/impl/ZendingDetails.cls.xml
  1. … 4 more files in changeset.
[UST1902] Enkel DHL Freight meegeven voor Hein. (Geen UPS)

- Voor Hein wordt enkel DHL berekend en weergegeven in zendingdetails.

- Voor Vanhoecke worden beide UPS en DHL berekend en weergegeven in zendingdetails.

    • -0
    • +18
    ./InternationaleZending/TransportFirmaBepaler.cls.xml
  1. … 16 more files in changeset.
[UST1853] [rvJCL] PvA Hein transportkost aanrekenen

- De verkoopprijs wordt bepaald vor hein met vaste prijs van 40 euro. Enkel voor DHL Freight.

- De verkoopprijs voor alle andere personalities wordt overgenomen van de berekende aankoopprijs uit de offertedata / items.En dus ook alle andere personalities.

    • -0
    • +19
    ./InternationaleZending/VerkoopprijsBerekenaar.cls.xml
  1. … 29 more files in changeset.
[UST1884] [rvJCL] Hein - Transportkost offertedata (domeinobject) en verzenditems (persistent) opsplitsen aankoop en verkoop

- rename methods,.

    • -0
    • +21
    ./InternationaleZending/impl/ZendingDetails.cls.xml
  1. … 2 more files in changeset.
!!! conflict met revision voor BLUM vhTest/Mock/apps/ediexport/aankooporderresponse/impl/BLUM/Service.cls.xml -> is renamed !!!!

[UST1884] [rvJCL] Hein - Transportkost offertedata (domeinobject) en verzenditems (persistent) opsplitsen aankoop en verkoop

- Verplaatsen van de Bewaar Verzendmogelijkheid naar zendingdetails klasse. In kader van toeveogen aankoopprijs en verkoopprijs.

    • -0
    • +21
    ./InternationaleZending/impl/ZendingDetails.cls.xml
  1. … 12 more files in changeset.
[UST1533] TAX export: vertalingen caché en intracto implementeren

- Taal wegschrijven in runtimecontext.

- Taal beschikbaar maken voor Exterenverkoopsrvice.

- Taal doorgeven tot in verzendingdetails voor vertalingen van de tranportmogelijkheden en carbonneutral opties.

    • -0
    • +12
    ./InternationaleZending/impl/ZendingDetails.cls.xml
  1. … 8 more files in changeset.
[UST1508] TAX export: dummy ExterneVerkoopService voor Intracto

- Hernoemen webmethod en resultaat

    • -1
    • +1
    ./InternationaleZending/impl/ZendingDetails.cls.xml
  1. … 10 more files in changeset.
[UST1508] TAX export: dummy ExterneVerkoopService voor Intracto - Refactor rename

WS.EC.Unishop.InternationaleZending.dto.Details -> WS.EC.ExterneVerkoop.MogelijkeVerzendingen

WS.EC.Unishop.InternationaleZending.dto.Detail -> WS.EC.ExterneVerkoop.Verzending

WS.EC.Unishop.InternationaleZending.dto.Optie -> WS.EC.ExterneVerkoop.Optie

WS.EC.Unishop.InternationaleZending.dto.SelectieOptie -> WS.EC.ExterneVerkoop.SelectieOptie

    • -1
    • +1
    ./InternationaleZending/impl/ZendingDetails.cls.xml
  1. … 16 more files in changeset.