Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Na uitgebreid intern overleg... toch maar niet http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Na uitgebreid intern overleg... toch maar niet

[ICT-348] [rvTVE] VHIP903: Voorraadsync: Uitz spacetower in AxProductValidator NIET igv SyncAll:
[ICT-348] [rvTVE] VHIP903: Voorraadsync: Uitz spacetower in AxProductValidator NIET igv SyncAll:
idem

idem

zie opmerking in de test van IsGeldig. zelfde bij de IsOngewenstProductInMigratieFilter ook in een paar testjes hieronder

zie opmerking in de test van IsGeldig.
zelfde bij de IsOngewenstProductInMigratieFilter
ook in een paar testjes hieronder

Direct de ..ProductTypeApiStub gebruiken. IndienMethodCalls kunnen overschreven worden (Verwacht... niet). De #dim kan dan weg en de Set ..ProductTypeApi hieronder ook. Deze constructie staat op no...

Direct de ..ProductTypeApiStub gebruiken. IndienMethodCalls kunnen overschreven worden (Verwacht... niet). De #dim kan dan weg en de Set ..ProductTypeApi hieronder ook.
Deze constructie staat op nog een paar andere testjes ook

[ICT-348] [rvTVE] VHIP903: Voorraadsync: Uitz spacetower in AxProductValidator NIET igv SyncAll:

- IsGeldigWithAxConfigId-Testjes grondig gerefactord

[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:

- Na review => LinkType => enum gebruikt ipv value als string

- Na review => LinkType => Bij IsGeldigWithAxConfigId n.v.t.

  1. … 2 more files in changeset.
DOM.PM.enu.LinkType (hieronder nog een paar keer) (en ook in ander testje)

DOM.PM.enu.LinkType (hieronder nog een paar keer) (en ook in ander testje)

my point exactly (zie comment). Afzonderen naar apart klaske dat getest kan worden

my point exactly (zie comment).
Afzonderen naar apart klaske dat getest kan worden

TestCases gemaakt voor beide public methods (en onderweg maken van product afgezonderd)

TestCases gemaakt voor beide public methods (en onderweg maken van product afgezonderd)

[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:

- Na review => Ook voor IsGeldigWithAxConfigId => TestCase

  1. … 1 more file in changeset.
ClassMethod!

ClassMethod!

Als ge de lijn toch verandert, kunt ge ineens de ' door $$$Not vervangen

Als ge de lijn toch verandert, kunt ge ineens de ' door $$$Not vervangen

Boyscout principe: er bestaan geen testjes van deze klasse. Misschien is het niet veel moeite om een paar dingen toe te voegen?

Boyscout principe: er bestaan geen testjes van deze klasse. Misschien is het niet veel moeite om een paar dingen toe te voegen?

Op den duur zou een mens zich beginnen afvragen of al deze StartVerwerking-code niet beter in een aparte klasse gestoken wordt. Ooit is dat begonnen als een klein methodje dat niet te veel doet (ju...

Op den duur zou een mens zich beginnen afvragen of al deze StartVerwerking-code niet beter in een aparte klasse gestoken wordt. Ooit is dat begonnen als een klein methodje dat niet te veel doet (juist wat parameters zetten enzo) en dan was het geen probleem dat het niet in een niet-testbare classmethod steekt. Met al de extra logica die er ondertussen is bijgekomen is dat eigenlijk niet goed meer.

[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:
[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden: