MaakDnaCodesVoorToeleveringLijn

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
klopt, maar ondertussen bestaat deze klasse al niet meer http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

klopt, maar ondertussen bestaat deze klasse al niet meer

Testje toevoegen die aantoont dat een DnaCode ouder dan 30 dagen, niks oplevert. Het is immers belangrijk dat er geen items van vroeger worden opgepikt, zo heb ik t toch begrepen.

Testje toevoegen die aantoont dat een DnaCode ouder dan 30 dagen, niks oplevert.
Het is immers belangrijk dat er geen items van vroeger worden opgepikt, zo heb ik t toch begrepen.

[ICT-1049] PA-UB-062: Parentcodes aanmaken indien nodig (ompak)

- Oude method verwijderd (incl testen)

- ParentDnaCodeBepaling toegevoegd (en testje)

  1. … 4 more files in changeset.
[ICT-1250] PA-UB-086: weerkerende nrs voor pallet/toelevering/...
[ICT-1250] PA-UB-086: weerkerende nrs voor pallet/toelevering/...
[ICT-1250] PA-UB-086: weerkerende nrs voor pallet/toelevering/...

- Aanmaaktijdstip invullen voor nieuwe DnaCodes

  1. … 1 more file in changeset.
[ICT-837] PA-UB-042 OrderId op DnaCode bewaren
[ICT-837] PA-UB-042 OrderId op DnaCode bewaren
[ICT-837] PA-UB-041 OrderId op DnaCode bewaren

- is gebeurd

  1. … 3 more files in changeset.
Omdat deze testklasse voorral het Service/Repo gedrag test. De DnaCodeRegistratie is een noodzakelijk kwaad dat buiten de scope van de test valt

Omdat deze testklasse voorral het Service/Repo gedrag test. De DnaCodeRegistratie is een noodzakelijk kwaad dat buiten de scope van de test valt

Heb code met try-catch uitgebreid zodat ie zeker op de lock - terechtkomt. In principe is dit de enige plek waar met die lock gewerkt wordt, dus zou er geen probleem mogen zijn

Heb code met try-catch uitgebreid zodat ie zeker op de lock - terechtkomt. In principe is dit de enige plek waar met die lock gewerkt wordt, dus zou er geen probleem mogen zijn

Waarom niet "VerwachtMethodCall" en Verifieer() ? mogelijk wel met Verwacht... ExactAantalKeer(0) en dan Idem bij de andere methods.

Waarom niet "VerwachtMethodCall" en Verifieer() ? mogelijk wel met Verwacht... ExactAantalKeer(0)

en dan Idem bij de andere methods.

Ik zou deze blok (lijn 19-23 of zoiets) in een private method zetten, met een naam à la "GeefVolgendeGeldige/Bruikbare/ZonderRareTekens/.../DnaDeelOfNummerOfzoiets" #BestNameEver http://subversion0...

Ik zou deze blok (lijn 19-23 of zoiets) in een private method zetten, met een naam à la "GeefVolgendeGeldige/Bruikbare/ZonderRareTekens/.../DnaDeelOfNummerOfzoiets"
#BestNameEver
Als deze methodnaam dan voldoende sprekend is, dan wordt deze code ineens een pak leesbaarder en begrijpbaarder.

Geen timeout gedefinieerd? --> oneindig lang wachten op lock dus?

Geen timeout gedefinieerd? --> oneindig lang wachten op lock dus?

[UST4219] PA-UB-028 Echte DnaCodes genereren die kunnen ingescand worden
[UST4219] PA-UB-028 Echte DnaCodes genereren die kunnen ingescand worden
[UST4219] PA-UB-028 Echte DnaCodes genereren die kunnen ingescand worden

- Testdoubles gebruiken om geen echte codes te laten genereren

  1. … 1 more file in changeset.
Kwestie van werkwijze, persoonlijke keuze, maar ik zou de .DnaCode en de .ParentDnaCode ook al hier in de test mee opnemen (ook als is de waarde momenteel nog fixed). Als dan later de implementatie...

Kwestie van werkwijze, persoonlijke keuze, maar ik zou de .DnaCode en de .ParentDnaCode ook al hier in de test mee opnemen (ook als is de waarde momenteel nog fixed).
Als dan later de implementatie wijzigt, dan zal ook meteen de test falen.

Uiteraard snap ik dat dit "short-term work in progress" is, dus geen echte zorgen. Ik wou gewoon mijn inzichten effe delen

Ik zou toch liever de GeefToelevering() éénmalig oproepen en in variabele steken, boven de while-loop. Dit enerzijds qua efficientie (om te mierenneuken), maar anderszijds : indien er toch iets fou...

Ik zou toch liever de GeefToelevering() éénmalig oproepen en in variabele steken, boven de while-loop.
Dit enerzijds qua efficientie (om te mierenneuken), maar anderszijds : indien er toch iets foutloopt, dan zal het ook duidelijker zijn.

[UST4224] PA-UB-030: ProductType API uitbreiden om meer prefixen te kunnen teruggeven

- juiste mock-klasse gebruiken in testjes

  1. … 2 more files in changeset.
[UST4212] PA-UB-022: DnaCode tabel vullen bij nieuwe toeleveringlijn
[UST4212] PA-UB-022: DnaCode tabel vullen bij nieuwe toeleveringlijn
[UST4212] PA-UB-022: DnaCode tabel vullen bij nieuwe toeleveringlijn

- Nieuwe service die toelaat om DnaCode-lijnen te genereren

  1. … 5 more files in changeset.