CacheAdminA_trunk2010/vhUnitTest/APPS/EC/Verkoop/impl/Externe/VerkoopService

MaakOfferte

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 27 Sep 16
Commits this week: 0
Total Lines of Code (LoC): 65
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
Friday 03 Feb 2017
2:07 pm

JoC

[UST2175] : Vh4_SVC : Test services klaar hebben voor intracto

- Restandje van het verwijderen van een TECH.Mockable implementatie van een interface klasse

Options
    • -15
    • +13
    ./Test.cls.xml
Tuesday 17 Jan 2017
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.

10:46 am

KurtC

[UST1939] Vh4_SVC: MaakWinkelkar

- Raiseevent voor OfferteGemaakt is verhuisd naar VerkoopService.

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.

Options
    • -2
    • +5
    ./Test.cls.xml
  2. … 7 more files in changeset.
Thursday 12 Jan 2017
12:03 pm

Jo Claes

RaiseEventOfferte is een implementatie detail en hoort niet op een interface klasse Naamgeving is niet ok om het gaat om een OfferteGemaaktEvent Beter om een EventRaiser klasse te gebruiken ( zoal...

RaiseEventOfferte is een implementatie detail en hoort niet op een interface klasse
Naamgeving is niet ok om het gaat om een OfferteGemaaktEvent

Beter om een EventRaiser klasse te gebruiken ( zoals daar is : BL.Derde.Agenda.event.EventRaiser )

Verder is het niet nodig om deze klasse expliciet te gaan uitmocken daar in het testing-framework het gehele pubsub gedeelte uitgemockt is.

Monday 09 Jan 2017
11:00 am

Kurt Callebaut

  • More
  • CR-3
  • started review
[UST1939] Vh4_SVC: MaakWinkelkar
[UST1939] Vh4_SVC: MaakWinkelkar
Saturday 07 Jan 2017
12:31 pm

Kurt Callebaut

Melding is veraglemeend naar "ExterneId x bestaat reeds". Daar er op data niveau geen relatie is tussen offerte en winkelkar, moet er een controle voor beide offerte en winkelkar gebeuren. Reposito...

Melding is veraglemeend naar "ExterneId x bestaat reeds".
Daar er op data niveau geen relatie is tussen offerte en winkelkar, moet er een controle voor beide offerte en winkelkar gebeuren. Repository geeft enkel bestaat viaexterneId en het is niet de bedoeling de repository een foutmelding te laten genereren. De meldingen zitten dus op niveau van de services. 1 controle enkel op ofwel offerte of winkelkar lijkt met gevaarlijk omdat het op data niveau niet opgevangen wordt. We willen ook geen dependency creeëren tussen offerte service en winkelkar service. Vandaar elk hun controle; dus de plaats waar de controles samenkomen is dan op verkoopservice niveau. Later misschien te refactoren.

Friday 06 Jan 2017
3:27 pm

Jo Claes

  • More
  • CR-3
  • resumed reviewing
3:26 pm

KurtC

[UST1939] Vh4_SVC: MaakWinkelkar

- RaiseOfferteEvent hernoemt naar RaiseOfferteGemaaktEvent.

Options
    • -1
    • +1
    ./Test.cls.xml
  2. … 4 more files in changeset.
3:11 pm

Kurt Callebaut

VerkoopService gebruikt geen repositories op dit moment. OfferteBewaarder niet vervangen door repository omdat dit op veel plaatsen zo gebruikt wordt, dan is er meer refactorinbg nodig dan enkel de...

VerkoopService gebruikt geen repositories op dit moment.
OfferteBewaarder niet vervangen door repository omdat dit op veel plaatsen zo gebruikt wordt, dan is er meer refactorinbg nodig dan enkel de verkoopservice.

3:07 pm

KurtC

[UST1939] Vh4_SVC: MaakWinkelkar

- Raiseentnaam gewijzigd, Eventraiser naar aparte file.

Options
    • -1
    • +1
    ./Test.cls.xml
  2. … 5 more files in changeset.
2:06 pm

Kurt Callebaut

Idem als LegeKenmerken. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Idem als LegeKenmerken.

2:05 pm

Kurt Callebaut

Okay

Okay

1:59 pm

Kurt Callebaut

Winkelkar was hier niet van belang aangezien de wijzigingen en testen enkel over externid gaan. Testen zijn zoals besproken aangepast naar 1 test waarin alle controles gebeuren op de gebruikte meth...

Winkelkar was hier niet van belang aangezien de wijzigingen en testen enkel over externid gaan. Testen zijn zoals besproken aangepast naar 1 test waarin alle controles gebeuren op de gebruikte methods binnen de MaakOfferte.

1:54 pm

KurtC

[UST1939] Vh4_SVC: MaakWinkelkar

- Na review alles controleren/Verwachten in 1 test.

Options
    • -34
    • +22
    ./Test.cls.xml
Thursday 05 Jan 2017
1:16 pm

Jo Claes

Het feit dat er op de DOM.EC.. winkelkar en APPS.VKP.Offerte een ExterneId staat is een intern gegeven dus de controle op winkelkar en offerte horen samen te zitten. Het zou in principe niet kunnen...

Het feit dat er op de DOM.EC.. winkelkar en APPS.VKP.Offerte een ExterneId staat is een intern gegeven dus de controle op winkelkar en offerte horen samen te zitten.
Het zou in principe niet kunnen dat de ExterneId reeds op de winkelkar staat en niet op de offerte dus is eigenlijk een controle op de offerte voldoende.
Dus de foutmelding die we naar de buitenwereld sturen is eerder één van alla "ID reeds gebruik" want intracto heeft geen weet van onze offertes en Winkelkarren.

1:12 pm

Jo Claes

Mock & Indien = Stub

Mock & Indien = Stub

1:11 pm

Jo Claes

Mock met $$$ElkeWaarde not done

Mock met $$$ElkeWaarde not done

1:10 pm

Jo Claes

Wat is de nood van deze klasse ?

Wat is de nood van deze klasse ?

1:09 pm

Jo Claes

OfferteBewaarder mag vervangen worden door OfferteRepository - OfferteBewaarder mag eventueel integraal verwijderd worden als het niet teveel inpakt heeft

OfferteBewaarder mag vervangen worden door OfferteRepository - OfferteBewaarder mag eventueel integraal verwijderd worden als het niet teveel inpakt heeft

11:07 am

Jo Claes

  • More
  • CR-3
  • joined review
9:38 am

Jo Claes

  • More
  • CR-3
  • started review
[UST1939] Vh4_SVC: MaakWinkelkar
[UST1939] Vh4_SVC: MaakWinkelkar
Friday 30 Dec 2016
1:48 pm

KurtC

[UST1939] Vh4_SVC: MaakWinkelkar

- Maakofferte maakt ook nieuwe winkelkar, dus controle externeid ook voor winkelkar toepassen en meegeven bij maakwinkelkar

Options
    • -6
    • +10
    ./Test.cls.xml
  2. … 1 more file in changeset.
Thursday 29 Dec 2016
3:53 pm

KurtC

[UST1939] Vh4_SVC: MaakWinkelkar

- Test externeId bewaard op offerte. (en verduidelijking dummy waarden.)

Options
    • -6
    • +34
    ./Test.cls.xml
3:48 pm

KurtC

[UST1939] Vh4_SVC: MaakWinkelkar

- %id is geefid() geworden

- raiseevent voor offerte naar offerteservice verplaatst om test op maakofferte te kunnen uitvoeren. (test controle op externeid.) hoort daar ook thuis

Options
    • -0
    • +27
    ./Test.cls.xml
  2. … 6 more files in changeset.
Tuesday 27 Sep 2016
10:03 am

Tom Vermeulen

[UST1834] vhintra gebruikt externeverkoopservice

- De testjes van ExterneVerkoopService verplaatsen

- Aangezien ik niet alle tests ken waarop dit invloed kan hebben (heb er wel een paar gedraaid), zal ik straks op het bord wel zien wat nog aangepast dient te worden. (of zoals in vervlogen tijden werd gezegd: "Commite eos. Novit enim Dashboardus qui sunt eius"

Options
  1. ./MaakOfferte
    • -0
    • +17
    ./Test.cls.xml
  3. … 58 more files in changeset.