KlantSessieDataBepaler.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Ik zou hier precies ook nog een test schrijven met waarde true voor IsSalesMedewerker. Kwestie van zeker te zijn dat de waarde niet verloren gaat na encrypt/decrypt.

Ik zou hier precies ook nog een test schrijven met waarde true voor IsSalesMedewerker.
Kwestie van zeker te zijn dat de waarde niet verloren gaat na encrypt/decrypt.

[ICT-5110] DATAOPT uitbreiden met extra parameter voor Häfele
[ICT-5110] DATAOPT uitbreiden met extra parameter voor Häfele
[ICT-5110] DATAOPT uitbreiden met extra parameter voor Häfele

- IsSalesMedewerken is nu een Boolean ipv een String

  1. … 5 more files in changeset.
[ICT-5110] DATAOPT uitbreiden met extra parameter voor Häfele

- KlantSessieData uitbreiden met parameter IsSalesMedewerker

- Uitbreiding KlantSessieDataBepaler om deze parameter te gebruiken in geval van een haefele klant

  1. … 1 more file in changeset.
[ICT-1564] Alpnach korting van 37.7 naar 25

- Voilà

  1. … 6 more files in changeset.
[ICT-926] Alpnachkorting aanpassen 36 -> 37.7
  1. … 2 more files in changeset.
[UW780] VH4: Alpnach SFS : Korting 35.7 -> 36.0 %
[UW780] VH4: Alpnach SFS : Korting 35.7 -> 36.0 %
[UW780] VH4: Alpnach SFS : Korting 35.7 -> 36.0 %

- Deploy-klasse

- Waarde aangepast + UT

  1. … 2 more files in changeset.
[UST4145] Vervolg van UST4131 (Alpnach nummer aanpassen)
[UST4145] Vervolg van UST4131 (Alpnach nummer aanpassen)
[UST4145] Vervolg van UST4131 (Alpnach nummer aanpassen)

- alles ineens verwijderd

  1. … 15 more files in changeset.
[ICT] Alpnach korting opnieuw op 35.7 zetten
  1. … 1 more file in changeset.
ja, maar code vliegt er over twee weken toch weer uit

ja, maar code vliegt er over twee weken toch weer uit

nope, code vliegt er over twee weken toch weer uit

nope, code vliegt er over twee weken toch weer uit

Eén instantie maken van legacypartij lijkt me voldoende.

Eén instantie maken van legacypartij lijkt me voldoende.

Hier ook 2 dezelfde testen met enkel alpnach en alpnachold als verschil ? vb één assert maken voor beide testen en klantid meegeven.

Hier ook 2 dezelfde testen met enkel alpnach en alpnachold als verschil ? vb één assert maken voor beide testen en klantid meegeven.

Los van de wijzigingen. Deze method Arrange() zegt niet veel, best hernoemen naar een beter naam.

Los van de wijzigingen. Deze method Arrange() zegt niet veel, best hernoemen naar een beter naam.

"hoe dan ook" lijkt me wat vaag. Bij welke actie gebeurd dit. Ik zou eerder opteren voor : Geeft steeds 35 weer bij het opvragen van klantsessiedata ? of overschrijft korting bij het zetten of opvr...

"hoe dan ook" lijkt me wat vaag. Bij welke actie gebeurd dit. Ik zou eerder opteren voor : Geeft steeds 35 weer bij het opvragen van klantsessiedata ? of overschrijft korting bij het zetten of opvragen van klantsessiedata.

Duplicate testcode op klantid na ?

Duplicate testcode op klantid na ?

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Moeten we ook testen voorzien met nieuwe Alpnach nummer ? (het oude nummer gaat niet meer gebruikt worden en dan zijn deze testen ook overbodig.)

Dubbele test enkel met verschil alpnachoud(). zou ik samenvoegen om duplicate code te vermijden en klantid meegeven als parameter. (result meegeven is in dit geval zelfs niet nodig bij AssertBevest...

Dubbele test enkel met verschil alpnachoud(). zou ik samenvoegen om duplicate code te vermijden en klantid meegeven als parameter. (result meegeven is in dit geval zelfs niet nodig bij AssertBevestigingschroevenVoorKlantID().

Wil dat zeggen dat de hardcoded value 35 in andere klassen zoals APPS.EC.impl.KlantSessieDataBepaler vervangen moet worden door deze ingestelde waarde ?

Wil dat zeggen dat de hardcoded value 35 in andere klassen zoals APPS.EC.impl.KlantSessieDataBepaler vervangen moet worden door deze ingestelde waarde ?