This is a list of all comments for CR-18.

Review Summary: No summary


----------------------------------------
File: APPS/AKP/AankoopControle/BLUM/impl/VrachtwagenBestanden.cls.xml
Revision Comment by Kurt Callebaut on 10 January 2017, 14:19 
http://subversion02:8060/cru/CR-18#c68

  Eens navragen waar persistent klassen komen te staan. bv onder
  /Data/

  Reply by Jo Willems on 10 January 2017, 17:16
  > Was zo besproken met [~JoC]

----------------------------------------
File: APPS/AKP/AankoopControle/BLUM/impl/VrachtwagenBestandenRepo.cls.xml
Revision Comment by Kurt Callebaut on 10 January 2017, 14:17 
http://subversion02:8060/cru/CR-18#c67

  Naam mag voluit : ...Repository.cls

----------------------------------------
File: vhUnitTest/APPS/AKP/AankoopControle/BLUM/impl/DespatchBerichtOntvangerV2/Ontvang/Test.cls.xml
Revision Comment by Kurt Callebaut on 10 January 2017, 13:39 
http://subversion02:8060/cru/CR-18#c64

  Duplicate code testdata mag gemeenschappelijk worden voor elke
  test.

Revision Comment by Kurt Callebaut on 10 January 2017, 13:26 
http://subversion02:8060/cru/CR-18#c62

  XMLCorrellaterMock wordt niet geverifieerd. Deze mag Dummy
  worden of Stub.

Revision Comment by Kurt Callebaut on 10 January 2017, 13:35 
http://subversion02:8060/cru/CR-18#c63

  Casing... #dim en ...RepoMock.

Revision Comment by Kurt Callebaut on 10 January 2017, 13:48 
http://subversion02:8060/cru/CR-18#c65

  Graag uitleg over deze test. Vanwaar één uur ? Kunnen er
  moeilijkheden zijn bij het verwerken in dit geval ?

  Reply by Jo Willems on 10 January 2017, 17:00
  > Een Vrachtwagen kan uit meerdere  despatch berichten bestaan.
  > na een bepaalde duur zijn zijn we zeker dat de bestanden van
  > een transport allemaal binnen zijn, en kunnen we de
  > vrachtwagen verwerken

----------------------------------------
File: APPS/AKP/AankoopControle/BLUM/impl/VrachtwagenVerwerker.cls.xml
Revision Comment by Kurt Callebaut on 10 January 2017, 14:14 
http://subversion02:8060/cru/CR-18#c66

  writes nog nodig ?

----------------------------------------
File: vhUnitTest/APPS/AKP/AankoopControle/BLUM/impl/VrachtwagenVerwerker.cls.xml
Revision Comment by Kurt Callebaut on 10 January 2017, 09:08 
http://subversion02:8060/cru/CR-18#c56

  Dubbele code vermijden zoals het aanmaken dummybestanden : kan
  verplaatst worden naar een method MaakDummyBestanden()...

Revision Comment by Kurt Callebaut on 10 January 2017, 09:05 
http://subversion02:8060/cru/CR-18#c54

  $$$Elkewaarde vervangen door de te verwachten waarde.

Revision Comment by Kurt Callebaut on 10 January 2017, 09:05 
http://subversion02:8060/cru/CR-18#c55

  $$$Elkewaarde vervangen door de te verwachten waarde.

----------------------------------------
File: APPS/AKP/AankoopControle/BLUM/impl/ToeleveringInfoVerzamelaarV2.cls.xml
Revision Comment by Kurt Callebaut on 10 January 2017, 14:26 
http://subversion02:8060/cru/CR-18#c69

  Method met veel lijnen opdelen in kleinere methods.

Revision Comment by Kurt Callebaut on 10 January 2017, 14:28 
http://subversion02:8060/cru/CR-18#c70

  Bestaat er een BlumProductRepository die %ExistsId en %OpenId
  implementeert ?

  Reply by Jo Willems on 10 January 2017, 17:14
  > Niet dat ik weet.

----------------------------------------
File: vhUnitTest/APPS/AKP/AankoopControle/BLUM/impl/DespatchNaarCSBSOConverteerderV2/Converteer/TestMultiDESADV.cls.xml
Revision Comment by Kurt Callebaut on 10 January 2017, 13:20 
http://subversion02:8060/cru/CR-18#c61

  Testnaam / Methodnaam mag meer informatie bevatten.

----------------------------------------
File: vhUnitTest/APPS/AKP/AankoopControle/BLUM/impl/DespatchNaarCSBSOConverteerderV2/Converteer/TestMultiPallet.cls.xml
Revision Comment by Kurt Callebaut on 10 January 2017, 13:19 
http://subversion02:8060/cru/CR-18#c60

  Testnaam mag meer informatie bevatten zodat het doel
  duidelijker is.

----------------------------------------
File: vhUnitTest/APPS/AKP/AankoopControle/BLUM/impl/DespatchNaarCSBSOConverteerderV2/Converteer/TestMultiPallet5Plus.cls.xml
Revision Comment by Kurt Callebaut on 10 January 2017, 13:17 
http://subversion02:8060/cru/CR-18#c59

  testnaam mag meer informatie bevatten.

----------------------------------------
File: APPS/AKP/AankoopControle/BLUM/impl/INVOIC02VerwerkerV2.cls.xml
Revision Comment by Kurt Callebaut on 10 January 2017, 15:04 
http://subversion02:8060/cru/CR-18#c71

  Kleinere methods met minder lijnen.

----------------------------------------
File: vhUnitTest/APPS/AKP/AankoopControle/BLUM/impl/INVOIC02VerwerkerV2/Test.cls.xml
Revision Comment by Kurt Callebaut on 10 January 2017, 11:43 
http://subversion02:8060/cru/CR-18#c57

  Blumfacturen enreceptieservice naamgevingen mogen Stub of Dummy
  worden ipv Mock aangezien er geen verificatie is.

Revision Comment by Kurt Callebaut on 10 January 2017, 13:12 
http://subversion02:8060/cru/CR-18#c58

  Niet geverifieerde mocks mogen een andere benaming hebben..
  stub of dummy.


---

ID: CR-18 http://subversion02:8060/cru/CR-18

Title: [UST2031] Blum EDI Vrachtwagen als niveau boven DESADV bericht.
Statement of Objectives:


State: Closed

Summary: 

Author: Jo Willems
Moderator: Jo Willems
Reviewers: (0 active, 1 completed*)
     Kurt Callebaut   (*)