Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UW554] DoorgestuurdNaarAX in BestelStatusHistoriek van Vhisie4-"winkelkarren".
[VRB511]Tijdelijk order niet sluiten indien er geen productorderlijnen zijn.
[VRB511]Tijdelijk order niet sluiten indien er geen productorderlijnen zijn.
[VRB511]Tijdelijk order niet sluiten indien er geen productorderlijnen zijn.

- Geen productorderlijnen aanbrengen als bezwaar.

- bijkomende test.

  1. … 1 more file in changeset.
[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding
[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding
Nu komt die "1" precies uit de lucht gevallen. Men moet al moeite doen en verder zoeken, om te vinden dat die vanuit de method GeefTvwTijdelijkOrder() komt. Voorstel: ..GeefTeVerwijderenTijdelijkO...

Nu komt die "1" precies uit de lucht gevallen. Men moet al moeite doen en verder zoeken, om te vinden dat die vanuit de method GeefTvwTijdelijkOrder() komt.

Voorstel:
..GeefTeVerwijderenTijdelijkOrder(TvwOrderID)
Do TeVerwijderenOrderRepositoryMock.VerwachtMethodCall("VerwijderViaID",TvwOrderID).DanTralala

idem in de volgende methods

zie volgende comment, hieronder.

zie volgende comment, hieronder.

Deze review-opmerking mocht ik niet maken (van JoW) : klassenaam in t enkelvoud.

Deze review-opmerking mocht ik niet maken (van JoW) :
klassenaam in t enkelvoud.

Tja, ik moet het melden, hé ;-P $Inject() kan best de OrderAPI newen i.p.v. Context.Instance() ... maar als de discussie nog gesloten is, dan laat je het maar zo.

Tja, ik moet het melden, hé ;-P
$Inject() kan best de OrderAPI newen i.p.v. Context.Instance()
... maar als de discussie nog gesloten is, dan laat je het maar zo.

[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding

- refactor naar juiste package

  1. … 20 more files in changeset.
[UW497] VHIP504 Verwijderen tijdelijk order - kan soms order niet verwijderen. - foutmelding

- inbouwen repo + registreren order indien verwijderen niet lukt.

- test gedrag indien orderverwijderen niet lukt.

  1. … 1 more file in changeset.
[ICT] VHIP504

- meer detail in logging indien verwijderen tijdelijk order mislukt.

  1. … 1 more file in changeset.
[ICT349] VHIP504 Tijdelijk order sluiter order niet verwijderen indien fout in processing

- review opmerkingen

- logger geinjecteerd, bijkomende test op loggen exceptie.

  1. … 1 more file in changeset.
  • More
  • CR-946
  • finished reviewing
Do ipv do

Do ipv do

Do ipv DO (ook hieronder)

Do ipv DO (ook hieronder)

ProductApiMock ipv ProductAPI

ProductApiMock ipv ProductAPI

Kunt ook de Logger injecteren en een test met een VerwachtMethodCall("Error") indien de geïnjecteerde OrderAPI.VerwijderOrder() $$$False teruggeeft Best de ORDNr die in de string gestoken wordt, me...

Kunt ook de Logger injecteren en een test met een VerwachtMethodCall("Error") indien de geïnjecteerde OrderAPI.VerwijderOrder() $$$False teruggeeft
Best de ORDNr die in de string gestoken wordt, met single quotes omringen

returnt http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

returnt

$$$AssertEquals(,$$$True) kunt ge vervangen door $$$AssertTrue()

$$$AssertEquals(,$$$True) kunt ge vervangen door $$$AssertTrue()

$$$DummyString

$$$DummyString

[ICT349] VHIP504 Tijdelijk order sluiter order niet verwijderen indien fout in processing
[ICT349] VHIP504 Tijdelijk order sluiter order niet verwijderen indien fout in processing
[ICT349] VHIP504 Tijdelijk order sluiter order niet verwijderen indien fout in processing

- Order niet verwijderen indien generatie bericht niet gelukt.

- extra logging indien er iets fout liep met het order verwijderen

  1. … 1 more file in changeset.
  • More
  • CR-738
  • finished reviewing
[UST3138] VHIP504: ORDER2AX: Validatie bij het sluiten van een tijdelijk order
[UST3138] VHIP504: ORDER2AX: Validatie bij het sluiten van een tijdelijk order