CacheAdminA_trunk2010/vhUnitTest/APPS/OrderUitvoering

SamenDoorsturenSFSControle

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 19 Jun 15
Commits this week: 0
Total Lines of Code (LoC): 21
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 05 Mar 2020
8:52 am

Laura Landuyt

Monday 27 Jan 2020
11:09 am

Kurt Callebaut

Thursday 16 Jan 2020
9:57 am

Wim Vermeulen

Monday 13 Jan 2020
9:19 am

Wim Vermeulen

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

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

Friday 10 Jan 2020
11:00 am

Wim Vermeulen

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.

10:38 am

Wim Vermeulen

"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.

Friday 20 Dec 2019
12:21 pm

Wim Vermeulen

Code voor ToeleveringsplitserMock niet in elke method apart toevoegen.

Code voor ToeleveringsplitserMock niet in elke method apart toevoegen.

11:48 am

Wim Vermeulen

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

11:33 am

Wim Vermeulen

MaatwerkProductStub mag nu terug weg hé.

MaatwerkProductStub mag nu terug weg hé.

11:20 am

Wim Vermeulen

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.

11:11 am

Wim Vermeulen

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é?

10:38 am

Wim Vermeulen

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

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

Thursday 19 Dec 2019
1:18 pm

Laura Landuyt

9:54 am

Wim Vermeulen

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?

9:45 am

Wim Vermeulen

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)

8:16 am

Kurt Callebaut

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.

8:05 am

Kurt Callebaut

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.

8:02 am

Kurt Callebaut

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)

7:46 am

Kurt Callebaut

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.

Wednesday 18 Dec 2019
12:19 pm

Laura Landuyt

[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