WebserverKlantenPerPersonalityTemplate.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: Haefele DE toevoegen

    • -0
    • +10
    ./WebserverKlantenPerPersonalityTemplate.cls.xml
  1. … 1 more file 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)
Moet blijkbaar ook niet wegens deprecated.

Moet blijkbaar ook niet wegens deprecated.

op de zelfde plaats als al de andere mp's http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif (nergens dus alleen VH wordt getest.)

op de zelfde plaats als al de andere mp's (nergens dus alleen VH wordt getest.)

Logica is gewoon omgedraaid. nieuwe personalities zullen geen code wijzigen meer nodig hebben.

Logica is gewoon omgedraaid. nieuwe personalities zullen geen code wijzigen meer nodig hebben.

JoC was het hiet niet mee eens. als je de enum gebruikt zie je de impact niet als deze verandert

JoC was het hiet niet mee eens. als je de enum gebruikt zie je de impact niet als deze verandert

De ontvanger lijkt mij een goede kandidaat om toe te voegen aan de klantinstellingenService

De ontvanger lijkt mij een goede kandidaat om toe te voegen aan de klantinstellingenService

Opkuis van deze klasse is wenselijk.

Opkuis van deze klasse is wenselijk.

Probleempje met de TAB toets? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif

Probleempje met de TAB toets?

Is dit in afwachting van een echte Noorse vertaling? Of moet op een ander niveau een fallback voorzien worden naar het Engels?

Is dit in afwachting van een echte Noorse vertaling? Of moet op een ander niveau een fallback voorzien worden naar het Engels?

Waarom zijn enkele personalities uitgeschakeld? --> inschakelen of wegsmijten (boyscout)

Waarom zijn enkele personalities uitgeschakeld? --> inschakelen of wegsmijten (boyscout)

Is deze test overbodig?

Is deze test overbodig?

Copy/paste code: gemeenschappelijke code groeperen en parametriseren

Copy/paste code: gemeenschappelijke code groeperen en parametriseren

Leesbaarheid verhogen door gebruik te maken van de personality enumeratie.

Leesbaarheid verhogen door gebruik te maken van de personality enumeratie.

Waar wordt dit getest?

Waar wordt dit getest?

Unishop is uitgefaseerd --> renamen

Unishop is uitgefaseerd --> renamen

BeslagTechnikk ontbreekt

BeslagTechnikk ontbreekt