Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
De = $$$ True is in principe niet nodig (en wordt meestal niet gezet), maar daarover kan lang gediscussieerd worden http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticon...

De = $$$ True is in principe niet nodig (en wordt meestal niet gezet), maar daarover kan lang gediscussieerd worden Zie de lijn erboven

Nitpicking, maar als het 123 is, is het niet random. Ofwel is het DummyWinkelkarId = $$$Integer als het echt niet belangrijk is, ofwel is het random en gebruik je de randomizer, ofwel moet het echt...

Nitpicking, maar als het 123 is, is het niet random. Ofwel is het DummyWinkelkarId = $$$Integer als het echt niet belangrijk is, ofwel is het random en gebruik je de randomizer, ofwel moet het echt 123 zijn

Kan je hier ##class(vhTest.Utils.APPS.VKP.Offerte).StandaardOfferte() of één van zijn vriendjes gebruiken?

Kan je hier ##class(vhTest.Utils.APPS.VKP.Offerte).StandaardOfferte() of één van zijn vriendjes gebruiken?

Ik vroeg me af of het interessanter is om de test in twee te splitsen zodat ze elk duiudelijk hun situatie testen, waardoor het beter opvalt wat er aan de hand is (door testomschrijving, en door va...

Ik vroeg me af of het interessanter is om de test in twee te splitsen zodat ze elk duiudelijk hun situatie testen, waardoor het beter opvalt wat er aan de hand is (door testomschrijving, en door variatiepunt in test. Een hele hoop van de asserts en opzet zijn dan gemeenschappelijk en kunnen in wat private methods gemikt worden.

Soit, 't is tijd om te stoppen, zal morgen verderdoen

fyi: in plaats van een hele lijst properties te asserten, zou je ook een AssertObject ofzo kunnen gebruiken.

fyi: in plaats van een hele lijst properties te asserten, zou je ook een AssertObject ofzo kunnen gebruiken.

ook hier werken met de fake datetimeapi

ook hier werken met de fake datetimeapi

Als de variabele ECHT geen bal uitmaakt, vervang hem dan door $$$DummyString (of $$$DummyInteger, ...). Dat maakt functioneel geen bal uit, maar maakt het voor de lezer duidelijk dat het echt niets...

Als de variabele ECHT geen bal uitmaakt, vervang hem dan door $$$DummyString (of $$$DummyInteger, ...).
Dat maakt functioneel geen bal uit, maar maakt het voor de lezer duidelijk dat het echt niets uitmaakt

Gebruik voor een vaste datum de DateTimeApi (bv Do ##class(vhTest.Fake.TECH.DateTime.DateTimeAPI).Activeer()) (voorbeelden in andere testen)

Gebruik voor een vaste datum de DateTimeApi (bv Do ##class(vhTest.Fake.TECH.DateTime.DateTimeAPI).Activeer()) (voorbeelden in andere testen)

Beter om de const te vervangen door een personality-enu

Beter om de const te vervangen door een personality-enu

ICT-1527: [rvTVE] EC: MP: Herbestel-app Häfele: flag schade optioneel toevoegen aan BesteldeOfferteMelder (HTTP POST)
ICT-1527: [rvTVE] EC: MP: Herbestel-app Häfele: flag schade optioneel toevoegen aan BesteldeOfferteMelder (HTTP POST)
IsInterneGebruikerToeglaten is altijd false, we zijn vandaag al dag 65468, dus dat tweede deel is altijd false. Eigenlijk mag dat dus weg, want wordt hieronder enkel 'ge-not' en zal dus altijd true...

IsInterneGebruikerToeglaten is altijd false, we zijn vandaag al dag 65468, dus dat tweede deel is altijd false. Eigenlijk mag dat dus weg, want wordt hieronder enkel 'ge-not' en zal dus altijd true zijn daar

vergelijkbare opmerking als SFS kaartje : boyscout-principe toepassen

vergelijkbare opmerking als SFS kaartje : boyscout-principe toepassen

quit als eerste lijn zetten

quit als eerste lijn zetten

dit is voor een groot deel copy-paste van andere code, maar misschien moet je toch even de ' door $$$Not vervangen, de 0/1 door $$$False/$$$True enz casing checken kan ook geen kwaad uitgecommentar...

dit is voor een groot deel copy-paste van andere code, maar misschien moet je toch even de ' door $$$Not vervangen, de 0/1 door $$$False/$$$True enz
casing checken kan ook geen kwaad
uitgecommentarieerde lijnen verwijderen
...

maw: toepassen van het boyscout-principe

$$$True ipv 1 en je kan die quit gerust op de eerste lijn zetten

$$$True ipv 1
en je kan die quit gerust op de eerste lijn zetten

[ICT-1219] [rvTVE] [rvJWI] EC: MP: ConfirmOrder-endpoint compatibel met www01 (Integratie webshop-vrijgave...
[ICT-1219] [rvTVE] [rvJWI] EC: MP: ConfirmOrder-endpoint compatibel met www01 (Integratie webshop-vrijgave...
[ICT-1219] [rvTVE] [rvJWI] EC: MP: ConfirmOrder-endpoint compatibel met www01 (Integratie webshop-vrijgave...
[ICT-1219] [rvTVE] [rvJWI] EC: MP: ConfirmOrder-endpoint compatibel met www01 (Integratie webshop-vrijgave...
nieuwe wijziging daarnet on-the-fly door WV gereviewed

nieuwe wijziging daarnet on-the-fly door WV gereviewed

Result-object heeft dezelfde structuur, maar is wel van een andere klasse : WS.EC.Unishop.OrderConfirmation.ConfirmOrderResult Typecasten? of laten zoals het is? Wat denk je?

Result-object heeft dezelfde structuur, maar is wel van een andere klasse : WS.EC.Unishop.OrderConfirmation.ConfirmOrderResult

Typecasten? of laten zoals het is?
Wat denk je?

[UST3774] Intractomelder doen werken
[UST3774] Intractomelder doen werken
  • More
  • CR-532
  • finished reviewing