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
[UST1920] Hein: Duitsland als land toelaten bij personality VH

- PersonalityID en KlantID in Sessie object aangepast voor TransportFirmaBepalerFactory.

  1. … 15 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.

    • -19
    • +6
    ./VerzendingMogelijkheidRepository.cls.xml
  1. … 12 more files in changeset.
[UST1620] TAX export: Extra info nr OV indien geen keuze kunnen aanbieden

- Mail sturen bij bevestiging van offerte en enkel indien berekeningFouten opgetreden zijn tijdens het opvragen van transportkosten.

    • -0
    • +34
    ./BerekeningFoutenRepository.cls.xml
  1. … 12 more files in changeset.
[UST1441] TAX export: ZetGekozenVerzendingMogelijkheid - unittest + implementatie

- hoort bij toevoegen manuele lijn.

- opslaan / opzoeken verzendingMogelijkheid.

-> te bespreken hoe ID1 aan te pakken ? hernoemen van Velden of is er een andere mogelijkheid.

    • -0
    • +7
    ./VerzendingMogelijkheidRepository.cls.xml
  1. … 4 more files in changeset.
[UST1441] TAX export: ZetGekozenVerzendingMogelijkheid - unittest + implementatie
    • -0
    • +7
    ./VerzendingMogelijkheidRepository.cls.xml
  1. … 7 more files in changeset.
[UST1515] TAX export: VerzendingMogelijkheidRepository - Mock / Test Bewaar functie.

Blijkbaar kan de verifieer van een mock klasse niet verifieren als het gaat om een nethod van een super klasse (base class.) die ook een mock class is. Voorlopig opgelost door de method ook in de (sub) mock klasse te steken. Maar deze kan verwijden veronderstel ik, als er wijzigingen zijn aan de implementatie klasse ??? (auto generate.)

    • -1
    • +7
    ./VerzendingMogelijkheidRepository.cls.xml
  1. … 4 more files in changeset.
[UST1515] TAX export: VerzendingMogelijkheidRepository - WSimpl.EC.ExterneVerkoop.VerzendingMogelijkheidRepository is mockable gemaakt.
    • -0
    • +20
    ./VerzendingMogelijkheidRepository.cls.xml
  1. … 6 more files in changeset.