IngegevenKenmerkenSifonladeHelper.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-2359] TBX BOMBOL builders porten naar .NET

- Fallback voorzien voor oude tbx lades die niet volledig ingevuld DOM kenmerken hebben

    • -1
    • +1
    ./IngegevenKenmerkenSifonladeHelper.cls.xml
  1. … 2 more files in changeset.
[ICT-1828] Ingegeven kenmerken aanmaken voor MVX

- Implementatie MVX IngegevenKenmerkenTypeAPI.IsLadeMetUitsparing.Test

    • -0
    • +6
    ./IngegevenKenmerkenSifonladeHelper.cls.xml
  1. … 1 more file in changeset.
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)

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

/IngegevenKenmerkenSifonladeHelper IsSifon controle aanpassen naar SifonType3

/TBXKenmerken sifon kenmerken aanpassen naar SifonType3

    • -1
    • +1
    ./IngegevenKenmerkenSifonladeHelper.cls.xml
  1. … 1 more file in changeset.
[UST3901] TAX-SY: Method IsSifonlade toepassen op TAX (DOM+Dto)
[UST3901] TAX-SY: Method IsSifonlade toepassen op TAX (DOM+Dto)
[UST3899] TAX-SY: Kenmerk Toepassing toevoegen
[UST3899] TAX-SY: Kenmerk Toepassing toevoegen
[UST3899] TAX-SY: Method IsSifonlade toepassen op TAX (DOM+Dto)

- IsSifonlade() voor TAX implementatie voor DOM-kenmerken

    • -0
    • +6
    ./IngegevenKenmerkenSifonladeHelper.cls.xml