ResponseHeader.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
heb ineens nog een test toegevoegd voor het geval de ValidateHeader zelf een exception zou raisen

heb ineens nog een test toegevoegd voor het geval de ValidateHeader zelf een exception zou raisen

Ik had ook deze twijfel, maar heb toch voor een warning gekozen omdat er geen 'harde fout' optreedt naar de gebruiker toe. Ze zullen daar wel een melding krijgen, maar niet "vast" zitten. In dat ge...

Ik had ook deze twijfel, maar heb toch voor een warning gekozen omdat er geen 'harde fout' optreedt naar de gebruiker toe. Ze zullen daar wel een melding krijgen, maar niet "vast" zitten. In dat geval zullen we wel gecontacteerd worden en in de logs gaan zoeken.

GeneriekeResponse OF SpecifiekeMethodResponse ? Kunnen we hier eens over babbelen? Ik ben persoonlijk evenveel voor als tegen! http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/ico...

GeneriekeResponse OF SpecifiekeMethodResponse ?
Kunnen we hier eens over babbelen?
Ik ben persoonlijk evenveel voor als tegen!

Assert OmschrijvingVoorLogging ook nog.

Assert OmschrijvingVoorLogging ook nog.

Ik twijfel tussen "Warning" en "Error", of zelfs ErrorMail. Er is ten slotte iets foutgelopen, dat wellicht moet opgelost worden. Dus ik vrees dat bij Warning de boodschap zal verloren gaan.

Ik twijfel tussen "Warning" en "Error", of zelfs ErrorMail.
Er is ten slotte iets foutgelopen, dat wellicht moet opgelost worden. Dus ik vrees dat bij Warning de boodschap zal verloren gaan.

Mag hier ook weg, want zit in superklasse.

Mag hier ook weg, want zit in superklasse.

[UST4204] PA-UB-021: Exceptions-handling backend
[UST4204] PA-UB-021: Exceptions-handling backend
[UST4201] PA-UB-021: Exceptions-handling backend

- Header uitgebreid

het tegenvoorbeeld is: zolang de test binnen 1 pagina past, is het niet altijd nodig om af te zonderen

het tegenvoorbeeld is: zolang de test binnen 1 pagina past, is het niet altijd nodig om af te zonderen

Stub, indien geen .Verifieer()

Stub, indien geen .Verifieer()

idem opeten

idem opeten

Mock moet stub zijn, maar ik zie dat deze 4 lijnen telkens opnieuw terugkeren, alvast in 3 van de 5 testmethods --> afzonderen in private method.

Mock moet stub zijn, maar ik zie dat deze 4 lijnen telkens opnieuw terugkeren, alvast in 3 van de 5 testmethods --> afzonderen in private method.

i.p.v. error opeten : beter als innerexception toevoegen.

i.p.v. error opeten : beter als innerexception toevoegen.

computernaam etc moet vermeld worden

computernaam etc moet vermeld worden

LoginCode en Initialen toevoegen aan de foutmelding + Computernaam (en tijdstip) ook tonen.

LoginCode en Initialen toevoegen aan de foutmelding + Computernaam (en tijdstip) ook tonen.

LoginCode en Initialen toevoegen aan de foutmelding

LoginCode en Initialen toevoegen aan de foutmelding

detail : deze is een VanHoeckeGebruikerBeheerderStub i.p.v. een Mock. --> Martijn had een leuk ezelsbruggetje om deze uit mekaar te houden --> Check MVI http://subversion02.vanhoecke.be/static/ogdo...

detail : deze is een VanHoeckeGebruikerBeheerderStub i.p.v. een Mock.
--> Martijn had een leuk ezelsbruggetje om deze uit mekaar te houden --> Check MVI

Opeet-code? Op z'n minst iets loggen? of andere actie.

Opeet-code?
Op z'n minst iets loggen? of andere actie.

Ik zou eerst de .Bewaar() uitvoeren en dan checken op volgende HasNext()+exception. Anderszijds zou de UpdateAanmelding() een method moeten zijn op de service en niet thuishoren in de Validator (ma...

Ik zou eerst de .Bewaar() uitvoeren en dan checken op volgende HasNext()+exception.
Anderszijds zou de UpdateAanmelding() een method moeten zijn op de service en niet thuishoren in de Validator (maar dan vervalt wellicht mijn opmerking hierboven).

Deze lijn bovenaan in de method zetten, dan moet je slecht 1x de ZoekViaInitialen() oproepen. Nu staat er 2x hetzelfde

Deze lijn bovenaan in de method zetten, dan moet je slecht 1x de ZoekViaInitialen() oproepen.
Nu staat er 2x hetzelfde

Index definieren op LoginCode en op Initialen --> performantie bij ZoekViaFilter()

Index definieren op LoginCode en op Initialen --> performantie bij ZoekViaFilter()

[UST4165] PA-UB-003: Service voorzien om gebruikers aan en af te melden
[UST4165] PA-UB-003: Service voorzien om gebruikers aan en af te melden
[UST4165] PA-UB-003: Service voorzien om gebruikers aan en af te melden

- Header toegevoegd aan response (beter voor later)

    • -0
    • +37
    ./ResponseHeader.cls.xml
  1. … 5 more files in changeset.