Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UST3804] VH4: Kortingen ook voor TBX

- prefactor vóór verwerken van reviewopmerkingen deel 2

- interface APPS.EC.KlantSessieDataBepaler weggewerkt

    • -15
    • +0
    ./EC/KlantSessieDataBepaler.cls.xml
  1. … 12 more files in changeset.
Uitleg zie in e-mail (vanwege syntax-highlighting) Concreet: In de UT doe je eerst ..ZetVerwachteExceptie() en daarna roep je ..AssertBewaarCorrecteKorting() op Binnen den Assert heb je dan uitera...

Uitleg zie in e-mail (vanwege syntax-highlighting)

Concreet:
In de UT doe je eerst ..ZetVerwachteExceptie() en daarna roep je ..AssertBewaarCorrecteKorting() op
Binnen den Assert heb je dan uiteraard ook de Verifieer.
Maar volgens mij (vreemd dat t spelleke ni crasht) kan je geen “Assert” en “Verwacht” en “Verifeer” doen als je een Error aan het testen bent.

en de rest staat in mijne mail ...

Misschien niet voor nu meteen, maar eerder voor een volgende "Refactor-sessie" http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif dat we de KlantSessieData...

Misschien niet voor nu meteen, maar eerder voor een volgende "Refactor-sessie" dat we de KlantSessieDataBepalen en de RefSessieDataBepaler uitmockbaar maken, zodat in de UT geen encrypted DataOpt (en Ref) moet doorgegeven worden.
Dan staat daar gewoon in de dto.KlantSessieData ingevuld dto.Korting=0.25
Nu staat er naast de encrypted DataOpt hoeveel de waarde van de korting is.

Ik laat de keuze aan jouw, om ofwel aan te passen, ofwel te markeren, zodat een volgende persoon dat meeneemt. Deze comment in Crucible zal echter verloren gaan, dus is niet voldoende als reminder, hé.

VH4: Kortingen ook voor TBX
VH4: Kortingen ook voor TBX
[UST3086] - Seefelder-logo op klantetik. en levbon

- OorsprongDetail invullen - bevat de ketting van webwinkelkarren

  1. … 6 more files in changeset.
  • More
  • CR-535
  • finished reviewing
[ICT] - Persistente velden niet private
Ja, want dat staat ook in de standaardDecrypteerRequest

Ja, want dat staat ook in de standaardDecrypteerRequest

Het zou natuurlijk ook kunnen zijn dat ik, bij het overzetten van al die verschillende methods/properties/..., deze per ongeluk ben overgeschoten...

Het zou natuurlijk ook kunnen zijn dat ik, bij het overzetten van al die verschillende methods/properties/..., deze per ongeluk ben overgeschoten...

'k weet niet of deze voorwaarde effectief wel klopt ; lijkt het mij niet beter te checken of het feit dat er moet gecontroleerd worden voor deze hostname ? 'k vermoed dat we naar een systeem moete...

'k weet niet of deze voorwaarde effectief wel klopt ; lijkt het mij niet beter te checken of het feit dat er moet gecontroleerd worden voor deze hostname ?

'k vermoed dat we naar een systeem moeten gaan om hostnames te koppelen en personalities en ervoor zorgen dat die hostnames per systeem eventueel anders kunnen ingesteld worden.

'k zou zeggen spel wat met de waarden die ja als dummy doorgeeft en zie welke rare effecten uw testen opeens zullen hebben. Als het dan echt dummies zijn zou ik er effectief ook dummy klassen van m...

'k zou zeggen spel wat met de waarden die ja als dummy doorgeeft en zie welke rare effecten uw testen opeens zullen hebben.
Als het dan echt dummies zijn zou ik er effectief ook dummy klassen van maken zodat ze niet de aandacht trekken in uw code en de nadruk ligt op hetgeen je wil benadrukken.

Hier zou ik geen dummy string gebruiken want je controleert er effectief op

Hier zou ik geen dummy string gebruiken want je controleert er effectief op

Wel raar dat hier opeens een stub staat ?? Is het zo belangrijk dat er "example.com" wordt teruggegeven

Wel raar dat hier opeens een stub staat ?? Is het zo belangrijk dat er "example.com" wordt teruggegeven

Niet echt nodig om response te hebben -> het is perfect mogelijk om gewoon de registreer uit te voeren aangezien je het toch niet kan controleren -> kwestie van het geen aandacht te geven

Niet echt nodig om response te hebben -> het is perfect mogelijk om gewoon de registreer uit te voeren aangezien je het toch niet kan controleren -> kwestie van het geen aandacht te geven

Als deze foutmelding naar de client gaat en zo lijkt het er op dan is deze verkeerd want de client weet niet dat wij intern in ons systeem de id prefixen om onderscheid te kunnen maken in de handel...

Als deze foutmelding naar de client gaat en zo lijkt het er op dan is deze verkeerd want de client weet niet dat wij intern in ons systeem de id prefixen om onderscheid te kunnen maken in de handelklantdata.

?? GeefID verwijderen maar GeefPrefix laten we staan ?? Hoe kan je dit verklaren ?

?? GeefID verwijderen maar GeefPrefix laten we staan ?? Hoe kan je dit verklaren ?

Reponse lijkt mij toch eerder een datastructuur dus al het moet genewed worden in de constructor is er iets mis. Ga ik nu echt moeten verplichten dat alle methods 2x moeten uitgevoerd worden in de...

Reponse lijkt mij toch eerder een datastructuur dus al het moet genewed worden in de constructor is er iets mis.

Ga ik nu echt moeten verplichten dat alle methods 2x moeten uitgevoerd worden in de testen en er gecontroleerd wordt dat het resultaat niet dezelfde instantie is dan die van de vorige call

[UST2849R] Vh4 Hein: HandelAuthenticator opkuisen

- paar dummies injecteren

    • -0
    • +15
    ./EC/KlantSessieDataBepaler.cls.xml
  1. … 1 more file in changeset.
Dummies injecten

Dummies injecten

Dummy Decrypteerder meegeven - dat die test niet opeens begint te falen omdat een bepaalde decryptie niet meer geldig is

Dummy Decrypteerder meegeven - dat die test niet opeens begint te falen omdat een bepaalde decryptie niet meer geldig is

Dummy Registreerder meegeven

Dummy Registreerder meegeven

alleen maar wat code verplaatst, voor de rest niets

alleen maar wat code verplaatst, voor de rest niets

[UST2849] Vh4 Hein: HandelAuthenticator opkuisen
[UST2849] Vh4 Hein: HandelAuthenticator opkuisen
  • More
  • CR-282
  • finished reviewing
Deze klasse mocht eigenlijk blijven bestaan en afhankelijk van welke offerte hij binnen kreeg ging door delegeren naar vhisie3 of vhisie4 versie, dan had je vorige opmerkingen qua fabriekje niet aa...

Deze klasse mocht eigenlijk blijven bestaan en afhankelijk van welke offerte hij binnen kreeg ging door delegeren naar vhisie3 of vhisie4 versie, dan had je vorige opmerkingen qua fabriekje niet aan uw been

Factory method hoort eigenlijk niet thuis op een service klasse

Factory method hoort eigenlijk niet thuis op een service klasse