Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Indien kritisch review, dan ontbreken nog enkele UT's. Maar gezien de korte tijd dat deze code nog zal bestaan, is het huidige niveau van code-coverage voldoende. Review O.K.

Indien kritisch review, dan ontbreken nog enkele UT's. Maar gezien de korte tijd dat deze code nog zal bestaan, is het huidige niveau van code-coverage voldoende.
Review O.K.

[ICT-2539] melding bij doorsturen picking orderlijn op leverdatum later dan vandaag
[ICT-2539] melding bij doorsturen picking orderlijn op leverdatum later dan vandaag
[ICT-2539] melding bij doorsturen picking orderlijn op leverdatum later dan vandaag

- opsplitsen bezwaren op orderniveau en orderlijn niveau

  1. … 2 more files in changeset.
[ICT-2539] melding bij doorsturen picking orderlijn op leverdatum later dan vandaag

- De nieuwe bepaler oproepen, en test op gedrag toegevoegd aan bestaande test. method hernoemd.

  1. … 1 more file in changeset.
[ICT-2539] melding bij doorsturen picking orderlijn op leverdatum later dan vandaag

- wat testcode uit vhUnitTest.APPS.OrderUitvoering.TestBezwaren op de juiste plaats gezet.

  1. ./Bezwaren/WachtOpAnderOrderVoorDoorsturen
Bij Probox en Pallet zet je dan achteraan : ... .DanReturn($$$True).ExactAantalKeer(0)

Bij Probox en Pallet zet je dan achteraan : ... .DanReturn($$$True).ExactAantalKeer(0)

Ik heb nog een klein verzoek voor verbetering : De "IndienMethodCall()" vervangen door "VerwachtMethodCall() + .Verifieer() Bij Probox en Pallet zet je dan achteraan : ... .DanReturn($$$True).Exac...

Ik heb nog een klein verzoek voor verbetering :

De "IndienMethodCall()" vervangen door "VerwachtMethodCall() + .Verifieer()
Bij Probox en Pallet zet je dan achteraan : ... .DanReturn($$$True).ExactAantalKeer(0)
Op deze manier krijg je ook meteen duidelijke feedback indien de test faalt.

"TBXSpoelbak" is ondertussen gewijzigd naar TBXSifon ( in de methodnaam ook http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif ) en hieronder bij "VHoss" oo...

"TBXSpoelbak" is ondertussen gewijzigd naar TBXSifon ( in de methodnaam ook )
en hieronder bij "VHoss" ook aub.

Code voor ToeleveringsplitserMock niet in elke method apart toevoegen.

Code voor ToeleveringsplitserMock niet in elke method apart toevoegen.

Eigenlijk moet je hier " ... HeeftProductSpecificatie").DanReturn($$$True) " zetten, want anders heeft de ToeleveringSplitser sowieso geen invloed op deze Test. En als ik dan nog een stapje verder ...

Eigenlijk moet je hier " ... HeeftProductSpecificatie").DanReturn($$$True) " zetten, want anders heeft de ToeleveringSplitser sowieso geen invloed op deze Test.
En als ik dan nog een stapje verder probeer te denken, dan kan je deze testmethod herleiden tot ca. 5 lijnen code, zoals een aantal andere tests in deze klasse.
Bij twijfel, geef maar een seintje.

FYI : het is normaal dat code in de testen soms onoverzichtelijk groot wordt (na verschillende evoluties in ontwikkeling). Dan heb je soms iemand anders nodig om zulke vereenvoudigingen aan te geven

MaatwerkProductStub mag nu terug weg hé.

MaatwerkProductStub mag nu terug weg hé.

Deze method mag terug verwijderd worden, want geen nieuwe testen meer maken voor TBX Spoelbak.

Deze method mag terug verwijderd worden, want geen nieuwe testen meer maken voor TBX Spoelbak.

Is duplicatie van de method eronder. Lijkt me overbodig, want je kan toch SifonLadeMet1Uitsparing() oproepen, hé?

Is duplicatie van de method eronder.
Lijkt me overbodig, want je kan toch SifonLadeMet1Uitsparing() oproepen, hé?

Klein detail : ik zou hier een ElseIf ... van maken

Klein detail : ik zou hier een ElseIf ... van maken

Zoals het nu staat zal de ToeleveringSplitser enkel toegepast worden bij verpakking V1 (= per stuk) Ik weet niet of er afgestemd is met NBO, wat met de andere verpakkingen?

