CacheAdminA_trunk2010/vhUnitTest/DOM/EC/Winkelkar/impl/WinkelkarRepository

Test.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 09 Jan 17
Commits this week: 0
Total Lines of Code (LoC): 49
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 17 Jan 2017
11:20 am

Kurt Callebaut

11:20 am

Kurt Callebaut

Uitzoeken landcodes ik maak hiervoor een kaartje aan. -> besproken met PVR. Idee om landen aan te bieden net zoals bij unishop is ok. Wordt later nog uitgewerkt.

Uitzoeken landcodes ik maak hiervoor een kaartje aan. -> besproken met PVR. Idee om landen aan te bieden net zoals bij unishop is ok. Wordt later nog uitgewerkt.

10:48 am

Kurt Callebaut

10:48 am

Kurt Callebaut

Bedenking : GemaaktofferteEvent zou zo dicht mogelijk bij het dataobject moeten zitten zodat een event geraiset wordt bij het aanmaken van een offerte, aangemaakt vanuit eender welk stuk code. (bv ...

Bedenking : GemaaktofferteEvent zou zo dicht mogelijk bij het dataobject moeten zitten zodat een event geraiset wordt bij het aanmaken van een offerte, aangemaakt vanuit eender welk stuk code. (bv op repository) Indien er enkel geraiset wordt vanuit verkoopservice, dan heeft de event waarschijnlijk een andere bedoeling/betekenis.

9:53 am

Jo Claes

  • More
  • CR-7
  • resumed reviewing
9:53 am

Jo Claes

  • More
  • CR-7
  • started review
[UST1960] Vh4_SVC : BereidVoorBestelling
[UST1960] Vh4_SVC : BereidVoorBestelling
Monday 16 Jan 2017
2:14 pm

Kurt Callebaut

2:14 pm

Kurt Callebaut

thx for the feedback.

thx for the feedback.

2:09 pm

Kurt Callebaut

Dan compileert de klasse niet.

Dan compileert de klasse niet.

2:08 pm

Kurt Callebaut

Dan compileert de klasse niet.

Dan compileert de klasse niet.

2:08 pm

Kurt Callebaut

Dan compileert de klasse niet.

Dan compileert de klasse niet.

2:01 pm

Kurt Callebaut

Methods ....ViaExterneId refactored naar ViaID voor Vhisie4.WInkelkar.... Repositories

Methods ....ViaExterneId refactored naar ViaID voor Vhisie4.WInkelkar.... Repositories

2:01 pm

Kurt Callebaut

Hierboven staat GeefViaId ? Methods ....ViaExterneId refactored naar ViaID voor Vhisie4.WInkelkar.... Repositories

Hierboven staat GeefViaId ?
Methods ....ViaExterneId refactored naar ViaID voor Vhisie4.WInkelkar.... Repositories

1:22 pm

Kurt Callebaut

Hieronder staat GeefViaID ?

Hieronder staat GeefViaID ?

12:54 pm

Kurt Callebaut

Laat maar staan, dan is het duidelijk dat er op 2 waarden gecontroleerd wordt.

Laat maar staan, dan is het duidelijk dat er op 2 waarden gecontroleerd wordt.

12:48 pm

Kurt Callebaut

OKay.

OKay.

12:48 pm

Kurt Callebaut

Een goeie week geleden heb je gezegd dat we niet offerte id en winkelkar id mogen gebruiken in foutmeldingen omdat het een en dezelfde functionaliteit is. We hebben dus voor externeid gekozen. Dus ...

Een goeie week geleden heb je gezegd dat we niet offerte id en winkelkar id mogen gebruiken in foutmeldingen omdat het een en dezelfde functionaliteit is. We hebben dus voor externeid gekozen. Dus laat externeid maar staan. WinkelkarId is gewoon de testwaarde. Niet blokkerend.

12:44 pm

Kurt Callebaut

Er is gekozen voor deze benaming. Hadden ze op voorhand maar moeten bedenken. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Er is gekozen voor deze benaming.
Hadden ze op voorhand maar moeten bedenken.

Friday 13 Jan 2017
11:09 am

Jo Claes

  • More
  • CR-21
  • finished reviewing
10:59 am

Jo Claes

Overbodig dus

Overbodig dus

10:58 am

Jo Claes

Foutmelding is beter Winkelkar met id 'WinkelkarId' bestaat niet.

Foutmelding is beter Winkelkar met id 'WinkelkarId' bestaat niet.

10:56 am

Jo Claes

Hier zou ik gaan voor de WinkelkarLijnRepositoryMock ipv de LijnRepositoryMock .. als we dan toch voluit gaan...

Hier zou ik gaan voor de WinkelkarLijnRepositoryMock ipv de LijnRepositoryMock .. als we dan toch voluit gaan...

10:55 am

Jo Claes

Ik zou de naam van variabele renamen naar Service ofwel moeten we de methods renamen naar Invoke.. Want het is nu een beetje onduidelijk VerwijderLijn.VerwijderLijn .. klikt een beetje raar ofwel ...

Ik zou de naam van variabele renamen naar Service ofwel moeten we de methods renamen naar Invoke..
Want het is nu een beetje onduidelijk VerwijderLijn.VerwijderLijn .. klikt een beetje raar

ofwel VerwijderLijnService.VerwijderLijn(
ofwel VerwijderLijn.Invoke(

10:52 am

Jo Claes

Dit soort type testen schrijf je ofwel met echte data - data opzetten en kijken of de repo zijn ding doet . ofwel met een spy die op zijn minst de Query prepared - zodat we zeker zijn dat het query...

Dit soort type testen schrijf je ofwel met echte data - data opzetten en kijken of de repo zijn ding doet .
ofwel met een spy die op zijn minst de Query prepared - zodat we zeker zijn dat het query statement kan uitgevoerd worden.

Het gevaar van dit soort testen is dat je uw Query statement in uw code gaat overkopieëren naar uw testen en als er een fout in het statement zit tabel naam dat bijvoorbeeld niet meer bestaat dat merk je het niet op.

10:47 am

Jo Claes

Lijkt mij overbodig

Lijkt mij overbodig

10:47 am

Jo Claes

Lijkt mij overbodig

Lijkt mij overbodig

10:47 am

Jo Claes

Lijkt mij overbodig

Lijkt mij overbodig

10:44 am

Jo Claes

GeefViaID

GeefViaID

10:44 am

Jo Claes

BestaatViaID

BestaatViaID

10:43 am

Jo Claes

Controles zijn niet nodig daar de GeefViaId op de winkelkar repo en de lijn repo de juiste foutmeldingen zullen geven

Controles zijn niet nodig daar de GeefViaId op de winkelkar repo en de lijn repo de juiste foutmeldingen zullen geven