Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[PRJ051-203] Orderoptie BackWidthM is leeg bij TBX spoelbak
  1. … 2 more files in changeset.
[PRJ051-148] Nieuwe orderopties voor de matten
  1. … 1 more file in changeset.
Deze lijkt me wel oké.

Deze lijkt me wel oké.

Deze toch nog eens nakijken aub. (i.e. na de commits van WimV) --> ik zou proberen met deze op "600" te zetten.

Deze toch nog eens nakijken aub. (i.e. na de commits van WimV)
--> ik zou proberen met deze op "600" te zetten.

Terug op 513 gezet + commentaar toegevoegd

Terug op 513 gezet + commentaar toegevoegd

Deze conditie blijkt overbodig, en zorgt (wellicht) voor de correcte fix voor dit kaartje. Onderstaande toevoeging mag terug weg. Reeds gecommit by WimV http://subversion02.vanhoecke.be/static/ogdo...

Deze conditie blijkt overbodig, en zorgt (wellicht) voor de correcte fix voor dit kaartje.
Onderstaande toevoeging mag terug weg.
Reeds gecommit by WimV

[ICT-2883] [rvWV] QRcodeNaarKenmerkenService faalt voor product QR-code 'LTX727' (productid: '587601') (Commit voor PeterV)

- Voorgaande correctie ongedaan gemaakt : de situatie werd verkeerd ingeschat (door onjuiste en dus zeer misleidende benamingen uit het verleden) en was de fix eigenlijk niet correct.

- "CorpusBinnenbreedte" was oorspronkelijk "InbouwBreedte", wat hetzelfde is als "InterneBreedte"

- vhTest.Utils klasse DomKenmerken terug aangepast + commentaar toegevoegd

[ICT-2883] [rvWV] QRcodeNaarKenmerkenService faalt voor product QR-code 'LTX727' (productid: '587601')

- Corpusbreedte opvullen

[ICT-2883] [rvWV] QRcodeNaarKenmerkenService faalt voor product QR-code 'LTX727' (productid: '587601')...
[ICT-2883] [rvWV] QRcodeNaarKenmerkenService faalt voor product QR-code 'LTX727' (productid: '587601')...
[ICT-2359] TBX BOMBOL builders porten naar .NET

- Toevoegen tbx objecten voor nieuwe test waarvan de testbestanden al gecommit waren

  1. … 1 more file in changeset.
[ICT-2359] TBX BOMBOL builders porten naar .NET

- Verder corrigeren/aanvullen van tbx configuraties voor de nieuwe integratietesten

  1. … 1 more file in changeset.
[ICT-2359] TBX BOMBOL builders porten naar .NET

- Ingegevenkenmerken corrigeren zodanig ze overeenkomen met legacykenmerken

  1. … 2 more files in changeset.
[ICT-2359] TBX BOMBOL builders porten naar .NET

- Toevoegen Tbx configuratie met stalen rug

  1. … 1 more file in changeset.
[ICT-2359] TBX BOMBOL builders porten naar .NET

- Nieuwe configuraties toevoegen aan testUtils voor TBX en zijn legacyKenmerken, mogelijks zijn de niet tbx legacykenmerken nog niet compleet

  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.

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

/TBXKenmerken Refactor code verwijderen generieke kenmerken voor sifonlade, gebruiken van reeds voorziene specifieke SifonLadeMet1Uitsparing kenmerken + verwijderen van tbxspoelbakkenmerken

/ProductieGroepBepalerTBX.GeefProductieGroep.Test aanpassen TbxSifonLade naar SifonLadeMet1Uitsparing

  1. … 1 more file in changeset.
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