WinkelkarService.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1527] [rvTVE] EC: MP: Herbestel-app Häfele: flag schade optioneel toevoegen aan BesteldeOfferteMelder (HTTP POST)

Integratie testen: IngaveTotBestelling: aanpassen voor Haefele DE zodat de tag DAMAGE_BY_TRANSPORTATION_OR_ASSEMBLY ook wordt weergegeven

  1. … 2 more files in changeset.
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)
[ICT337] Applicatie goed krijgen op de runtime context zodat deze in de logging gebruikt kan worden
  • More
  • CR-532
  • finished reviewing
#dimmekes nakijken zie GeefMaatwerkLijnen()

#dimmekes nakijken zie GeefMaatwerkLijnen()

Ofwel test je dat MaatwerkLijn1 en MaatwerkLijn2 in de lijst zitten als je dat niet doet is het totaal niet noodzakelijk om daar variabelen aan te verspillen

Ofwel test je dat MaatwerkLijn1 en MaatwerkLijn2 in de lijst zitten als je dat niet doet is het totaal niet noodzakelijk om daar variabelen aan te verspillen

Lijkt niet echt te kloppen As WS.vhisie4.Winkelkar.dto.MaatwerkLijn ; Misschien beter om AssertEquals(WinkelkarUtils.GeefMaatwerkLijnen(Winkelkar).Count(),0) te schrijven dan kan de dim niet verke...

Lijkt niet echt te kloppen As WS.vhisie4.Winkelkar.dto.MaatwerkLijn ;

Misschien beter om AssertEquals(WinkelkarUtils.GeefMaatwerkLijnen(Winkelkar).Count(),0) te schrijven dan kan de dim niet verkeerd zijn

[UST2039] Vh4_SVC: Sync web naar caché
[UST2039] Vh4_SVC: Sync web naar caché
  • More
  • CR-88
  • finished reviewing
Vraagje : Vanwaar de relatie dat 1 leverdatum niet asap is ? Kan 1 leverdatum geen "onmiddellijk leverbaar" betekenen? Of heeft asap een andere betekenis

Vraagje : Vanwaar de relatie dat 1 leverdatum niet asap is ? Kan 1 leverdatum geen "onmiddellijk leverbaar" betekenen? Of heeft asap een andere betekenis

  • More
  • CR-88
  • started review
[UST2064] Vh4_SVC: BewaarMaatwerkLijn
[UST2064] Vh4_SVC: BewaarMaatwerkLijn
[UST2064] Vh4_SVC: BewaarMaatwerkLijn

- GroepNaam bepalen op hoger niveau (WS.EC ipv APPC.EC -> ook nodig voor de WijzigMaatwerklijn

  • More
  • CR-46
  • finished reviewing
[UST2039] Vh4_SVC: Sync web naar caché

- refact voegtoemaatwerk -LijnID weergeven.

[UST2064] Vh4_SVC: BewaarMaatwerkLijn

- Blijkbaar had den dezen hier een quickfix gedaan in den deploy en zijn er nu merge conflicten..

[UST2064] Vh4_SVC: BewaarMaatwerkLijn

- GroepNaamBepaler werd niet geinjecteerd.

  1. … 2 more files in changeset.