Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Niet alle commits van ICT242 zijn opgenomen : 'k weet het - kzou de SVN repo hier ook moeten aan toevoegen ofwel die code in de common steken ben er nog niet 100% wat hier mee te doen. Geen testen ...

Niet alle commits van ICT242 zijn opgenomen : 'k weet het - kzou de SVN repo hier ook moeten aan toevoegen ofwel die code in de common steken ben er nog niet 100% wat hier mee te doen.
Geen testen ==> needs resolution -> De testen zitten dus in de svn-repo

  • More
  • CR-188
  • finished reviewing
Niet alle commits van ICT242 zijn opgenomen in deze review *Geen testen ==> needs resolution http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif *Ik zie g...
  • Niet alle commits van ICT242 zijn opgenomen in deze review
  • Geen testen ==> needs resolution
  • Ik zie geen inhoudelijke wijzigingen, enkel code generatie aanpassingen (bv. package name, cdata)
[ICT242] - Synchronizer terug werkend krijgen
[ICT242] - Synchronizer terug werkend krijgen
[ICT242] - Synchronizer terug werkend krijgen

- Export formaat gewijzigd -> Sync blijft maar melden dat deze moeten gesynced worden

  1. … 4 more files in changeset.
Was zo besproken met [~JoC]

Was zo besproken met [~JoC]

Niet dat ik weet.

Niet dat ik weet.

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

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

  • More
  • CR-18
  • finished reviewing
Kleinere methods met minder lijnen.

Kleinere methods met minder lijnen.

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

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

Method met veel lijnen opdelen in kleinere methods.

Method met veel lijnen opdelen in kleinere methods.

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

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

Naam mag voluit : ...Repository.cls

Naam mag voluit : ...Repository.cls

writes nog nodig ?

writes nog nodig ?

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

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

Duplicate code testdata mag gemeenschappelijk worden voor elke test.

Duplicate code testdata mag gemeenschappelijk worden voor elke test.

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

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

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

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

Testnaam / Methodnaam mag meer informatie bevatten.

Testnaam / Methodnaam mag meer informatie bevatten.

Testnaam mag meer informatie bevatten zodat het doel duidelijker is.

Testnaam mag meer informatie bevatten zodat het doel duidelijker is.

testnaam mag meer informatie bevatten.

testnaam mag meer informatie bevatten.

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

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

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

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

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

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

$$$Elkewaarde vervangen door de te verwachten waarde.

$$$Elkewaarde vervangen door de te verwachten waarde.

$$$Elkewaarde vervangen door de te verwachten waarde.

$$$Elkewaarde vervangen door de te verwachten waarde.