Zoals het nu staat zal de ToeleveringSplitser enkel toegepast worden bij verpakking V1 (= per stuk)
Ik weet niet of er afgestemd is met NBO, wat met de andere verpakkingen?

De ..IsObject() is niet nodig op de IngegevenKenmerkenTypeAPI, deze zal altijd aanwezig zijn (want in constructor gedefinieerd via $$$Inject() ) Voor de ToeleveringSplitser hierboven is dat niet he...

De ..IsObject() is niet nodig op de IngegevenKenmerkenTypeAPI, deze zal altijd aanwezig zijn (want in constructor gedefinieerd via $$$Inject() )
Voor de ToeleveringSplitser hierboven is dat niet het geval, dus daar is het wel oké om te testen met $IsObject)

[ICT-878] [+KCA] HX: TBX sifonlades in aparte toelevering/batch indien aantal>10

refactoring

/vhUnitTest.APPS.OrderUitvoering.impl.UTTandemboxProduct change extend %RegisteredObject to TECH.RegisteredObject

/vhUnitTest.APPS.OrderUitvoering.SamenDoorsturenSFSControle.UTTandemboxProduct delete extend TECH.RegisteredObject want andere extend klasse erft hier reeds van over TECH.RegisteredObject

    • -1
    • +1
    ./SamenDoorsturenSFSControle/UTTandemboxProduct.cls.xml
Deze was moeilijk terug te vinden. kijk even naar TBX gedeelte in deze klasse : *APPS.Halux.common.ProductieService Geen idee of hier testen voor bestaan.

Deze was moeilijk terug te vinden. kijk even naar TBX gedeelte in deze klasse :

  • APPS.Halux.common.ProductieService


Geen idee of hier testen voor bestaan.

Omzetten naar TECH.reigistered object is ok. Hier op zich niet zo erg. Naar de toekomst toe een tip. Wij proberen refactoring en wijzigingen apart te committen. Zo is het duidelijk wat refact is en...

Omzetten naar TECH.reigistered object is ok. Hier op zich niet zo erg.
Naar de toekomst toe een tip. Wij proberen refactoring en wijzigingen apart te committen.
Zo is het duidelijk wat refact is en wat nieuwe functionaliteit is.
Kan anders verwarrend worden.

Als de superclass overerft van registered object, dan is er hier geen nood aan en mag dan terug weg. (Geeft geen fouten in de code)

Als de superclass overerft van registered object, dan is er hier geen nood aan en mag dan terug weg.
(Geeft geen fouten in de code)

Ik denk dat deze $$IsObject() moet zijn. Mijn fout. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Ik denk dat deze $$IsObject() moet zijn. Mijn fout.

[ICT-878] [+KCA] HX: TBX sifonlades in aparte toelevering/batch indien aantal>10
[ICT-878] [+KCA] HX: TBX sifonlades in aparte toelevering/batch indien aantal>10
[ICT-878] [+KCA] HX: TBX sifonlades in aparte toelevering/batch indien aantal>10

/vhUnitTest.APPS.OrderUitvoering.SamenDoorsturenControle.UTProductTypeAPI Add GeefMaatwerkProduct

vhUnitTest.APPS.OrderUitvoering.SamenDoorsturenSFSControle.UTProductTypeAPI Add GeefMaatwerkProduct

/vhUnitTest.APPS.OrderUitvoering.SamenDoorsturenControle.UTTandemboxProduct Add HeeftProductSpecificatie

/vhUnitTest.APPS.OrderUitvoering.SamenDoorsturenSFSControle.UTTandemboxProduct Add HeeftProductSpecificatie

    • -1
    • +7
    ./SamenDoorsturenControle/UTTandemboxProduct.cls.xml
    • -1
    • +8
    ./SamenDoorsturenControle/UTProductTypeAPI.cls.xml
    • -1
    • +7
    ./SamenDoorsturenSFSControle/UTTandemboxProduct.cls.xml
    • -1
    • +8
    ./SamenDoorsturenSFSControle/UTProductTypeAPI.cls.xml
[ICT-817] Productiegroep Specials PM voor meer dan 10 LBX smalle lades

- Paar rare klassen waarbij SFS zelfs niet overerft.. missen implementatie productlijnen.

- fix test

    • -0
    • +8
    ./SamenDoorsturenSFSControle/UTToelevering.cls.xml
    • -0
    • +8
    ./SamenDoorsturenControle/UTToelevering.cls.xml