HeaderValidator.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[UW749] Syncen code
  1. … 9 more files in changeset.
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

- Headervalidatie mockable gemaakt en beetje implementatie voorzien

    • -0
    • +18
    ./HeaderValidator.cls.xml
  1. … 5 more files in changeset.